Code review comment for lp:~ksamak/compiz/switcher_add_size_multiplier

Revision history for this message
ksamak (ksamak) wrote :

Hi sam, and thank you for the review.

I fixed the coding style, according to the original indents, although i don't fully understand why there could be eg: 4 spaces, then 2 tabs, then 2 spaces in some lines ;-)

I took into account your comments on float multiplications of course, i should have seen that on my own...
I also deleted the boolean flag switch, which appeared to work in one of my tests (in a VM), but was apparently a twitch in gsettings.

thanks again, hope this revision is up to standards.

review: Needs Resubmitting

« Back to merge proposal