(readdir): If FindFirstFile/FindNextFile return in cFileName a file name that

includes `?' characters, use the 8+3 alias in cAlternateFileName instead.

diff --git a/src/ChangeLog b/src/ChangeLog
index 675fcaaaa54f74ef1800cff879db1641324c9596..f07cd43112964c1b790803e446a41dc4bef4d691 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,9 @@
+2008-03-26  Eli Zaretskii  <eliz@gnu.org>
+
+       * w32.c (readdir): If FindFirstFile/FindNextFile return in
+       cFileName a file name that includes `?' characters, use the 8+3
+       alias in cAlternateFileName instead.
+
 2008-03-26  Chong Yidong  <cyd@stupidchicken.com>
 
        * Version 22.2 released.

diff --git a/src/w32.c b/src/w32.c
index 8a970e0c25e47c52157e602475e2a5016ff0e03a..e40f9a4b289f54e475db8a6f3f5e56fdc04cb9c0 100644 (file)
--- a/src/w32.c
+++ b/src/w32.c
@@ -1779,6 +1779,8 @@ closedir (DIR *dirp)
 struct direct *
 readdir (DIR *dirp)
 {
+  int downcase = !NILP (Vw32_downcase_file_names);
+
   if (wnet_enum_handle != INVALID_HANDLE_VALUE)
     {
       if (!read_unc_volume (wnet_enum_handle,
@@ -1816,11 +1818,23 @@ readdir (DIR *dirp)
   dir_static.d_reclen = sizeof (struct direct) - MAXNAMLEN + 3 +
     dir_static.d_namlen - dir_static.d_namlen % 4;
 
-  dir_static.d_namlen = strlen (dir_find_data.cFileName);
-  strcpy (dir_static.d_name, dir_find_data.cFileName);
+  /* If the file name in cFileName[] includes `?' characters, it means
+     the original file name used characters that cannot be represented
+     by the current ANSI codepage.  To avoid total lossage, retrieve
+     the short 8+3 alias of the long file name.  */
+  if (_mbspbrk (dir_find_data.cFileName, "?"))
+    {
+      strcpy (dir_static.d_name, dir_find_data.cAlternateFileName);
+      /* 8+3 aliases are returned in all caps, which could break
+        various alists that look at filenames' extensions.  */
+      downcase = 1;
+    }
+  else
+    strcpy (dir_static.d_name, dir_find_data.cFileName);
+  dir_static.d_namlen = strlen (dir_static.d_name);
   if (dir_is_fat)
     _strlwr (dir_static.d_name);
-  else if (!NILP (Vw32_downcase_file_names))
+  else if (downcase)
     {
       register char *p;
       for (p = dir_static.d_name; *p; p++)