From: SourceForge.net <no...@so...> - 2004-06-18 20:38:09
|
Patches item #966053, was opened at 2004-06-03 15:47 Message generated for change (Comment added) made by dgp You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=310894&aid=966053&group_id=10894 Category: 10. Objects Group: None >Status: Closed >Resolution: Accepted Priority: 5 Submitted By: Don Porter (dgp) >Assigned to: Don Porter (dgp) Summary: stop repeated findExecutable searches Initial Comment: Patch includes two improvements. 1) In TclpFindExecutable() note that successful searches have been saved and returned forever. Only one search permitted. On the other hand, failed searches have been invited to come back and spend time and effort failing over and over again. The changes to tclUnixFile.c make it so that a failure result is also saved and returned forever without the expense of repeating the search operations. 2) In tclEncoding.c, I've renamed TclFindEncodings to FindEncodings to match its status as a file static routine. That makes it clearer that the only caller is Tcl_FindExecutable, and that caller has already called TclpFindExecutable, so there's no need for FindEncodings to do it again. (The source of some of the repetition addressed in 1) ). Without the need to call TpFE, FindEncodings also needed no argument, so it was removed. ---------------------------------------------------------------------- >Comment By: Don Porter (dgp) Date: 2004-06-18 16:38 Message: Logged In: YES user_id=80530 applied to HEAD. ---------------------------------------------------------------------- Comment By: Don Porter (dgp) Date: 2004-06-03 16:28 Message: Logged In: YES user_id=80530 ok, just a test command that needed an update. Here's the revised patch. ---------------------------------------------------------------------- Comment By: Don Porter (dgp) Date: 2004-06-03 16:06 Message: Logged In: YES user_id=80530 hhmmpph, well that patch makes 5 tests fail. Investigating... ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=310894&aid=966053&group_id=10894 |