Eli Zaretskii wrote:
> Would it be okay to use 'unsigned' instead of 'uint8_t' on all
> platforms?

Yes, this is a good idea. Done:


2018-05-12  Bruno Haible  <br...@clisp.org>

        glob: Choose 'dirent_type' in a way that works better on mingw.
        Reported and suggested by Eli Zaretskii <e...@gnu.org>.
        * lib/glob.c (dirent_type): Define as uint_fast32_t.

diff --git a/lib/glob.c b/lib/glob.c
index 61930e7..c4ba994 100644
--- a/lib/glob.c
+++ b/lib/glob.c
@@ -78,7 +78,9 @@
 
 static const char *next_brace_sub (const char *begin, int flags) __THROWNL;
 
-typedef uint_fast8_t dirent_type;
+/* The type of ((struct dirent *) 0)->d_type is 'unsigned char' on most
+   platforms, but 'unsigned int' in the mingw from mingw.org.  */
+typedef uint_fast32_t dirent_type;
 
 #if !defined _LIBC && !defined HAVE_STRUCT_DIRENT_D_TYPE
 /* Any distinct values will do here.


Reply via email to