Re: [PATCH 1/5] t/helper: stop using `the_index`

[Karthik Nayak <karthik.188@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1050 bytes --]

Patrick Steinhardt <ps@pks.im> writes:

> Convert test-helper tools to use `the_repository->index` instead of
> `the_index`.
>
> Signed-off-by: Patrick Steinhardt <ps@pks.im>
> ---
>  t/helper/test-cache-tree.c           | 17 ++++++------
>  t/helper/test-dump-cache-tree.c      |  5 ++--
>  t/helper/test-dump-split-index.c     | 11 ++++----
>  t/helper/test-dump-untracked-cache.c |  3 +--
>  t/helper/test-lazy-init-name-hash.c  | 39 ++++++++++++++--------------
>  t/helper/test-read-cache.c           |  9 +++----
>  t/helper/test-scrap-cache-tree.c     |  7 +++--
>  t/helper/test-write-cache.c          |  3 +--
>  8 files changed, 43 insertions(+), 51 deletions(-)
>
> diff --git a/t/helper/test-cache-tree.c b/t/helper/test-cache-tree.c
> index e7236392c8..dc89ecfd71 100644
> --- a/t/helper/test-cache-tree.c
> +++ b/t/helper/test-cache-tree.c
> @@ -1,4 +1,3 @@
> -#define USE_THE_INDEX_VARIABLE
>

Nit: It would've been nice to mention what this macro is used for.
Perhaps maybe in the cover letter since its removed from all the commits.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 690 bytes --]