Re: 7.6.40

In the last episode (Oct 24), Marcus Comstedt (ACROSS) (Hail Ilpalazzo!) @ Pike (-) developers forum said:

Line 969 in modules/files/efuns.c:

969 if ((err = readdir_r(dir, tmp, &d)) || !d) {

This does not seem right. readdir_r only takes two arguments on Solaris, the second of which should not be NULL (which is the value "tmp" has here).

You can tell Solaris to use the 3-args readdir_r by defining _POSIX_PTHREAD_SEMANTICS (or defining _POSIX_C_SOURCE to a value >= 199506L). Apparently this option has existed since Solaris 2.5.

http://docs.sun.com/app/docs/doc/816-5168/6mbb3hrm4?a=view

Using this flag might be a good idea for Pike 7.7 (it'd probably cause too many code changes to go into 7.6).