Hi Martin,
On Tue, 18 Dec 2018, Martin Ågren wrote:
> On Mon, 17 Dec 2018 at 22:56, Jonathan Nieder <jrnieder@xxxxxxxxx> wrote:
> > That makes experimenting with the stripspace command unnecessarily
> > fussy. Fix it by discovering the git directory gently, as intended
> > all along.
>
> > if (mode == STRIP_COMMENTS || mode == COMMENT_LINES) {
> > - setup_git_directory_gently(NULL);
> > + setup_git_directory_gently(&nongit);
> > git_config(git_default_config, NULL);
> > }
>
> Makes me wonder if `setup_git_directory_gently()` should BUG if it
> receives NULL. That would require some reshuffling in setup.c, since
> `setup_git_directory()` calls out to it with NULL... Just thinking out
> loud. In any case, and more importantly, this is the last user providing
> NULL except for the one in `setup_git_directory()`.
We could rename `setup_git_directory_gently()` to
`setup_git_directory_gently_2()`, make that `static` and then call it from
`setup_git_directory_gently()` and `setup_git_directory()`.
Ciao,
Dscho
