[patch] cosmetics for configure* in gcc

Andreas Tobler andreast-list@fgznet.ch
Sun Jul 27 23:28:00 GMT 2008


Hi Ralf!

Ralf Wildenhues wrote:
> * Paolo Bonzini wrote on Sun, Jul 27, 2008 at 12:10:46PM CEST:
>>> Ok, for trunk _after_ the important merges have been done?
>> Yes, another possibility is to add a
>>
>> 	m4_bpatsubst([$N], [[ \n\t][ \n\t]*], [ ])
>>
>> in the macros (but \n and \t should be expanded as newline and tab  
>> respectively).  As you prefer.
> 
> Or, even better, use m4_do:
>  ACX_PROG_CC_WARNING_OPTS(
>    [m4_do([-W -Wall -Wwrite-strings  -Wstrict-prototypes ],
>           [-Wmissing-prototypes -Wcast-qual], [loose_warn])
> 

Indeed, from my POV much better, I was looking for something like this.
Thanks for the cool hint.

How about this one then?


Thanks,
Andreas

2008-07-27  Andreas Tobler  <a.tobler@schweiz.org>

	* configure.ac: Use the m4_do macro to concatenate the warnings into
	one string in ACX_PROG_CC_WARNING_OPTS,
	ACX_PROG_CC_WARNING_ALMOST_PEDANTIC and ACX_PROG_CC_WARNINGS_ARE_ERRORS.
	* configure: Regenerate.

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: config_ws1.diff
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20080727/a4975d24/attachment.ksh>


More information about the Gcc-patches mailing list