On Sun, 29 Oct 2017 21:22:34 +0100
Stefano Brivio <sbri...@redhat.com> wrote:

> +     addrp_width -= netid_width + 1 * !!netid_width;
> +     addrp_width -= state_width + 1 * !!state_width;

Doing !! here is being too bit tricky for code that is not performance 
sensitive.
Just use an if statement or ? :

> @@ -4373,6 +4374,8 @@ int main(int argc, char *argv[])
>                       netid_width++;
>               else if (state_width)
>                       state_width++;
> +             else
> +                     odd_width_pad=" ";

Missing whitespace.

Reply via email to