Re: [PATCH v3 01/10] config: Codestyle cleanups.

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Felipe Contreras
Date: Wednesday, February 18, 2009 - 2:18 am

On Tue, Feb 17, 2009 at 6:33 PM, Sverre Rabbelier <srabbelier@gmail.com> wrote:

Hmm, right. I was thinking on Junio and Johanness that already know
the context, so they don't need an introduction, but I forgot about
other people that might be interested in giving this a review.

In any case, this patch series is a try to re-organize the code of
'git config' and the main change is to use parseopt for better
maintainability and option parsing. A few functionality changes are
provided, like the fact that now the arguments don't need to be in a
specific order: (--global -l vs -l --global), and improvements in
error reporting. Otherwise it should work exactly the same as before
(except with a much better --help).

Cheers.

-- 
Felipe Contreras
--
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:
[PATCH v3 01/10] config: Codestyle cleanups., Felipe Contreras, (Tue Feb 17, 6:52 am)
[PATCH v3 02/10] config: Cleanup editor action., Felipe Contreras, (Tue Feb 17, 6:52 am)
[PATCH v3 03/10] config: Make git_config() more flexible., Felipe Contreras, (Tue Feb 17, 6:52 am)
[PATCH v3 05/10] config: Reorganize get_color*., Felipe Contreras, (Tue Feb 17, 6:52 am)
[PATCH v3 06/10] config: Use parseopt., Felipe Contreras, (Tue Feb 17, 6:52 am)
[PATCH v3 09/10] config: Disallow multiple variable types., Felipe Contreras, (Tue Feb 17, 6:52 am)
Re: [PATCH v3 01/10] config: Codestyle cleanups., Sverre Rabbelier, (Tue Feb 17, 9:33 am)
Re: [PATCH v3 01/10] config: Codestyle cleanups., Felipe Contreras, (Wed Feb 18, 2:18 am)
Re: [PATCH v3 03/10] config: Make git_config() more flexible, Felipe Contreras, (Wed Feb 18, 2:30 pm)