Hi,

On Mon, 2008-03-03 at 21:49 +0100, Carsten Hey wrote:
> please allow " severity 1234 minor, " instead of " severity 1234 minor , "

The major problem with supporting the syntax is that it requires either
breaking currently supported functionality, or introducing inconsistency
in the way different options are parsed.

Specifically:

bts retitle 469210 [bts] please allow severity 1234 minor, reassign it general 
, tag it moreinfo

is supported syntax for retitling the bug and then tagging it (but /not/
reassigning). If we treat "minor," and "minor ," as equivalent then the
command above will no longer work as expected (it will become three
operations rather than two).

An alternative would be to special-case the "retitle" command so that it
must always be terminated with " , " (or " . ") but that would then be
inconsistent.

[I realise the example is somewhat convoluted but that doesn't change
the general point imho]

Regards,

Adam



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to