Revert r86764, r89134, r86827 -- added a second opaque boolean parameter to Linker...
authorBrion Vibber <brion@users.mediawiki.org>
Tue, 14 Jun 2011 23:50:40 +0000 (23:50 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Tue, 14 Jun 2011 23:50:40 +0000 (23:50 +0000)
commit87786e84dd9ae558e4748eddabb407e3a1ca3b9c
treed4f94cac012bc1306a10d873c8af0ae554a84481
parentfb0c86db643ac13ae15f4592a9ccd01dc888fdce
Revert r86764, r89134, r86827 -- added a second opaque boolean parameter to Linker::commentBlock() which appeared to mostly just turn it into Linker::formatComment().

commentBlock() exists for the sole purpose of embedding a comment into parentheses if it exists so you can append it to a line of text -- if you're not putting stuff in parentheses, don't use commentBlock() because you're not generating a parenthesized comment block.
Opaque boolean parameters are also very poor form, especially when tacking on multiple ones. There was already a nasty optional '$local' boolean param, forcing all uses of this other parameter to add *two* parameters, making illegible stuff like 'false, false'.
includes/ImagePage.php
includes/Linker.php
includes/specials/SpecialListfiles.php