On Sat, Mar 24 2018, Nguyễn Thái Ngọc Duy wrote: > + if (pack->in_pack_by_idx) { > + if (p->index <= 0) > + die("BUG: found_pack should be NULL " > + "instead of having non-positive index"); > + e->in_pack_idx = p->index; > + } else The indentation after the die() here is wrong. GCC complaining about it: ./pack-objects.h: In function ‘oe_set_in_pack’: ./pack-objects.h:203:3: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation] if (p->index <= 0) ^~ ./pack-objects.h:206:4: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’ e->in_pack_idx = p->index; ^