[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Bug-gnu-arch] Re: [bug #5900] Use -3 as the short option for --three-wa
From: |
Tom Lord |
Subject: |
[Bug-gnu-arch] Re: [bug #5900] Use -3 as the short option for --three-way, instead of -t |
Date: |
Sun, 12 Oct 2003 18:56:40 -0700 (PDT) |
> From: Miles Bader <address@hidden>
> On Sun, Oct 12, 2003 at 08:29:49PM -0400, address@hidden wrote:
> > Given that you mush all your changes into a single line instead of
giving
> > me clean changesets to merge from (and that I generally just star-merge
> > from you in batches rather than cherry picking):
> Hey that's not fair -- my changesets are very clean. If you want multiple
> branches to merge from, you gotta give some hint as to how you want things
> split up...
Chill, chill.
It's a dinky little program, really. Sure, it'd be swell to have a
separate branch for each logical change etc..... but it'd be
affectatious overkill for this source base. That wasn't a complaint
your responding too -- just an observation to provide context.
(Your changes are generally great. For the most part, the biggest
complaint I have is occaisional code formatting nits -- but I've known
you too long to think they're worth pointing out :-)
> > As a coding convention, tla does not use decimal digits as option names.
> > Do we have to have a debate on why or will you please just withdraw this
> > change?
> Certainly I'll revert it.
Thanks.
> Care to say why the rule though (I've never heard that particular one
before)?
I dunno. Well, ok -- really I don't want to be pinned down as having
just a single definitive argument. Some vague reasons:
* -[digit]+
is an argument syntax for passing an arbitrary integer, and that's
potentially useful -- but not if you take over some -[digit] for an
option.
* it's too fucked up
Too many programs handle -[digit]* too differently. It's a fucked
up area of potential CLI syntax -- so I just like to avoid it
entirely. (e.g., the unfortunate confusion between `-' as "here's
an option" and `-' as "negative number, coming up.")
-t