<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">&lt;<a href="mailto:jesse@nicira.com" target="_blank">jesse@nicira.com</a>&gt;</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 &lt;<a href="mailto:azhou@nicira.com">azhou@nicira.com</a>&gt; wrote:<br>

&gt; Did not realize that the __CHECK_ENDIAN__ flags has to be passed by hand to<br>
&gt; enable those checks.  This is what I use to verify the fix.  Any better way<br>
&gt; to enable this check?<br>
&gt;<br>
&gt; make C=2 CF=&quot;-D__CHECK_ENDIAN__&quot;<br>
<br>
</div>That&#39;s how run it (actually I do a slightly extended version):<br>
make C=2 CF=&quot;-Wsparse-all -D__CHECK_ENDIAN__&quot;<br>
<br>
You could put this in a script or environment variable if you want.<br>
<div class="im"><br>
&gt; 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>