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;
        ^

Reply via email to