Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH v2] builtin/receive-pack.c: use parse_options API

From: Junio C Hamano <hidden>
Date: 2016-06-15 23:08:35

Sidhant Sharma [off-list ref] writes:
quoted
Matthieu Moy [off-list ref] writes:
quoted
Thanks. This version looks good to me.
I'll queue this with your "Reviewed-by:" to 'pu', just as a
Microproject reward ;-).  Given that the program will never see an
interactive use from a command line, however, I am not sure if it is
worth actually merging it down thru 'next' to 'master'.
Having said that, the resulting code looks more modular in that
adding a new option or extending the behaviour of an existing option
may be easier with the patch going forward, so after all we might
be better off taking it to the production.  I'll think about it a
bit more and decide.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help