util: remove duplicate code in format

util.format contains an idential if statement within each branch
of switch. Move it before the switch statement for cleaner code
and better performance.

PR-URL: #15098
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>

lib/util.js

@@ -111,51 +111,35 @@ function format(f) {
         ++i;
         continue;
       }
+      if (lastPos < i)
+        str += f.slice(lastPos, i);
       switch (f.charCodeAt(i + 1)) {
         case 100: // 'd'
-          if (lastPos < i)
-            str += f.slice(lastPos, i);
           str += Number(arguments[a++]);
           break;
         case 105: // 'i'
-          if (lastPos < i)
-            str += f.slice(lastPos, i);
           str += parseInt(arguments[a++]);
           break;
         case 102: // 'f'
-          if (lastPos < i)
-            str += f.slice(lastPos, i);
           str += parseFloat(arguments[a++]);
           break;
         case 106: // 'j'
-          if (lastPos < i)
-            str += f.slice(lastPos, i);
           str += tryStringify(arguments[a++]);
           break;
         case 115: // 's'
-          if (lastPos < i)
-            str += f.slice(lastPos, i);
           str += String(arguments[a++]);
           break;
         case 79: // 'O'
-          if (lastPos < i)
-            str += f.slice(lastPos, i);
           str += inspect(arguments[a++]);
           break;
         case 111: // 'o'
-          if (lastPos < i)
-            str += f.slice(lastPos, i);
           str += inspect(arguments[a++],
                          { showHidden: true, depth: 4, showProxy: true });
           break;
         case 37: // '%'
-          if (lastPos < i)
-            str += f.slice(lastPos, i);
           str += '%';
           break;
         default: // any other character is not a correct placeholder
-          if (lastPos < i)
-            str += f.slice(lastPos, i);
           str += '%';
           lastPos = i = i + 1;
           continue;