Hi All,
I imagine it has previously been discussed, and I've noticed a creep
throughout the repository of people (Jeroen being one of them) doing it. But
we should probably really have everything stylized the same.
I keep the API fairly up to date, and on a similar vein, have been adding
braces across other files as I go along.
Obviously, this gives us the benefit of as much of our code as possible
meeting our Coding Conventions [1], though, might cause some people some
issues, more so with some patches. However, isn't there a svn setting to
ignore whitespace when doing that sort of action?
It'd also be nice if stylize would add braces to one line if's. [2]
Thanks
Sam
[1]
http://www.mediawiki.org/wiki/Coding_convention
[2]
https://bugzilla.wikimedia.org/show_bug.cgi?id=22621