Re: [PATCH v2 2/9] setup_git_directory(): use is_dir_sep() helper
- From
Jeff King <peff@peff.net>
- Date
- Mar 3, 2017, 03:37 UTC
- Message-ID
- <20170303033714.5h7qrq57hdqwoi3y@sigill.intra.peff.net>
- In-Reply-To
- <d1342d28fd402615f1f94d9190559070ed04b0d7.1488506615.git.johannes.schindelin@gmx.de>
On Fri, Mar 03, 2017 at 03:04:07AM +0100, Johannes Schindelin wrote:
Show 7 quoted lines
> It is okay in practice to test for forward slashes in the output of > getcwd(), because we go out of our way to convert backslashes to forward > slashes in getcwd()'s output on Windows. > > Still, the correct way to test for a dir separator is by using the > helper function we introduced for that very purpose. It also serves as a > good documentation what the code tries to do (not "how").
Makes sense, but...
Show 7 quoted lines
> @@ -910,7 +910,8 @@ static const char *setup_git_directory_gently_1(int *nongit_ok) > return setup_bare_git_dir(&cwd, offset, nongit_ok); > > offset_parent = offset; > - while (--offset_parent > ceil_offset && cwd.buf[offset_parent] != '/'); > + while (--offset_parent > ceil_offset && > + !is_dir_sep(dir->buf[offset_parent]));
What is "dir"? I'm guessing this patch got reordered and it should stay as cwd.buf?
-Peff