[PATCH v0 1/3] gas: minor reformatting in command line help and doc
Matthieu Longo
Matthieu.Longo@arm.com
Fri Jun 28 18:14:00 GMT 2024
>On 27.06.2024 15:50, Matthieu Longo wrote:
>>
>> - help message: add missing comma between the short and long option
>
>Why "missing"? IOW who or what says there ought to be commas? Looking at just objcopy, objdump, and readelf, it's objdump being the outlier in having commas there.
ld, nm have commas as well. Python argparse also prints the short and long options separated with a comma. Same for git. The commas seem a bit more common.
Gas has currently a mix of both, so I opted for the comma between short and long options.
Do you have any objection to keep commas here ?
>You also move -W / --no-warn down for no apparent reason. Them being ahead of --warn is more logical imo, as warnings are on by default.
>(Same for their handling, where I view the resulting code churn as yet less relevant.)
I will change it in the next revision.
>> - as doc:
>> - brief summary of how to invoke gas: separate [-w] [-x] on a new line as those
>> two options have nothing to do with the warning options.
>> - reordering of the warning options to have the same order as the listing.
>
>Except that in order to "need" to do that you first had to change the listing.
>
>> - no-warn option description: change an "and" to a "or", as it is
>> either the short
>
>Along with that I think "options" there wants to change to singular.
>
>Jan
More information about the Binutils
mailing list