Skip to content

lib: normalize indentation in parentheses#14125

Closed
Trott wants to merge 1 commit intonodejs:masterfrom
Trott:normalize-indentation
Closed

lib: normalize indentation in parentheses#14125
Trott wants to merge 1 commit intonodejs:masterfrom
Trott:normalize-indentation

Conversation

@Trott
Copy link
Member

@Trott Trott commented Jul 7, 2017

In anticipation of stricter indentation linting, normalize indentation
of code in parentheses.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

lib tools

@Trott
lib: normalize indentation in parentheses
7a4adf2 
In anticipation of stricter indentation linting, normalize indentation
of code in parentheses.
@Trott Trott added lib / src Issues and PRs related to general changes in the lib or src directory. tools Issues and PRs related to the tools directory. labels Jul 7, 2017
@nodejs-github-bot nodejs-github-bot added readline Issues and PRs related to the built-in readline module. tls Issues and PRs related to the tls subsystem. labels Jul 7, 2017
addaleax
addaleax approved these changes Jul 7, 2017
View reviewed changes
lib/readline.js
if (col < maxColumns - 1) {
for (var s = 0, itemLen = item.length; s < width - itemLen;
s++) {
for (var s = 0, itemLen = item.length; s < width - itemLen; s++) {
Copy link
Member

@addaleax addaleax Jul 7, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By the way, this could be simplified to for (var s = 0; s < width - item.length; s++), V8 should be able to handle that.

Copy link
Member Author

@Trott Trott Jul 7, 2017 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Saving that for Code + Learn next week. :-D

@refack
Copy link
Contributor

refack commented Jul 7, 2017

@Trott so this is your secret plan to dominate the "pulse"

image

@Trott
Copy link
Member Author

Trott commented Jul 8, 2017

CI: https://ci.nodejs.org/job/node-test-pull-request/9039/

@Trott
Copy link
Member Author

Trott commented Jul 10, 2017

Landed in e0af017

@Trott Trott closed this Jul 10, 2017
Trott added a commit that referenced this pull request Jul 10, 2017
@Trott
lib: normalize indentation in parentheses
e0af017 
In anticipation of stricter indentation linting, normalize indentation
of code in parentheses.

PR-URL: #14125
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
addaleax pushed a commit that referenced this pull request Jul 11, 2017
@Trott @addaleax
lib: normalize indentation in parentheses
a98ff3e 
In anticipation of stricter indentation linting, normalize indentation
of code in parentheses.

PR-URL: #14125
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@addaleax addaleax mentioned this pull request Jul 11, 2017
Merged
addaleax pushed a commit that referenced this pull request Jul 18, 2017
@Trott @addaleax
lib: normalize indentation in parentheses
b12b8c2 
In anticipation of stricter indentation linting, normalize indentation
of code in parentheses.

PR-URL: #14125
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Fishrock123 pushed a commit that referenced this pull request Jul 19, 2017
@Trott @Fishrock123
lib: normalize indentation in parentheses
4d41502 
In anticipation of stricter indentation linting, normalize indentation
of code in parentheses.

PR-URL: #14125
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Aug 15, 2017
Closed
7 tasks
Trott added a commit to Trott/io.js that referenced this pull request Aug 15, 2017
@Trott
lib: normalize indentation in parentheses
4e52ea5 
In anticipation of stricter indentation linting, normalize indentation
of code in parentheses.

PR-URL: nodejs#14125
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Aug 16, 2017
@Trott @MylesBorins
lib: normalize indentation in parentheses
a75ba80 
In anticipation of stricter indentation linting, normalize indentation
of code in parentheses.

Backport-PR-URL: #14835
PR-URL: #14125
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Aug 16, 2017
Merged
MylesBorins pushed a commit that referenced this pull request Sep 5, 2017
@Trott @MylesBorins
lib: normalize indentation in parentheses
0cacd6c 
In anticipation of stricter indentation linting, normalize indentation
of code in parentheses.

Backport-PR-URL: #14835
PR-URL: #14125
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This was referenced Sep 5, 2017
Closed
Closed
Closed
Closed
Closed
@Trott Trott deleted the normalize-indentation branch January 13, 2022 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasnell jasnell jasnell approved these changes

@addaleax addaleax addaleax approved these changes

@lpinca lpinca lpinca approved these changes

@targos targos targos approved these changes

@cjihrig cjihrig cjihrig approved these changes

@BridgeAR BridgeAR BridgeAR approved these changes

+2 more reviewers

@refack refack refack approved these changes

@vsemozhetbyt vsemozhetbyt vsemozhetbyt approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

lib / src Issues and PRs related to general changes in the lib or src directory. readline Issues and PRs related to the built-in readline module. tls Issues and PRs related to the tls subsystem. tools Issues and PRs related to the tools directory.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.