Code review comment for lp:~vanvugt/compiz/fix-1028809
Revision history for this message
| Daniel van Vugt (vanvugt) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
35a32c5
(Get the code!)
The whitespace change is intentional. I switched tabs to spaces to guarantee the text lines up with "/* Note: " regardless of the tab size. Code should always look correct and properly aligned regardless of the tab size you're using. But I admit that's not true for most of compiz.
Yes the XML change is technically correct. However I think only CCSM ever pays attention to it.