doc,test: minor improvements to O_DSYNC

Backport-PR-URL: #15653 PR-URL: #15547 Refs: #15451 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Oct 7, 2017 · dec383a · dec383a
1 parent 8ec81a8
commit dec383a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/doc/api/fs.md b/doc/api/fs.md
@@ -2867,12 +2867,13 @@ The following constants are meant for use with `fs.open()`.
   </tr>
   <tr>
     <td><code>O_SYNC</code></td>
-    <td>Flag indicating that the file is opened for synchronous I/O.</td>
+    <td>Flag indicating that the file is opened for synchronized I/O with write
+    operations waiting for file integrity.</td>
   </tr>
   <tr>
     <td><code>O_DSYNC</code></td>
-    <td>Flag indicating that the file is opened for synchronous I/O
-    with write operations waiting for data integrity.</td>
+    <td>Flag indicating that the file is opened for synchronized I/O with write
+    operations waiting for data integrity.</td>
   </tr>
   <tr>
     <td><code>O_SYMLINK</code></td>

diff --git a/test/parallel/test-fs-open-flags.js b/test/parallel/test-fs-open-flags.js
@@ -79,12 +79,10 @@ assert.throws(
   /^Error: Unknown file open flag: null$/
 );
 
-if (common.isLinux === true) {
+if (common.isLinux) {
   const file = `${__dirname}/../fixtures/a.js`;
 
-  fs.open(file, O_DSYNC, common.mustCall(function(err, fd) {
-    assert.ifError(err);
-  }));
+  fs.open(file, O_DSYNC, common.mustCall(assert.ifError));
 }
 
 function escapeRegExp(string) {