Re: [PATCH 2/8] write_ref_sha1(): Move write elision test to callers

2015-02-12 Thread Junio C Hamano
Michael Haggerty writes: > write_ref_sha1() previously skipped the write if the reference already > had the desired value, unless lock->force_write was set. Instead, > perform that test at the callers. > > Two of the callers (in rename_ref()) unconditionally set force_write > just before calling

[PATCH 2/8] write_ref_sha1(): Move write elision test to callers

2015-02-09 Thread Michael Haggerty
write_ref_sha1() previously skipped the write if the reference already had the desired value, unless lock->force_write was set. Instead, perform that test at the callers. Two of the callers (in rename_ref()) unconditionally set force_write just before calling write_ref_sha1(), so they don't need t