Skip to content

Commit ac9a365

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 de6ed36 commit ac9a365

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
@@ -836,11 +836,6 @@ static const char *parse_interpreter(const char *cmd)
836836
char *p, *opt;
837837
int n, fd;
838838

839-
/* don't even try a .exe */
840-
n = strlen(cmd);
841-
if (n >= 4 && !strcasecmp(cmd+n-4, ".exe"))
842-
return NULL;
843-
844839
fd = open(cmd, O_RDONLY);
845840
if (fd < 0)
846841
return NULL;

0 commit comments

Comments
 (0)