On Mon, 3 Dec 2007, [utf-8] Dag-Erling Sm??rgrav wrote:

Olivier Houchard <[EMAIL PROTECTED]> writes:
  Log:
  Change the casts from (pthread_mutex_t *) to (void *) to keep gcc quiet.
  Anybody with a cleaner solution feel free to change it.

Declare dd_lock as a struct pthread_mutex * instead of a void * in
src/include/dirent.h.

dd_lock cannot be declared as struct pthread_mutex *, since pthread_mutex
is not permitted in the application namespace of dirent.h.  dirent.h
is (was) careful about namespaces.  E.g., it keeps out of the application
namespace for its own struct _dirdesc.  This doesn't work for pthread_mutex
since that is misdesigned.  (I think pthread_mutex_t is supposed to
hide the implementation and reduce namespace problems, but struct
pthread was only declared in <pthread.h> where it is permitted in the
namespace, so <pthread.h> doesn't bother naming it struct _pthread_mutex.
Other headers can't define it with a different name (tag) because different
tags give different structs.)

Bruce
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "[EMAIL PROTECTED]"

Reply via email to