On Tue, Dec 12, 2017 at 11:53 AM, Brandon Williams <bmwill@xxxxxxxxxx> wrote: > This series removes the remaining users of the index compatibility macros and > ensures that future uses of the macros will result in compiler errors. Thanks for converting the submodule code to avoid these old macros. Specifically the submodule code will benefit once we have a repository object available throughout the code base, so it is a great start. I reviewed the series and would call out patch 2 to be a bugfix that could go independently, but the whole series is fine as-is with me. Thanks, Stefan