BlockLevelPass: minor changes due to initial code review
authorTim Starling <tstarling@wikimedia.org>
Fri, 6 May 2016 04:30:40 +0000 (14:30 +1000)
committerTim Starling <tstarling@wikimedia.org>
Fri, 6 May 2016 04:42:58 +0000 (14:42 +1000)
commita5159d8f2384a50eaf4451c4c0c5159a644f60fa
tree1b14fa6c23e3284965ec5cea39f06d370ee79a63
parent2e3c1f87e2a20aa1bfb9aec2a0b8429ce175e2ac
BlockLevelPass: minor changes due to initial code review

* Improve some comments
* In getCommon() use min() to improve readability
* Use strict equals where possible
* Use camel case in variable names
* Remove the "case 0" optimisation, made sense in PHP 4 but those are
  compile-time constants now and are presumably treated as such in both
  PHP and HHVM.
* oLine -> inputLine, no idea what "o" stood for
* pos -> colonPos, lt -> ltPos: descriptive
* stack -> level, paragraphStack -> pendingPTag: they're not stacks
* Remove "m" prefix from member variable names

Change-Id: I6c1144c792ba3e1935be88a009a6d6c110d11090
includes/parser/BlockLevelPass.php