* Make ParserOptions private variables private.
authorNick Jenkins <nickj@users.mediawiki.org>
Tue, 17 Oct 2006 08:49:27 +0000 (08:49 +0000)
committerNick Jenkins <nickj@users.mediawiki.org>
Tue, 17 Oct 2006 08:49:27 +0000 (08:49 +0000)
commit55886772f49f58117296a01361da06c94e6f3330
tree1aad90159899c0cd1d8a255ef607e14ad2d4f73d
parent8fb74bfd49a8b203a6433512a578559875a27b73
* Make ParserOptions private variables private.
* Remove two unused globals.
* Remove pass-by-ref usage when arg is never modified (objects automatically passed-by-ref in PHP5)
* Removed some unused local variables.
* Mark a possibly dubious pass-by-ref on an uninitialized var as "FIXME: Please check that this initialization is correct", and added what I presume is the correct initialization.
* Declare preg_match arg 3 (output matches argument) as an array() before using it. Nothing wrong with not doing this, just feels slightly more explicit to me than a function call that also implictly creates and populates an array.
* My editor has automatically removed trailing spaces at the end of lines (which is not inherently a bad thing, but I have turned this option off now to help minimize diff sizes in future).
* Have confirmed that "php maintenance/parserTests.php --quiet --quick --color=no" shows no regressions, and that "php maintenance/fuzz-tester.php --specific-test=editPageTest --max-runtime=5" still shows no errors.
includes/Parser.php