User "Dantman" posted a comment on MediaWiki.r93654. Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/93654#c20655 Commit summary:
Follows-up r93515 CR: Whitespace Comment: I know that variables are function scoped not block scoped (well, ;) unless you use let, but we don't need to go there). I just said it makes more sense semantically for the code being written to put the len inside the for () loop because while the var is technically function scoped, so is <var>n</var> and they are both only relevant inside the context of the for() loop. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
