<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Oct 23, 2013 at 6:37 AM, Derek Buitenhuis <span dir="ltr"><<a href="mailto:derek.buitenhuis@gmail.com" target="_blank">derek.buitenhuis@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">On 10/23/2013 12:33 PM, Gopu Govindaswamy wrote:<br>

> (pic.poc * 3 * (width * height * pixelbytes) / 2) this statement is giving us wrong result if width and height is big<br>
<br>
</div>So cast it properly instead of making a new stack variable?<br>
<br>
Or at the very least, explain that you are doing this to obtain 64-bit<br>
precision.<br>
<br>
A commit message saying "X is modified" with no reason as to *why* is<br>
completely useless.</blockquote></div><div><br></div><div>Yeah, 4k recon dump files were busted, the commit message should mention that.</div><div><br></div><div>optimally, we would use the posix file offset data type:  off_t</div>
<div><br></div><div>We would probably have to add that to our hacked stdint.h for Windows.</div><div><br></div><div>Also, this same fix needs to be done for the Y4M file writer</div><div><br></div>-- <br>Steve Borho
</div></div>