Sure, will send out v4 of the patch soon.

On Wed, Jun 12, 2013 at 9:42 AM, Jesse Gross <je...@nicira.com> wrote:

> On Tue, Jun 11, 2013 at 8:49 PM, Andy Zhou <az...@nicira.com> wrote:
> > Did not realize that the __CHECK_ENDIAN__ flags has to be passed by hand
> to
> > enable those checks.  This is what I use to verify the fix.  Any better
> way
> > to enable this check?
> >
> > make C=2 CF="-D__CHECK_ENDIAN__"
>
> That's how run it (actually I do a slightly extended version):
> make C=2 CF="-Wsparse-all -D__CHECK_ENDIAN__"
>
> You could put this in a script or environment variable if you want.
>
> > The following incremental patch fixes those warnings.
>
> Would you mind sending out a new version with these changes rolled in?
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to