Re: recent commit broke src/dired.c on OSX

["Jan Djärv" <jan.h.d@swipnet.se>]

Juanma Barranquero skrev 2010-07-25 19.30:
> On Sun, Jul 25, 2010 at 19:23, Randal L. Schwartz<merlyn@stonehenge.com>  wrote:
>
>>     -extern DIR *opendir ();
>>     -extern struct direct *readdir ();
>>     +extern DIR *opendir (char *);
>>     +extern struct direct *readdir (DIR *);
>
>> Needs to be backed out, or made more portable.
>
> Sure. I'd prefer to make it more portable (that already compiles on
> Windows / MinGW and Ubuntu / GCC).
>
> What is the right prototype in your system? (You didn't attach the
> error output, BTW). Could you send a patch with the relevant #ifdef?
>

AFAIK, the correct prototype for opendir is

   DIR *opendir(const char *);

even on Ubunt, as well as OSX and of course POSIX and Single Unix 
Specification.  But on GNU/Linux SYSV_SYSTEM_DIR is defined, but not on OSX. 
So on GNU/Linux, the wrong prototype is not seen by the compiler.

I think it is an error to extern declare system functions, the proper include 
file should be used instead.
All the #ifdefs that try to figure out if dirent.h (the POSIX-mandated header 
file for opendir) is present should be replaced with a configure check for 
dirent.h.  Then SYSV_SYSTEM_DIR can go away.

	Jan D.