stream: delete redundant code

陈刚 · MayaLekova · commit c83005c7c49b · 2018-05-08T17:35:14.000+02:00
`state.corkedRequestsFree` of a writable stream is always not null. PR-URL: nodejs#18145 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
diff --git a/lib/_stream_writable.js b/lib/_stream_writable.js
@@ -647,11 +647,9 @@ function onCorkedFinish(corkReq, state, err) {
     cb(err);
     entry = entry.next;
   }
-  if (state.corkedRequestsFree) {
-    state.corkedRequestsFree.next = corkReq;
-  } else {
-    state.corkedRequestsFree = corkReq;
-  }
+
+  // reuse the free corkReq.
+  state.corkedRequestsFree.next = corkReq;
 }
 
 Object.defineProperty(Writable.prototype, 'destroyed', {

Original file line number	Diff line number	Diff line change
`@@ -647,11 +647,9 @@ function onCorkedFinish(corkReq, state, err) {`
`647`	`647`	`cb(err);`
`648`	`648`	`entry = entry.next;`
`649`	`649`	`}`
`650`		`- if (state.corkedRequestsFree) {`
`651`		`- state.corkedRequestsFree.next = corkReq;`
`652`		`- } else {`
`653`		`- state.corkedRequestsFree = corkReq;`
`654`		`- }`
	`650`	`+`
	`651`	`+ // reuse the free corkReq.`
	`652`	`+ state.corkedRequestsFree.next = corkReq;`
`655`	`653`	`}`
`656`	`654`
`657`	`655`	`Object.defineProperty(Writable.prototype, 'destroyed', {`