<div dir="ltr">Sure, will send out v4 of the patch soon. <br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 12, 2013 at 9:42 AM, Jesse Gross <span dir="ltr"><<a href="mailto:jesse@nicira.com" target="_blank">jesse@nicira.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Jun 11, 2013 at 8:49 PM, Andy Zhou <<a href="mailto:azhou@nicira.com">azhou@nicira.com</a>> wrote:<br>
> Did not realize that the __CHECK_ENDIAN__ flags has to be passed by hand to<br>
> enable those checks. This is what I use to verify the fix. Any better way<br>
> to enable this check?<br>
><br>
> make C=2 CF="-D__CHECK_ENDIAN__"<br>
<br>
</div>That's how run it (actually I do a slightly extended version):<br>
make C=2 CF="-Wsparse-all -D__CHECK_ENDIAN__"<br>
<br>
You could put this in a script or environment variable if you want.<br>
<div class="im"><br>
> The following incremental patch fixes those warnings.<br>
<br>
</div>Would you mind sending out a new version with these changes rolled in?<br>
</blockquote></div><br></div>