From: Jonathan Tan <jonathantanmy@google.com>
Date: Thu, 11 Oct 2018 21:58:23 GMT
Subject: Re: [PATCH 03/19] object-store: allow read_object_file_extended to read from arbitrary repositories
Message-ID: <20181011215823.182711-1-jonathantanmy@google.com>
In-Reply-To: <20181011211754.31369-4-sbeller@google.com>

> @@ -1413,10 +1414,10 @@ void *read_object_file_extended(const struct object_id *oid,
>  	const char *path;
>  	struct stat st;
>  	const struct object_id *repl = lookup_replace ?
> -		lookup_replace_object(the_repository, oid) : oid;
> +		lookup_replace_object(r, oid) : oid;

Firstly, patches 1 and 2 are obviously correct.

This lookup_replace_object() is a bit tricky in that at 'master',
register_replace_ref() (indirectly called by lookup_replace_object())
did not handle arbitrary repositories correctly, but 'next' has a patch
that solves this, so this shouldn't be an issue. The other function
changes look fine. So this patch looks correct too.