User "Krinkle" posted a comment on MediaWiki.r80572. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/80572#c16682 Commit summary:
Fix bug in mw.Map + fix bug 26801 + wrapper mediawiki.special.upload.js * Map.prototype.exists was checking the type of the wrong variable. Fixed now, checking multiple at once is now possible (as documented) * bug 26801: No references to legacy globals, use mw.config everywhere in core (grep search for 'wg' in /resources returned only * Addressed some JS Lint warnings (combining var statements, faster and less code); * Removed trailing tabs and spaces Comment: Why ? A (something) to string comparison doesn't have to be strict and is also slower in most browsers AFAIK. The JSHint validator only warns for non-strict comparisons to 0, null, '', false etc. which is indeed wrong in most cases. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
