User "Krinkle" posted a comment on MediaWiki.r93654.
Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/93654#c20539
Commit summary:
Follows-up r93515 CR: Whitespace
Comment:
I don't care how the indention is here, I changed it because someone thought it
was better and I didn't disagree.
Alternatives:
<source lang="javascript">
var
// Foobar
foo = 'bar',
bar = 'barbarabar',
// Lorem ipsum
lorem = 'ipsum';
</source>
<source lang="javascript">
var foo = 'bar', // Foobar
bar = 'barbarabar',
lorem = 'ipsum'; // Lorem ipsum
</source>
<source lang="javascript">
var foo = 'bar', // Foobar
bar = 'barbarabar',
lorem = 'ipsum'; // Lorem ipsum
</source>
Currently:
<source lang="javascript">
// Foobar
var foo = 'bar',
bar = 'barbarabar',
// Lorem ipsum
lorem = 'ipsum';
</source>
Previously:
<source lang="javascript">
// Foobar
var foo = 'bar',
bar = 'barbarabar',
// Lorem ipsum
lorem = 'ipsum';
</source>
_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
