Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 552e25c

Browse filesBrowse files
TrottMyles Borins
authored andcommitted
lib,test: update in preparation for linter update
ESLint 2.9.0 fixes some bugs that resulted in minor issues not being caught by ESLint 2.7.0. Update instances of our code that will be flagged when we upgrade to ESLint 2.9.0. PR-URL: #6498 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
1 parent 037291e commit 552e25c
Copy full SHA for 552e25c

File tree

Expand file treeCollapse file tree

3 files changed

+8
-1
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

3 files changed

+8
-1
lines changed
Open diff view settings
Collapse file

‎lib/buffer.js‎

Copy file name to clipboardExpand all lines: lib/buffer.js
+1Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -263,6 +263,7 @@ function byteLength(string, encoding) {
263263
case 'ascii':
264264
case 'binary':
265265
// Deprecated
266+
// eslint-disable-next-line no-fallthrough
266267
case 'raw':
267268
case 'raws':
268269
return len;
Collapse file

‎lib/readline.js‎

Copy file name to clipboardExpand all lines: lib/readline.js
+1Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -977,6 +977,7 @@ exports.emitKeypressEvents = emitKeypressEvents;
977977
*/
978978

979979
// Regexes used for ansi escape code splitting
980+
// eslint-disable-next-line no-control-regex
980981
const metaKeyCodeReAnywhere = /(?:\x1b)([a-zA-Z0-9])/;
981982
const functionKeyCodeReAnywhere = new RegExp('(?:\x1b+)(O|N|\\[|\\[\\[)(?:' + [
982983
'(\\d+)(?:;(\\d+))?([~^$])',
Collapse file

‎test/parallel/test-http-destroyed-socket-write2.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-destroyed-socket-write2.js
+6-1Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,17 @@ server.listen(common.PORT, function() {
2929
switch (er.code) {
3030
// This is the expected case
3131
case 'ECONNRESET':
32-
// On windows this sometimes manifests as ECONNABORTED
32+
break;
33+
34+
// On Windows, this sometimes manifests as ECONNABORTED
3335
case 'ECONNABORTED':
36+
break;
37+
3438
// This test is timing sensitive so an EPIPE is not out of the question.
3539
// It should be infrequent, given the 50 ms timeout, but not impossible.
3640
case 'EPIPE':
3741
break;
42+
3843
default:
3944
assert.strictEqual(er.code,
4045
'ECONNRESET',

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.