Jay Catherwood
2010-Jul-22 15:13 UTC
[compiz] Initializing active window lists in animation
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>
Possibly Parallel Threads
- [PATCH] plugin conflict between Minimize & Animation: add feature "windowanimations"
- [PATCH] plugin conflict resolution between Fade & Animation
- Java animations, Google Earth flashing with Compiz enabled
- Tried out Beryl (Animation pugin)
- Newest animation plugin