> I still think it should check the pitch and not the width.<br>

<br>
yes<br>
<br>
> Also since height is incremented by 2 in the asm I think it should also<br>

> check for height % 2.<br>

<br>
is it even possible to have i420 frames which are not mutliples of 2 ?<br><br><div class="gmail_quote">2012/2/10 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net">remi@remlab.net</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, 10 Feb 2012 12:22:38 +0000, Asad Mehmood<br>
<<a href="mailto:asad78611@googlemail.com">asad78611@googlemail.com</a>><br>
wrote:<br>
<div class="im">>> The same issue still exists, if you have a width of 520 for instance,<br>
>> it's a<br>
>> multiple of 4, but not a multiple of 16.<br>
>> It will crash on your device.<br>
><br>
> I might have misunderstood this ...<br>
><br>
> But I though that if the width was a multiple of 4 for the output the<br>
> pitch would be a multiple of 16 in RGBA space.<br>
><br>
> e.g. A width of 520 will have a pitch of 2080?<br>
<br>
</div>You seem to assume that<br>
<br>
  pitch == pixel_width * bytes_per_pixel<br>
<br>
while actually:<br>
<br>
  pitch >= pixel_width * bytes_per_pixel<br>
<div class="im HOEnZb"><br>
--<br>
Rémi Denis-Courmont<br>
<a href="http://www.remlab.net/" target="_blank">http://www.remlab.net/</a><br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>