Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > + /* > + * Between object types show tags, then commits, and finally > + * trees and blobs. > + * > + * The object_type enum is commit, tree, blob, tag, but we > + * want tag, commit, tree blob. Cleverly (perhaps too The missing comma between "tree blob" on the second line made me read this comment twice, which made me notice the lack of "and" before "tag" on the previous line. Assignment is "commit, tree, blob, and then tag" but we want "tag, commit, tree and then blob", perhaps?