<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Wed, Jul 16, 2014 at 7:13 PM, Keith Packard <span dir="ltr"><<a href="mailto:keithp@keithp.com" target="_blank">keithp@keithp.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">Michel Dänzer <<a href="mailto:michel@daenzer.net">michel@daenzer.net</a>> writes:<br>
<br>
> After several complaints about this revert from users of other DEs<br>
> (because it broke their systray icons again), I tried to reproduce the<br>
> kwin problem shown on<br>
> <a href="http://markus.members.selfnet.de/xorg/glamor_kwin.png" target="_blank">http://markus.members.selfnet.de/xorg/glamor_kwin.png</a> without the revert<br>
> again, but couldn't. Markus, can you still reproduce this problem<br>
> without the revert? If yes, please provide detailed instructions to<br>
> reproduce. If not, it was probably fixed by something else in the meantime.<br>
<br>
</div>I think I understand the code and now believe that both the original<br>
fixed versions are incorrect. However, I cannot reproduce any of the<br>
problems we've had reported. I've placed an explanation of the bug and<br>
the fix in the patch attached here. In theory, this should fix *both*<br>
the original kwin problem and the systray icons. I would love to see<br>
this tested soon if possible. I wouldn't mind slipping this into 1.16;<br>
it seems "low risk" to me, and solves a fairly serious regression with<br>
glamor.<br>
<br></blockquote><div>The patch fixes the systray icon issue that I have experienced, thanks. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><br>
--<br>
<a href="mailto:keith.packard@intel.com">keith.packard@intel.com</a><br>
<br>_______________________________________________<br>
<a href="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</a>: X.Org development<br>
Archives: <a href="http://lists.x.org/archives/xorg-devel" target="_blank">http://lists.x.org/archives/xorg-devel</a><br>
Info: <a href="http://lists.x.org/mailman/listinfo/xorg-devel" target="_blank">http://lists.x.org/mailman/listinfo/xorg-devel</a><br></blockquote></div><br></div></div>