Messages in this thread |  | | Date | Wed, 20 Nov 2013 09:16:46 +0200 | From | Adrian Hunter <> | Subject | Re: [PATCH 2/4] perf tools: Allow '--inherit' as the negation of '--no-inherit' |
| |
On 19/11/13 18:20, David Ahern wrote: > On 11/19/13, 1:12 AM, Adrian Hunter wrote: >> On 18/11/13 23:15, David Ahern wrote: >>> Why not cleanup the options for the commands and move all of the no-xxxx to >>> just xxxx? Anyone using no-xxxx would still just work by the existing code. >> >> Interesting idea but the short and long options are a combination, and short >> options don't work the same way e.g. -i and --no-inherit go together. >> > > If inherit is on by default and -i / --inherit is the option then -i is a > no-op and --no-inherit disables it. That's the same effect has having > inherit on by default and an option to disable it called --no-inherit.
Well, if you are going to change the meaning of -i and deny the users a short option to disable inheritance, then it is no longer a cleanup - it is a functional change.
|  |