Hi,

I have some comments regarding some new commits and cherry-picks to the
0.8 branch.

> commit 2d00dd0d241768e2b4bce3ef448411804a760ed5
> Author: Erkin Bahceci <erkin...@gmail.com>
> Date:   Mon Apr 7 11:02:12 2008 -0500
> 
>     Avoid conflict with Animation by suppressing fading.
> 
> commit ce6cf1fb06525fe76b0bdba08e1548070f7b88b4
> Author: Kristian Lyngstol <krist...@bohemians.org>
> Date:   Sat Feb 7 01:30:09 2009 +0100
> 
>     Add ABI-saftey net and bump ABI
>     
>     The void *reserved; variables will allow us to add a single real pointer
>     during 0.8 without breaking the ABI.

I don't think we want those two in the master branch:
- The animation fade conflict IMO should only be present in stable
branches because while it improves the user experience for new users,
it's not a "correct" solution, so it shouldn't be on master.
- I am not sure why we'd need an "ABI safety net" on master. I
definitely see a reason to have it in the stable branch, but on master I
see no problems in ABI bumps.

> commit 1734e848e5e262d2a965f4a15c4067b0fd791bde
> Author: Jigish Gohil <cyber...@opensuse.org>
> Date:   Wed Nov 26 16:32:07 2008 +0530
> 
>     remove zoom from POTFILES.in too
> 

> commit e789f81a44309336131ffbe914dcc1e9619a63cc
> Author: Kristian Lyngstol <krist...@linpro.no>
> Date:   Sun Nov 23 14:56:31 2008 +0100
> 
>     Remove zoom plugin, this is replaced by Enhanced Zoom in Compiz Fusion

I thought we wanted to keep the zoom plugin in 0.8 because of various
regressions ezoom has compared to zoom (e.g. missing manual panning).
Did I miss anything?

Regards,

Danny

_______________________________________________
Dev mailing list
Dev@lists.compiz-fusion.org
http://lists.compiz-fusion.org/mailman/listinfo/dev

Reply via email to