Re: [PATCH v2 48/94] builtin/apply: rename 'prefix_' parameter to 'prefix'

On Sat, May 14, 2016 at 8:27 PM, Junio C Hamano wrote: > Christian Couder writes: > >> On Thu, May 12, 2016 at 10:43 PM, Junio C Hamano wrote: >>> Junio C Hamano writes: >>> Up to this point, the

Re: [PATCH v2 48/94] builtin/apply: rename 'prefix_' parameter to 'prefix'

On Fri, May 13, 2016 at 9:42 PM, Christian Couder wrote: > On Thu, May 12, 2016 at 9:56 PM, Junio C Hamano wrote: >> Christian Couder writes: >> >>> This is just a small cleanup. >> >> ... which may have been better

Re: [PATCH v2 48/94] builtin/apply: rename 'prefix_' parameter to 'prefix'

Christian Couder writes: > On Thu, May 12, 2016 at 10:43 PM, Junio C Hamano wrote: >> Junio C Hamano writes: >> >>> Up to this point, the conversion looks quite sensible, even though I >>> think the organization of fields in

Re: [PATCH v2 48/94] builtin/apply: rename 'prefix_' parameter to 'prefix'

On Thu, May 12, 2016 at 10:43 PM, Junio C Hamano wrote: > Junio C Hamano writes: > >> Up to this point, the conversion looks quite sensible, even though I >> think the organization of fields in apply_state do not look logical. > > I'd stop here for now, as

Re: [PATCH v2 48/94] builtin/apply: rename 'prefix_' parameter to 'prefix'

On Thu, May 12, 2016 at 9:56 PM, Junio C Hamano wrote: > Christian Couder writes: > >> This is just a small cleanup. > > ... which may have been better happened at 09/94. Ok, I will squash it in 09/94. -- To unsubscribe from this list: send the

Re: [PATCH v2 48/94] builtin/apply: rename 'prefix_' parameter to 'prefix'

Junio C Hamano writes: > Up to this point, the conversion looks quite sensible, even though I > think the organization of fields in apply_state do not look logical. I'd stop here for now, as everything before this step looks uncontroversial. Anybody whose tasked to move the

Re: [PATCH v2 48/94] builtin/apply: rename 'prefix_' parameter to 'prefix'

Christian Couder writes: > This is just a small cleanup. ... which may have been better happened at 09/94. Up to this point, the conversion looks quite sensible, even though I think the organization of fields in apply_state do not look logical. -- To unsubscribe