> /* Remember to update object flag allocation in object.h */ > +#define REACHABLE (1u<<15) > #define PARENT1 (1u<<16) > #define PARENT2 (1u<<17) > #define STALE (1u<<18) Update the object flag allocation in object.h. > +int reachable(struct commit *from, int with_flag, int assign_flag, > + time_t min_commit_date) In this and previous patches: I think it's better to use "unsigned int" as the data type for a flag, just like in clear_commit_marks(). Other than that, this and all previous patches look good.