On Thu, 17 Dec 2015 22:22:18 +0900
Lorenzo Colitti <lore...@google.com> wrote:

> diff --git a/misc/ss.c b/misc/ss.c
> index 0dab32c..be70c41 100644
> --- a/misc/ss.c
> +++ b/misc/ss.c
> @@ -160,6 +160,9 @@ struct filter
>       int states;
>       int families;
>       struct ssfilter *f;
> +     struct {
> +             __u8 kill:1;
> +     } action;
>  };
>  
>  stati

Please just make it a boolean or integer, not a structure wrapped around a bit 
field.
If you need to extend in the future then change it then. Please don't write code
based on speculative future additions.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to