resourceloader: Optimise mw.loader.register()
authorTimo Tijhof <krinklemail@gmail.com>
Fri, 20 Nov 2015 01:14:09 +0000 (01:14 +0000)
committerOri.livneh <ori@wikimedia.org>
Tue, 24 Nov 2015 20:28:45 +0000 (20:28 +0000)
commit9a9b930a962c34d4a6f52ea24b044def84afc40c
treefbf46fb18e82ca983199e85d21ac6377bb067c25
parent30c7d4df7f9d59f2aa3b319d73e44d91696c6994
resourceloader: Optimise mw.loader.register()

* Remove redundant 'len' variables. This is a dated micro-optimisation
  that is no longer useful and can actually have an adverse effect.

* Remove [.length] assignment. Use native push() instead.

* Remove JSLint-ism of "i += 1" instead of "i++".

* Remove redundant 'delete' operation for local 'unresolved'
  object. This falls naturally out of scope. Mutating this object
  to save memory probably wastes more cycles than it saves.

Change-Id: Ib3cb40d2fef696078bd64585db9498326f08b2d2
resources/src/mediawiki/mediawiki.js
tests/qunit/suites/resources/mediawiki/mediawiki.test.js