Re: [Patch, option handling] optc-gen.awk - support || in EnabledBy()

2014-11-28 Thread Joseph Myers
On Sun, 23 Nov 2014, Tobias Burnus wrote: Build on x86-64-gnu-linux with checking that the resulting options.c remains the same. OK for the trunk? OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [Patch, option handling] optc-gen.awk - support || in EnabledBy()

2014-11-23 Thread Tobias Burnus
Manuel López-Ibáñez wrote: For what is worth, the patch looks good to me, except that the form EnabledBy(@var{opt} || @var{opt2} needs documentation in doc/options.texi. Done. Perhaps it could be more condensed as (untested!): Seems to work - hence: Done. A version handling

[Patch, option handling] optc-gen.awk - support || in EnabledBy()

2014-11-22 Thread Tobias Burnus
In fortran/*.opt, I'd like to use: Fortran Var(warn_tabs) Warning EnabledBy(Wall || Wpedantic) However, || is not supported by EnabledBy. This patch adds it. I checked that option.c remains the same after the patch. I attached the patch twice: Once with -w -U16 to show the modification

Re: [Patch, option handling] optc-gen.awk - support || in EnabledBy()

2014-11-22 Thread Manuel López-Ibáñez
For what is worth, the patch looks good to me, except that the form EnabledBy(@var{opt} || @var{opt2} needs documentation in doc/options.texi. Perhaps it could be more condensed as (untested!): Index: optc-gen.awk === ---