Re: [PATCH v6 3/3] core: convert build-time USE_NSEC into runtime core.useNanosec

Ben Knoble [off-list ref] writes:

quoted

I still am worried that something that sits this deep in the
callchain can easily BUG() when working on a repository that is not
the_repository due to the use of repo_config_values(), and we might
be better off adopting safe default when istate->repo is different
from the_repository, but other than that, I think the series is in
great shape.

Thanks.

[administrivia: wrap overly long lines]

Yea. See previous messages re: convincing the test apparatus to
set this globally. If I could run it that way at least locally, it
would go a little ways towards scaring those BUGs out into the
light.

I am not worried too much about the current code.  I am more worried
about how much this will hinder future development of new features,
e.g., diff or status recursively going into submodules without
spawning subprocesses, which is done for grep already.  Testing and
seeing 'git grep --recurse-submodule' not hitting a BUG() does not
assure us all that much, as I do not think it needs to deal with
racily clean entries any specially.

Thanks.