On Tue, May 8, 2018 at 8:21 PM, Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> wrote: > > On Tue, May 08 2018, Nguyễn Thái Ngọc Duy wrote: > >> The current document mentions OBJ_* constants without their actual >> values. A git developer would know these are from cache.h but that's >> not very friendly to a person who wants to read this file to implement >> a pack file parser. >> >> Similarly, the deltified representation is not documented at all (the >> "document" is basically patch-delta.c). Translate that C code in >> English. > > Thanks, will drop my version from v4, although a comment for the enum in > cache.h pointing the reader to these docs would be very useful. True. I will add some together with the pack-format.txt update. -- Duy