Re: [PATCH] use pop_commit() for consuming the first entry of a struct commit_list

2015-10-27 Thread René Scharfe
Am 26.10.2015 um 22:33 schrieb Junio C Hamano: René Scharfe writes: Instead of open-coding the function pop_commit() just call it. This makes the intent clearer and reduces code size. Signed-off-by: Rene Scharfe --- builtin/fmt-merge-msg.c | 9 +++-- builtin/merge.c | 12 +++

Re: [PATCH] use pop_commit() for consuming the first entry of a struct commit_list

2015-10-26 Thread Junio C Hamano
René Scharfe writes: > Instead of open-coding the function pop_commit() just call it. This > makes the intent clearer and reduces code size. > > Signed-off-by: Rene Scharfe > --- > builtin/fmt-merge-msg.c | 9 +++-- > builtin/merge.c | 12 +--- > builtin/reflog.c|

[PATCH] use pop_commit() for consuming the first entry of a struct commit_list

2015-10-24 Thread René Scharfe
Instead of open-coding the function pop_commit() just call it. This makes the intent clearer and reduces code size. Signed-off-by: Rene Scharfe --- builtin/fmt-merge-msg.c | 9 +++-- builtin/merge.c | 12 +--- builtin/reflog.c| 6 +- builtin/rev-parse.c |