Spage has uploaded a new change for review. Change subject: Don't leak e3 and murmur3 into global scope. ......................................................................
Don't leak e3 and murmur3 into global scope. In ?debug=1 mode I noticed global (window) scope had e3 and murmur3 defined. This file is the culprit. I changed it to declare within an immediate anonymous function. Change-Id: I75423161ab54697f33aad5ba58751f01ddddc237 --- M lib/murmur3.js 1 file changed, 58 insertions(+), 56 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/E3Experiments refs/changes/19/38819/1 -- To view, visit https://gerrit.wikimedia.org/r/38819 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I75423161ab54697f33aad5ba58751f01ddddc237 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/E3Experiments Gerrit-Branch: master Gerrit-Owner: Spage <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
