Hi Duy, On Mon, Jun 11, 2018 at 9:05 AM, Duy Nguyen <pclo...@gmail.com> wrote: > On Sat, Jun 9, 2018 at 9:58 PM Elijah Newren <new...@gmail.com> wrote: >> I read over the rest. Found a small grammatical error in a commit >> message. Found multiple places that still need conversion, from >> pushing up &the_index usages to callers of ll-merge.c and sha1-file.c >> instead of having them in those files, to mixes of _cache_ and _index_ >> functions as in apply.c and merge-recursive.c. However, Duy pointed >> out there was more work to do, > > Yes. This is just fyi, 40 patches later, i'm down to leaving the_index > in three files outside builtin/: merge-recursive, notes-merge.c and > transport.c. Even after the conversion we may need some more follow-up > patches because it now shows places where we should _not_ touch the > index at all, which may involve not simply passing NULL index_state to > some functions, but fixing them up to tolerate NULL index_state. So > it's going to be a few patch series until the_index is gone for good > [1]. > > [1] but like cheap horror movies, there's always a sequel: > the_repository is still spread in many places and hides dependencies > in the same way. We can't do anything about it though until struct > repository conversion is more or less complete.
I was just about to send you an email to ask if you were continuing on with the series. I need diff-lib.c converted in order to make the changes Junio suggested to index_has_changes at https://public-inbox.org/git/xmqqvaaz5jcv....@gitster-ct.c.googlers.com/. Since you're already working on that, I won't duplicate your effort. Thanks for tackling all of this. :-)
