Christian Couder <christian.couder@xxxxxxxxx> writes: > Signed-off-by: Christian Couder <chriscool@xxxxxxxxxxxxx> > --- > builtin/apply.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/builtin/apply.c b/builtin/apply.c > index 506357c..c45e481 100644 > --- a/builtin/apply.c > +++ b/builtin/apply.c > @@ -57,6 +57,8 @@ struct apply_state { > int unidiff_zero; > > int update_index; > + > + int unsafe_paths; > }; Having said I like the way this series moves only a few variables at a time to limit the scope of each step. it gets irritating to see all these unnecessary blank lines in the structure definition added by each step, which would mean all of these patches need to fix them in the next reroll. -- 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