> Hey MCR1. Excellent work, here's some comments so far.
>
> 8 + CompOption::Vector o(0);
> 9 dndWindow->ungrabNotify ();
> 10
> 11 + screen->handleCompizEvent ("expo", "start_viewport_switch", o);
>
> I believe there is a constant somewhere, either as a static member of
> CompOption or just as a global called "noOptions". I'd suggest using that.
>
Hmm, I do not know exactly what to change here, as searching through the code
shows it is almost always used like that, if there are no options - this is from
wall.cpp for example:
> Hey MCR1. Excellent work, here's some comments so far. >ungrabNotify (); >handleCompizEv ent ("expo", "start_ viewport_ switch" , o);
>
> 8 + CompOption::Vector o(0);
> 9 dndWindow-
> 10
> 11 + screen-
>
> I believe there is a constant somewhere, either as a static member of
> CompOption or just as a global called "noOptions". I'd suggest using that.
>
Hmm, I do not know exactly what to change here, as searching through the code
shows it is almost always used like that, if there are no options - this is from
wall.cpp for example:
CompOption: :Vector o(0); >handleCompizEv ent ("wall", "start_ viewport_ switch" , o);
...
...
screen-
I am quite clueless here, but the version we use seems correct...