<div dir="ltr">I disagree. As the new code does not depend on the 32/64 difference, we don't need to have two almost identical tests.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 19, 2017 at 4:48 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif"><span class="">
<p><span style="color:rgb(33,33,33);font-size:12pt">> You wrote different code for 32-bit than 64-bit, and that is why you needed it. I'm going to fix it and remove this test.</span><br>
</p>
</span><div style="color:rgb(33,33,33)">
<div>
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">But even with the same or almost the same code, 64 bit header still will have <span style="color:rgb(33,33,33);font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px;background-color:rgb(255,255,255)">ch_reserved</span>
 field, when<br>
</div>
<div class="gmail_extra">32 bit header does not have it. And size of fields will be different anyways.<br>
</div>
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">I think we need test for 32 bit at least because of absence of <span style="color:rgb(33,33,33);font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px;background-color:rgb(255,255,255)">ch_reserved</span>​ field.<span class="HOEnZb"><font color="#888888"><br>
</font></span></div><span class="HOEnZb"><font color="#888888">
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">George.<br>
</div>
</font></span></div>
</div>
</div>

</blockquote></div><br></div>