On Fri, Jul 20, 2018 at 6:35 PM Derrick Stolee <dstolee@xxxxxxxxxxxxx> wrote: > > There are many places in Git that use a commit walk to determine > reachability between commits and/or refs. A lot of this logic is > duplicated. > > I wanted to achieve the following: > > Consolidate several different commit walks into one file I'm surprised get_shallow_commits() in shallow.c didn't make the cut. It's no problem though if you already considered it and decided it was better left alone. > Reduce duplicate reachability logic > Increase testability (correctness and performance) > Improve performance of reachability queries What's your recommendation on adding new commit reachability code? I might have to add one to fix prune_shallow() if I don't find anything fit. I guess the code should go to commit-reach.c too? -- Duy
