On Tue, Sep 28, 2021 at 04:18:41AM +0200, Ævar Arnfjörð Bjarmason wrote: > This improves fsck error reporting, see the examples in the commit > messages of 16/17 and 17/17. To get there I've lib-ified more things > in object-file.c and the general object APIs, i.e. now we'll return > error codes instead of calling die() in these cases. > > v6 of this got a very detailed review from Taylor Blau (thanks a > lot!), for the v6 see: > https://lore.kernel.org/git/cover-v6-00.22-00000000000-20210907T104558Z-avarab@xxxxxxxxx/ > > The v7 had a couple of trivial shellscripting issues, a typo'd > test_oid variable, and a warning on a "test" comparison. For v7 see > https://lore.kernel.org/git/cover-v7-00.17-00000000000-20210920T190304Z-avarab@xxxxxxxxx/ Thanks; I looked at the range-diff and it addresses both of my comments. This series looks good to me. Thanks, Taylor