On Tue, Feb 14, 2012 at 1:14 PM, Patrick Baggett <span dir="ltr"><<a href="mailto:baggett.patrick@gmail.com">baggett.patrick@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> Ack, I realized I didn't CC that to the list, thanks for bringing that up.</blockquote><div>Sorry, I CC'd to wrong mailing list. </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <div class="HOEnZb"><div class="h5"><br><div class="gmail_quote">On Tue, Feb 14, 2012 at 2:45 PM, Anuj Phogat <span dir="ltr"><<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>></span> wrote:<br> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On Mon, Feb 13, 2012 at 11:40 AM, Patrick Baggett <span dir="ltr"><<a href="mailto:baggett.patrick@gmail.com" target="_blank">baggett.patrick@gmail.com</a>></span> wrote:<br> </div><div class="gmail_quote"><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> Would it be more appropriate to compare the product of the width and height? e.g.<div><br></div><div> if (width * height > decompress->Width* decompress->Height)</div></blockquote><div> </div></div><div>Yes, this looks more appropriate. Tested it with piglit quick.tests. No regressions.</div> <div>Brian, do you have any comments on Patrick's suggestion?</div><div><div> </div></div></div></blockquote></div> </div></div></blockquote></div><br>