Re: fixincludes for darwin7.9.0 long double

[Geoff Keating <geoffk at geoffk dot org>]

On 15/07/2005, at 7:35 AM, Bruce Korb wrote:

> Eric wrote:
>
> > > This fixincludes patch should fix the problem with long double
> > > functions on Darwin 7.9.0 / Mac OS 10.3.9.
> >
> > I'm seeing this on x86-64:
>
> With the test result addition and one little suggested tweak:
>
> +fix = {
> +  hackname  = AAB_darwin7_9_long_double_funcs_2;
> +  mach      = "*-*-darwin7.9*";
> +  files     = math.h;
> +  select    = '#include[ \t]+\"';
> +  c_fix     = format;
> +  c_fix_arg = "%1<%2.h>";
> +
> +  c_fix_arg = '([ \t]*#[ \t]*include[ \t]+)"([a-z0-9/]+)\.h"';
>
> The file name expression ought to be:  ([a-z0-9_-]+)\.h
> yes?  Assuming that hyphenated or underscore-separated names
> are possible.  Capital letters seem unlikely.

It really only has to match architecture/ppc/math.h, since that's the  
file being fixed by the previous rule.  If any fix at all is applied  
to math.h, all the "" includes will be fixed by other places in  
fixincludes, but without this rule math.h was being left alone, so  
the previous rule won't work.

Thanks for fixing the tests!  I couldn't work out what needed to be  
done.

> +  test_text = '#include "architecture/ppc/math.h"';
> +};
>
> Cheers - Bruce

Attachment: smime.p7s
Description: S/MIME cryptographic signature