René Scharfe <l.s.r@xxxxxx> writes: > This allows us to get rid of several write-only variables. > > Signed-off-by: Rene Scharfe <l.s.r@xxxxxx> > --- > builtin/checkout.c | 3 +-- > builtin/receive-pack.c | 3 +-- > ref-filter.c | 7 ++----- > reflog-walk.c | 6 ++---- > transport.c | 3 +-- > wt-status.c | 4 +--- > 6 files changed, 8 insertions(+), 18 deletions(-) Both patches looked good to me. It's a bit surprising that we care so much about where a symbolic ref points at, but it is understandable from the list of files affected.