<div dir="ltr">Hi<br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 17, 2015 at 11:32 AM, Lukas Venhoda <span dir="ltr"><<a href="mailto:lvenhoda@redhat.com" target="_blank">lvenhoda@redhat.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"><br><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I'm wondering if this could cause issues when blitting transparent<br>
surfaces to a 16bpp canvas, as the canvas_get_image_internal documentation<br>
mentions this as a usecase (and the only caller with want_original =<br>
TRUE is canvas_draw_alpha_blend)<br>
(NB: my understanding is that "want_original" really means<br>
"keep_original_image_format")<br>
<br></blockquote><div><br></div></span><div>I'll try to test the 16bpp again on Win7, because WinXP 16bpp had it's own weird issues.<br></div><div>But from what I understand, it shouldn't be an issue, because setting want_original to TRUE<br>will just convert the canvas to the correct one when using canvas_draw_alpha_blend() anyway right?<span class="HOEnZb"></span></div></div></div></div>
</blockquote></div><br></div><div class="gmail_extra">So you were right, when I use this fix with 16bpp on Win7, some areas in properties windows are black.<br></div><div class="gmail_extra">I also found out, that this patch is not actually needed, because it's the other way around.<br></div><div class="gmail_extra">We always convert, when saved_want_original == FALSE, and on 16bpp we DO NOT want to convert.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">This patch is then not needed, and actually breaks 16bpp on ppc.<br></div><div class="gmail_extra">Rest should be ok.<br clear="all"></div><div class="gmail_extra"><br>-- <br><div class="gmail_signature">Lukas Venhoda</div>
</div></div>