fixup: simplify

nodejs · Jul 30, 2021 · d50ed00 · d50ed00
1 parent 8d348e8
commit d50ed00
Showing 1 changed file with 5 additions and 20 deletions.
diff --git a/lib/api/readable.js b/lib/api/readable.js
@@ -182,7 +182,7 @@ async function consume (stream, type) {
       resolve,
       reject,
       length: 0,
-      body: undefined
+      body: type === kTextType || type === kJSONType ? '' : []
     }
 
     stream
@@ -255,28 +255,13 @@ function consumeEnd (consume) {
   }
 }
 
-function consumePush (consume, chunk, encoding) {
+function consumePush (consume, chunk) {
+  consume.length += chunk.length
   if (consume.type === kTextType || consume.type === kJSONType) {
-    consumePushString(consume, chunk, encoding)
+    consume.body += chunk
   } else {
-    consumePushBuffer(consume, chunk, encoding)
-  }
-}
-
-function consumePushString (consume, chunk, encoding) {
-  if (!consume.body) {
-    consume.body = ''
-  }
-  consume.length += chunk.length
-  consume.body += chunk
-}
-
-function consumePushBuffer (consume, chunk, encoding) {
-  if (!consume.body) {
-    consume.body = []
+    consume.body.push(chunk)
   }
-  consume.length += chunk.byteLength
-  consume.body.push(chunk)
 }
 
 function consumeFinish (consume, err) {