Re: [PATCH/RFC] gitopt - command-line parsing enhancements

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Junio C Hamano <junkio@...>
Cc: <git@...>
Date: Tuesday, May 9, 2006 - 3:48 pm

Junio C Hamano <junkio@cox.net> wrote:

I wasn't overly happy with the addition of global variables to existing
files and the way they're set (setup_revisions).  But at least they're
static.  Of course, I'm not yet certain that I haven't introduced new
bugs.  All the tests pass, at least...


They look weird at first, but I think they help readability and
maintainability once you get used to them.  They let you focus on the
important part of the function while hiding the boring parts from you.
Quite elegant, imho.

-- 
Eric Wong
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH/RFC] gitopt - command-line parsing enhancements, Eric Wong, (Tue May 9, 3:48 pm)
[PATCH 2/6] update-index: convert to using gitopt, Eric Wong, (Tue May 9, 1:06 am)
[PATCH 3/6] ls-tree: convert to gitopt, Eric Wong, (Tue May 9, 1:06 am)
[PATCH 4/6] ls-files: convert to using gitopt, Eric Wong, (Tue May 9, 1:06 am)