Re: [PATCH] tools/winedump: Change duplicate flag.

[Austin English]

On Apr 7, 2017 2:22 PM, "Zebediah Figura"  wrote:

On 04/07/2017 02:09 PM, Alexandre Julliard wrote:
> Zebediah Figura  writes:
> > diff --git a/tools/winedump/main.c b/tools/winedump/main.c
> > index a4ebb43..5c683f6 100644
> > --- a/tools/winedump/main.c
> > +++ b/tools/winedump/main.c
> > @@ -226,7 +226,7 @@ static const struct my_option option_table[] = {
> >    {"-v",    SPEC, 0, do_verbose,  "-v              Show lots of detail

while working (verbose)."},
> > {"dump",  DUMP, 0, do_dump,     "dump <file>     Dump the contents of

'file' (dll, exe, lib...)"},
> > {"-C",    DUMP, 0, do_symdmngl, "-C              Turn on symbol

demangling"},
> > - {"-f",    DUMP, 0, do_dumphead, "-f              Dump file header

information"},
> > - {"-F",    DUMP, 0, do_dumphead, "-F              Dump file header

information"},
> Not a big deal, but is there a reason for changing this?

The -f flag is assigned to two different options, yielding this one
unreachable.

Please also update the manpage.

Thanks for catching this!