Hi David, First of all, thanks for your great work with Compiz/Xgl :)
I will put disablePostprocessFX in privates. For texCoordSize, we have the Blurfx plugin which needs it. And you might have other plugins in compiz in the future, which use the vertices and indices produced by another plugin (animation or wobbly) to do effects similar to blurring, reflections, etc., and therefore need texCoordSize. It's used together with CompWindow.texUnits so I think it would make sense for it to be in CompWindow instead of in privates. And about the common code with minimize: Actually Animation was written to replace Minimize completely. I don't know if you would consider that as an option though. Otherwise, putting the common code in core can be considered. Thanks. PS: I'm maintaining Animation in Beryl, but I don't have lots of spare time, so I can't promise to maintain it in Compiz as well (which might possibly be a different task if the two projects diverge more). Also, if people want future changes in Beryl's animation.c to be incorporated into Compiz's animation.c, I won't have time for the necessary conversion, testing, etc. for Compiz. So if somebody else (like Mike?) could take the responsibility of testing and maintenance on Compiz and, if necessary, inform me about bugs/problems, I would be glad. Sorry. (I'm pretty busy). - Erkin On 10/4/06, David Reveman <[EMAIL PROTECTED]> wrote:
On Wed, 2006-10-04 at 03:04 -0500, Erkin Bahceci wrote: > Hi, Hey, I'm OK with adding this plugin to the compiz repository if you or someone else can maintain it. Some notes below. Lets not add disablePostprocessFX for now as I don't know the details of blurfx and I'm not sure exactly why it's needed. We can add it or figure out a more appropriate solution when we need it. I guess adding indexCount to CompWindow struct makes sense as we have a indices pointer in there. I think texCoordSize should go into the animation plugins private window struct for now as it's the only plugin that I'm aware of that will make use of it. Other than that I think it's OK to go in. Looking through the code quickly I found that some code from the minimize plugin has been duplicated. We might want to consider sharing some of that code by putting it in the core. -David
_______________________________________________ compiz mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/compiz
