"Danwe" posted a comment on MediaWiki.r104990.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/104990#c26914
Commit summary for MediaWiki.r104990:
description message name corrected and minor formatting.
Danwe's comment:
"Semantic MediaWiki" isn't in this svn repository, right. But Maps is being
developed within this svn actually and its using this style more than just a
few times (if I am not mistaken 33 "else(if) {" vs 15 "} else(if) {". Since I
have been working on Maps a while ago, It might even be possible that I have
adopted that style from there.
IMHO its nothing worth arguing about a lot. It also seems like there is no
definite agenda on this. If there is going to be and if other extensions, which
are more excessive in using this, are going to change it, then I would be
happy to follow as well. Anyway, I see no point in further counting how often
this is being used where exactly. You could as well pick as good as any other
construct of the language and fight about where to draw a line exactly. MW
Coding conventions aren't that strict always, you will see if you take a look
at "switch case" or optional spaces behind control structures.
_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
