<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Thu, Dec 17, 2015 at 2:32 PM, Christophe Fergeau <span dir="ltr"><<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
> So you were right, when I use this fix with 16bpp on Win7, some areas in<br>
> properties windows are black.<br>
> I also found out, that this patch is not actually needed, because it's the<br>
> other way around.<br>
<br>
</span>I thought it was needed in order to byteswap the surfaces?<span class=""><font color="#888888"><br>
</font></span></blockquote></div><br><div><div><div><div>It byteswaps when saved_want_original == FALSE, and as you pointed out, that happens most of the time.<br></div>The one time, that saved_want_original == TRUE is when we blit the transparent surface to 16bpp canvas.<br></div>When that happens, we don't want to byteswap, because that for some reason causes artifacts.<br></div><br></div>Originally I thought, that there are more cases, when saved_want_original == TRUE,<br>but we still want to byteswap, but as it turns out, that is not the case, and we never want to byteswap,<br>when saved_want_original == TRUE.<br clear="all"><br>-- <br><div class="gmail_signature">Lukas Venhoda</div>
</div></div>