Re: [PATCH] AIX fixincludes for pthread.h

2011-11-08 Thread Bruce Korb
Hi David, On Tue, Nov 8, 2011 at 10:20 AM, David Edelsohn wrote: >>> Okay? >> >> Well, okay, but > > Also, there are other fixincludes fixes for missing braces that > directly add the braces, not a wrapper.  They are not multi-line > macros, but there is a precedent on the style of solution.

Re: [PATCH] AIX fixincludes for pthread.h

2011-11-08 Thread David Edelsohn
On Tue, Nov 8, 2011 at 11:56 AM, Bruce Korb wrote: > On Mon, Nov 7, 2011 at 12:59 PM, David Edelsohn wrote: >> Okay? > > Well, okay, but I'd prefer one fix something like the following (and I confess > I've not had any test time...) because it looks simpler to me: Also, there are other fixinclud

Re: [PATCH] AIX fixincludes for pthread.h

2011-11-08 Thread David Edelsohn
Bruce, I don't want to add a layer and PTHREAD_ONCE_INIT_INTERNAL. Thanks, David

Re: [PATCH] AIX fixincludes for pthread.h

2011-11-08 Thread Bruce Korb
On Mon, Nov 7, 2011 at 12:59 PM, David Edelsohn wrote: > Okay? Well, okay, but I'd prefer one fix something like the following (and I confess I've not had any test time...) because it looks simpler to me: /* * pthread.h on AIX defines PTHREAD_ONCE_INIT without enough braces. * Rewrite the de