commit 4654c32122bc05b4ca8b6d117c5a490a399500c6
author Andreas Mohr <amohr@student.ei.uni-stuttgart.de> Sun Feb 20 19:15:34 2000 +0000
committer Alexandre Julliard <julliard@winehq.org> Sun Feb 20 19:15:34 2000 +0000
tree cfa2b8542cfa6fbc2ce84d1d281f2c4d7a7cbfbf
parent ec7d7fa3804474fe496ccea9ef40a3c85504f55b

It seems to be GetModuleFileName16 that checks exe version on whether
to return long or short paths, not GetModuleFileNameA.

loader/module.c

diff --git a/loader/module.c b/loader/module.c
index 53ea615..bac3f6a 100644
--- a/loader/module.c
+++ b/loader/module.c
@@ -1296,6 +1296,11 @@
 
 /***********************************************************************
  *              GetModuleFileNameA      (KERNEL32.235)
+ *
+ * GetModuleFileNameA seems to *always* return the long path;
+ * it's only GetModuleFileName16 that decides between short/long path
+ * by checking if exe version >= 4.0.
+ * (SDK docu doesn't mention this)
  */
 DWORD WINAPI GetModuleFileNameA( 
 	HMODULE hModule,	/* [in] module handle (32bit) */
@@ -1307,10 +1312,7 @@
     if (!wm) /* can happen on start up or the like */
     	return 0;
 
-    if (PE_HEADER(wm->module)->OptionalHeader.MajorOperatingSystemVersion >= 4.0)
-      lstrcpynA( lpFileName, wm->filename, size );
-    else
-      lstrcpynA( lpFileName, wm->short_filename, size );
+    lstrcpynA( lpFileName, wm->filename, size );
        
     TRACE("%s\n", lpFileName );
     return strlen(lpFileName);