On 2020-Sep-04, Thomas Munro wrote:

> @@ -10,6 +10,7 @@ struct dirent
>  {
>       long            d_ino;
>       unsigned short d_reclen;
> +     unsigned char d_type;
>       unsigned short d_namlen;
>       char            d_name[MAX_PATH];
>  };
> @@ -20,4 +21,26 @@ DIR                   *opendir(const char *);
>  struct dirent *readdir(DIR *);
>  int                  closedir(DIR *);
>  
> +/* File types for 'd_type'. */
> +enum
> +  {
> +     DT_UNKNOWN = 0,
> +# define DT_UNKNOWN          DT_UNKNOWN

Uhm ... what do these #defines do?  They look a bit funny.

Would it make sense to give this enum a name, and then use that name in
struct dirent's definition, instead of unsigned char?

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


Reply via email to