Re: [PATCH] stripspace: allow -s/-c outside git repository
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Dec 18, 2018, 12:00 UTC
- Message-ID
- <nycvar.QRO.7.76.6.1812181258550.43@tvgsbejvaqbjf.bet>
- In-Reply-To
- <CAN0heSoekZ_vjR6DRjSAfw74hn_NU3-5TFf6-Hn9z2D_zE-mNg@mail.gmail.com>
Hi Martin,
On Tue, 18 Dec 2018, Martin Ågren wrote:
Show 16 quoted lines
> On Mon, 17 Dec 2018 at 22:56, Jonathan Nieder <jrnieder@gmail.com> 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