<div dir="ltr">So, what should I do ? Is it okay to make a cast for every multiplication of stride by height ? I'm asking because I don't want to work on a patch that will be denied because my solution is wrong.<br>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/9/9 LE GARREC Vincent <span dir="ltr"><<a href="mailto:legarrec.vincent@gmail.com" target="_blank">legarrec.vincent@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div><div><div>I didn't wanted doing such a thing because :<br> 1) rowstride will never have a value higher than 2^31 so you will definitely lose 4 octets. But okay, nobody cases about it.<br></div> 2) I didn't check that bits_image is private so I wanted avoid a breakage. So, you're right.<br>


</div> 3) It still needs lots of change because if I change bits_image.rowstride in size_t, I will have to change also lots of variable (like the uint32_t byte_stride of the function arm_simd_fill) and parameters of function (like mips_dspr2_fill). And this is just one example but I think there's plenty.<br>


<br></div>So I think that both ways need lots of work. With my solution, I just have to find every variable which have "stride" in its name and cast every multiplication.<br></div><div class="HOEnZb"><div class="h5">

<div class="gmail_extra"><br><br>
<div class="gmail_quote">2013/9/9 Bill Spitzak <span dir="ltr"><<a href="mailto:spitzak@gmail.com" target="_blank">spitzak@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


It looks like making bits->rowstride into a size_t would be a better solution than doing this casting?<br>
<br>
LE GARREC Vincent wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>
Dear,<br>
<br>
I tried to create a very big image (height*stride > 2^31) but it looks like there's some difficulties. I reported it (<a href="https://bugs.freedesktop.org/show_bug.cgi?id=69014" target="_blank">https://bugs.freedesktop.org/<u></u>show_bug.cgi?id=69014</a>) but the solution needs to change more than just few lines.<br>



<br>
Does someone could review the patch I made for one file (<a href="https://bugs.freedesktop.org/attachment.cgi?id=85448" target="_blank">https://bugs.freedesktop.org/<u></u>attachment.cgi?id=85448</a>) ? If it's okay, I'll make a patch for all files.<br>



<br>
And after that, I'll make another patch for cairo that have exactly the same problem.<br>
<br>
Regards,<br>
Vincent LE GARREC<br>
<br>
<br></div></div>
------------------------------<u></u>------------------------------<u></u>------------<br>
<br>
______________________________<u></u>_________________<br>
Pixman mailing list<br>
<a href="mailto:Pixman@lists.freedesktop.org" target="_blank">Pixman@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/pixman" target="_blank">http://lists.freedesktop.org/<u></u>mailman/listinfo/pixman</a><br>
</blockquote>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>