This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFA:] more glibc fixinclude adjustments for extern inline
- From: Bruce Korb <bkorb at gnu dot org>
- To: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 15 Jan 2008 11:16:14 -0800
- Subject: Re: [RFA:] more glibc fixinclude adjustments for extern inline
- References: <200801090228.m092SYd3005646@ignucius.se.axis.com>
- Reply-to: bkorb at gnu dot org
Hi Hans-Peter,
How's this? (committed :)
Hans-Peter Nilsson wrote:
>> Date: Wed, 9 Jan 2008 03:16:18 +0100
>> From: Hans-Peter Nilsson <hp@axis.com>
>
>> Index: README
>> ===================================================================
>> --- README (revision 131363)
>> +++ README (working copy)
>> @@ -231,8 +231,8 @@
>>
>> This would wrap a traditional #define inside of a "#ifndef"/"#endif"
>> pair. The second "c_fix_arg" may be omitted *IF* there is
>> - a select clause and the first one matches the text you want
>> - replaced. You may delete text by supplying an empty string for
>> + a "select" clause. The "select" clause is then used as the second
>> + "c_fix_arg". You may delete text by supplying an empty string for
>> the format (the first "c_fix_arg").
>
> or maybe change that last line to
> 'the replacement format (the first "c_fix_arg").'
> (as "the format" could still be confused for "the match").
This is used as follows:
c_fix = format;
c_fix_arg = "#ifndef %1\n%0\n#endif";
c_fix_arg = "#define[ \t]+([A-Z][A-Z0-9a-z_]*).*";
This would wrap a one line #define inside of a "#ifndef"/"#endif"
pair. The second "c_fix_arg" may be omitted *IF* there is a select
clause and the first one identifies the text you wish to reformat.
It will then be used as the second "c_fix_arg". You may delete the
selected text by supplying an empty string for the format (the first
"c_fix_arg").