[ovs-dev] [PATCH] ovs-atomic: __GNUC__ == 4 is enough
Linhaifeng
haifeng.lin at huawei.com
Tue Sep 22 01:06:06 UTC 2020
>
> On 9/19/20 8:08 AM, Linhaifeng wrote:
> > 1. include config.h to avoid include different atomic head file
>
> All .c files should include config.h as their first include, so config.h should always
> be already included at the time we including ovs-atomic.h. Do you know the .c
> file that doesn't include config.h?
> If so, please, fix it instead. Header files should not include config.h by
> themselves.
>
OK. Thank you.
> > 2. __GNUC__ == 4 is enough
>
> This is technically correct, but it doesn't worth the work for fixing it unless
> you're fixing some real issue at the same time.
>
OK. Thank you.
More information about the dev
mailing list