buffer: fast-case for empty string in byteLength

When the string is empty, calling the binding is unnecessary and slow. PR-URL: #1441 Reviewed-by: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Christian Tellnes <christian@tellnes.no>
nodejs · Apr 16, 2015 · 431673e · 431673e · Fishrock123 · Apr 16, 2015
1 parent 62f5f4c
commit 431673e
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/lib/buffer.js b/lib/buffer.js
@@ -276,6 +276,9 @@ function byteLength(string, encoding) {
   if (typeof(string) !== 'string')
     string = String(string);
 
+  if (string.length === 0)
+    return 0;
+
   switch (encoding) {
     case 'ascii':
     case 'binary':