User "Nikerabbit" posted a comment on MediaWiki.r99101.
Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/99101#c23880
Commit summary:
More documentation
Added 1 TODO
Comment:
<pre>
+ /**
+ * @return MediaWikiMessageChecker
+ */
public function getChecker() {
</pre>
I'd argue it's implementation detail what kind of MessageChecker it returns.
<pre>
+ * @param $tables array|stromg
</pre>
omg?
I don't see the point of documenting params of hook functions.
_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
