Re: [RFC] Re: Convert 'git blame' to parse_options()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Mon, 23 Jun 2008, Jeff King wrote:
> 
> There is already a discussion underway about the proper solution.

No there isn't, not any I have seen.

The recursion discussion has been going on for ages. Nothing happened, 
because people claimed it was too complex. And it doesn't solve all the 
_other_ issues where the current behaviour is just painful.

That said, I don't care how this gets fixed. If you want to fix it some 
other way, fine. Just _do_ it, dammit. And make the interface easier to 
use for incremental changes of existing code.

Because no, I don't care about builtin-blame.c in particular either. But I 
*do* care about the fact that currently it's sometimes a total *bitch* to 
convert the trivial ad-hoc argument parsing to use parse_options(). It 
takes effort and thinking.

And I seriously doubt that recursive or anything else will help that. But 
I _guarantee_ that an incremental mode will make it much much easier to do 
partial conversions of all the easy cases. And _that_ is what I care 
about.

		Linus
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux