lib: update indentation of ternaries

In preparation for stricter indentation linting and to increase code clarity, update indentation for ternaries in lib. PR-URL: #14247 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Jul 18, 2017 · 6361565 · 6361565
1 parent 7a116d4
commit 6361565
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 10 deletions.
diff --git a/lib/events.js b/lib/events.js
@@ -262,8 +262,8 @@ function _addListener(target, type, listener, prepend) {
   } else {
     if (typeof existing === 'function') {
       // Adding the second element, need to change to array.
-      existing = events[type] = prepend ? [listener, existing] :
-                                          [existing, listener];
+      existing = events[type] =
+        prepend ? [listener, existing] : [existing, listener];
     } else {
       // If we've already got an array, just append.
       if (prepend) {

diff --git a/lib/internal/freelist.js b/lib/internal/freelist.js
@@ -9,8 +9,9 @@ class FreeList {
   }
 
   alloc() {
-    return this.list.length ? this.list.pop() :
-                              this.ctor.apply(this, arguments);
+    return this.list.length ?
+      this.list.pop() :
+      this.ctor.apply(this, arguments);
   }
 
   free(obj) {

diff --git a/lib/net.js b/lib/net.js
@@ -64,7 +64,7 @@ function createHandle(fd) {
 
 function getNewAsyncId(handle) {
   return (!handle || typeof handle.getAsyncId !== 'function') ?
-      newUid() : handle.getAsyncId();
+    newUid() : handle.getAsyncId();
 }
 
 

diff --git a/lib/url.js b/lib/url.js
@@ -825,8 +825,8 @@ Url.prototype.resolveObject = function resolveObject(relative) {
       //occasionally the auth can get stuck only in host
       //this especially happens in cases like
       //url.resolveObject('mailto:local1@domain1', 'local2@domain2')
-      const authInHost = result.host && result.host.indexOf('@') > 0 ?
-                       result.host.split('@') : false;
+      const authInHost =
+        result.host && result.host.indexOf('@') > 0 && result.host.split('@');
       if (authInHost) {
         result.auth = authInHost.shift();
         result.host = result.hostname = authInHost.shift();
@@ -902,13 +902,13 @@ Url.prototype.resolveObject = function resolveObject(relative) {
 
   // put the host back
   if (noLeadingSlashes) {
-    result.hostname = result.host = isAbsolute ? '' :
-                                    srcPath.length ? srcPath.shift() : '';
+    result.hostname =
+      result.host = isAbsolute ? '' : srcPath.length ? srcPath.shift() : '';
     //occasionally the auth can get stuck only in host
     //this especially happens in cases like
     //url.resolveObject('mailto:local1@domain1', 'local2@domain2')
     const authInHost = result.host && result.host.indexOf('@') > 0 ?
-                     result.host.split('@') : false;
+      result.host.split('@') : false;
     if (authInHost) {
       result.auth = authInHost.shift();
       result.host = result.hostname = authInHost.shift();