Re: [PATCH] New --pe-dll-characteristcs switch for PE ld.

[Christopher Faylor <cgf-use-the-mailinglist-please at sourceware dot org>]

On Mon, Mar 09, 2009 at 06:24:01AM +0000, Dave Korn wrote:
>Christopher Faylor wrote:
>>I don't see any reason for a user to have to remember two things
>>(--dll-characteristics something) when they could just use --something.
>>If implemented as a simple option, you could get useful information
>>from --help, and you wouldn't have to invent a new way of parsing
>>options.
>
>The syntax isn't unprecedented, there are three other options that take
>an comma-separated list for the argument.

I wasn't referring to the comma.  I was referring to the requirement of
parsing suboptions.  Are you saying there are other ld options which
take keywords like this?


>(I added a couple of other separators that made sense because there may
>be integers present.  I also took as an example the strace --mask
>syntax.)

Nevertheless, you added a non-trivial amount of code to deal with
parsing these things as suboptions.

The Cygwin strace option is for a mask which controls what strace
outputs.  The ld option would not represent a mask.  It is just an
arbitrary collection of keyworeds.  You are grouping suboptions together
only because Microsoft decided to have disparate behavior controlled by
one unsigned short field in the PE header.

I think I'm done arguing now because I seem to be repeating myself.

cgf