Re: [PATCH v6 1/6] Move init_skiplist() outside of fsck
- Date: Mon, 15 Apr 2019 09:32:06 -0400
- From: Barret Rhoden <brho@xxxxxxxxxx>
- Subject: Re: [PATCH v6 1/6] Move init_skiplist() outside of fsck
On 4/10/19 3:04 PM, Ævar Arnfjörð Bjarmason wrote:
On Wed, Apr 10 2019, Barret Rhoden wrote:
init_skiplist() took a file consisting of SHA-1s and comments and added
the objects to an oidset. This functionality is useful for other
This change would be much easier to review if you led with a commit
where you s/Invalid SHA-1/invalid object name/ (lower-case while we're
at it), s/skip list/object name/ etc, and did that rename of the "hash"
to "name" variable if you're so inclined.
Then you'd end up with a small refactoring change that changes the tests
(or even just make the tests grep for e.g. "Could not open.*:
does-not-exist" instead), and the moving of the function would be
entirely caught by the rename detection.
Can do. I'll split this up in the next round.