Re: [PATCH v5 05/24] files-backend: move "logs/" out of TMP_RENAMED_LOG

2017-03-02 Thread Duy Nguyen
On Wed, Mar 1, 2017 at 12:19 AM, Michael Haggerty wrote: >> @@ -2513,7 +2513,7 @@ static int files_delete_refs(struct ref_store >> *ref_store, >> * IOW, to avoid cross device rename errors, the temporary renamed log must >> * live into logs/refs. >> */ >> -#define TMP_RENAMED_LOG "logs/ref

Re: [PATCH v5 05/24] files-backend: move "logs/" out of TMP_RENAMED_LOG

2017-02-28 Thread Michael Haggerty
On 02/22/2017 03:04 PM, Nguyễn Thái Ngọc Duy wrote: > This makes reflog path building consistent, always in the form of > > strbuf_git_path(sb, "logs/%s", refname); > > It reduces the mental workload a bit in the next patch when that > function call is converted. > > Signed-off-by: Nguyễn Th

[PATCH v5 05/24] files-backend: move "logs/" out of TMP_RENAMED_LOG

2017-02-22 Thread Nguyễn Thái Ngọc Duy
This makes reflog path building consistent, always in the form of strbuf_git_path(sb, "logs/%s", refname); It reduces the mental workload a bit in the next patch when that function call is converted. Signed-off-by: Nguyễn Thái Ngọc Duy --- refs/files-backend.c | 12 ++-- 1 file cha