On Fri, 3 May 2019, Jakub Jelinek wrote:

> Hi!
> 
> On Thu, May 02, 2019 at 01:04:07PM +0200, Jakub Jelinek wrote:
> > Well, that doesn't answer the question.
> > I was wondering why you couldn't:
> > 
> > 2019-05-02  Jakub Jelinek  <ja...@redhat.com>
> > 
> >     * opts.h (finish_options): Remove lang_mask argument.
> >     (print_help, help_option_argument): Declare.
> >     * opts.c (print_help): Remove forward declaration, no longer static.
> >     (finish_options): Remove lang_mask argument, don't call print_help
> >     here.
> >     * opts-global.c (decode_options): Adjust finish_option caller, call
> >     print_help here.
> 
> On Thu, May 02, 2019 at 01:13:22PM +0200, Martin Liška wrote:
> > On 5/2/19 1:04 PM, Jakub Jelinek wrote:
> > > Well, that doesn't answer the question.
> > > I was wondering why you couldn't:
> > 
> > Ah sorry, you are right. The patch you suggested
> > is obviously nicer than what we have currently in trunk.
> 
> Bootstrapped/regtested successfully now on x86_64-linux and i686-linux, ok
> for trunk?

OK.

Richard.

Reply via email to