On 08.05.2019 12:20, Darrell Ball wrote:
>
>
> On Wed, May 8, 2019 at 1:41 AM Ilya Maximets <[email protected]
> <mailto:[email protected]>> wrote:
>
> > On Thu, May 02, 2019 at 05:29:16PM -0700, Darrell Ball wrote:
> >> There are a few cases where structure copy can be replaced by
> >> memcpy(), for possible portability benefit. This is because
> >> the structures involved have padding and elements of the
> >> structure are used to generate hashes.
> >>
> >> Signed-off-by: Darrell Ball <dlu998 at gmail.com <http://gmail.com>>
> >
> > Thanks for the backports. I applied them to branch-2.9.
>
> Hi. These patches broke the clang build on branch-2.9 and it looks
> like not only the build:
>
>
> Thanks for the report Ilya
>
> Besides the clang false positive, what else was broken ?
Nothing. Was an issue with -Wno-null-pointer-arithmetic on branch-2.9 that I
mistakenly treated as some other problem. We, probably, may want to backport
a7021b08b0d5 ("configure: Disable -Wnull-pointer-arithmetic Clang warning.")
because branches <= 2.9 fails to build with relatively new clang.
Thanks for the fix for false positives. I'll check it.
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
