fs: refactor deprecated functions for readability

This is a small refactor to make an object more readable (IMO). Yeah, I spent a bit longer looking at the code and misunderstanding it than I care to admit right now. PR-URL: nodejs#12910 Reviewed-By: Yuta Hiroto <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: James M Snell <[email protected]>
jasnell · May 11, 2017 · 6f449db · 6f449db
1 parent c79deaa
commit 6f449db
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/lib/fs.js b/lib/fs.js
@@ -2140,10 +2140,8 @@ WriteStream.prototype.destroySoon = WriteStream.prototype.end;
 var SyncWriteStream = internalFS.SyncWriteStream;
 Object.defineProperty(fs, 'SyncWriteStream', {
   configurable: true,
-  get: internalUtil.deprecate(() => {
-    return SyncWriteStream;
-  }, 'fs.SyncWriteStream is deprecated.', 'DEP0061'),
-  set: internalUtil.deprecate((val) => {
-    SyncWriteStream = val;
-  }, 'fs.SyncWriteStream is deprecated.', 'DEP0061')
+  get: internalUtil.deprecate(() => SyncWriteStream,
+                              'fs.SyncWriteStream is deprecated.', 'DEP0061'),
+  set: internalUtil.deprecate((val) => { SyncWriteStream = val; },
+                              'fs.SyncWriteStream is deprecated.', 'DEP0061')
 });