Skip to content

Commit d57acb7

Browse files
committed
mingw: do not special-case .exe files anymore
Since baaf233 (connect: improve check for plink to reduce false positives, 2015-04-26), t5601 writes out a `plink.exe` for testing that is actually a shell script. So the assumption that the `.exe` extension implies that the file is *not* a shell script is now wrong. The original idea to special-case `.exe` files was probably to help performance, but since we are in a code path that involves spawning a new process (which in and of itself is pretty slow on Windows anyway), we pursue a better idea to improve performance elsewhere: we try to convert scripts into builtins and to reduce the number of spawned processes by adding more internal API calls. Signed-off-by: Johannes Schindelin <[email protected]>
1 parent 333053c commit d57acb7

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

compat/mingw.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -853,11 +853,6 @@ static const char *parse_interpreter(const char *cmd)
853853
char *p, *opt;
854854
int n, fd;
855855

856-
/* don't even try a .exe */
857-
n = strlen(cmd);
858-
if (n >= 4 && !strcasecmp(cmd+n-4, ".exe"))
859-
return NULL;
860-
861856
fd = open(cmd, O_RDONLY);
862857
if (fd < 0)
863858
return NULL;

0 commit comments

Comments
 (0)