Re: [PATCH] AIX fixincludes for pthread.h

[Bruce Korb]

On Mon, Nov 7, 2011 at 12:59 PM, David Edelsohn <dje....@gmail.com> 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 define as a dual define with the original name now containing
 *  the missing braces.
 */
fix = {
    hackname  = aix_once_init;
    mach      = "*-*-aix*";
    files     = "pthread.h";
    select    = "(#define[ \t]+)(PTHREAD_ONCE_INIT)( \\\\\n"
                "\\{)";

    c_fix       = format;
    c_fix_arg = "%1%2 { %2_INTERNAL }\n%1%2_INTERNAL%3";

    test_text = <<- _EOF_
        #if FUMBLE
        #define PTHREAD_ONCE_INIT \
        { \
               ... _PTH_FLAGS_INIT64, \
        }
        #else
        #define PTHREAD_ONCE_INIT \
        { \
               ... 2, \
        }
        #endif
        _EOF_;
};