Date: Fri, 7 Dec 2007 18:15:13 +1100 (EST) From: Bruce Evans <brde@optusnet.com.au> To: =?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav?= <des@des.no> Cc: cvs-src@freebsd.org, cvs-all@freebsd.org, src-committers@freebsd.org, Olivier Houchard <cognet@freebsd.org>, Bruce Evans <brde@optusnet.com.au> Subject: Re: cvs commit: src/lib/libc/gen closedir.c Message-ID: <20071207181448.P871@delplex.bde.org> In-Reply-To: <86odd4chdn.fsf@ds4.des.no> References: <200711192157.lAJLvSu6057416@repoman.freebsd.org> <868x4beu46.fsf@ds4.des.no> <20071204115427.D3192@delplex.bde.org> <86odd4chdn.fsf@ds4.des.no>
next in thread | previous in thread | raw e-mail | index | archive | help
This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --0-411707526-1197011713=:871 Content-Type: TEXT/PLAIN; charset=X-UNKNOWN; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE On Thu, 6 Dec 2007, [utf-8] Dag-Erling Sm=C3=B8rgrav wrote: > Bruce Evans <brde@optusnet.com.au> writes: >> dd_lock cannot be declared as struct pthread_mutex *, since pthread_mute= x >> is not permitted in the application namespace of dirent.h. dirent.h >> is (was) careful about namespaces. E.g., it keeps out of the applicatio= n >> namespace for its own struct _dirdesc. This doesn't work for pthread_mu= tex >> 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= =2E >> Other headers can't define it with a different name (tag) because differ= ent >> tags give different structs.) > > Since struct pthread_mutex is not part of the API, how about simply > renaming it to struct _pthread_mutex to circumvent the namespace issues? Seems right in theory. Bruce --0-411707526-1197011713=:871--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20071207181448.P871>