[compiz] Initializing active window lists in animation

Jay Catherwood jay.catherwood at gmail.com
Thu Jul 22 08:13:31 PDT 2010


Hi,

We discussed this on IRC. I thought I would send in the patch, simple as it
is.

The issue is that PrivateAnimScreen's constructor in animation.cpp
initializes mPluginActive twice and PrivateAnimWindow doesn't initialize its
mPluginActive at all. I think the second initialization loop should be moved
to the PrivateAnimWindow constructor.

This fixes a bug for me where open window animations don't always fire.

The patch is attached.

Thanks,
Jay
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/compiz/attachments/20100722/85f716c7/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Initialize-pw-mPluginActive-correctly.patch
Type: text/x-patch
Size: 1165 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/compiz/attachments/20100722/85f716c7/attachment.bin>


More information about the compiz mailing list