On Thu, Jul 19, 2018 at 03:25:15PM +0200, Martin Liška wrote:
> Few weeks ago I added new Deprecated flag for options. Apparently, there's
> one similar called Ignore. Thus I moved all Deprecated to Ignore and for
> all Ignored I do a warning of following format:
>
> $ xgcc: warning: switch ‘-mmpx’ is no longer supported
Is that what we want for all the Ignore options? Looking at {,*/}*.opt,
I see a lot of options that have those Warn(switch %qs is no longer supported)
and a lot of them that do not, while with your patch it will now warn all.
Especially when it is a warning without corresponding -W... option that can
be quite nasty.
Wouldn't be better to just remove Deprecated and keep the Ignore behavior it
had? Or make Deprecated effectively an alias for
Ignore Warn(switch %qs is no longer supported)
and use it for those options that did that?
Jakub
