fs: add consistent flag fall throughs
authorColin Ihrig <cjihrig@gmail.com>
Fri, 9 May 2014 02:26:26 +0000 (22:26 -0400)
committerTrevor Norris <trev.norris@gmail.com>
Tue, 13 May 2014 18:48:26 +0000 (11:48 -0700)
stringToFlags() has fall throughs in a case statement.
However, they are not consistently implemented. This commit adds
consistency.

Signed-off-by: Trevor Norris <trev.norris@gmail.com>
lib/fs.js

index b61a1f5..cdcc13d 100644 (file)
--- a/lib/fs.js
+++ b/lib/fs.js
@@ -374,9 +374,11 @@ function stringToFlags(flag) {
 
   switch (flag) {
     case 'r' : return O_RDONLY;
-    case 'rs' : return O_RDONLY | O_SYNC;
+    case 'rs' : // fall through
+    case 'sr' : return O_RDONLY | O_SYNC;
     case 'r+' : return O_RDWR;
-    case 'rs+' : return O_RDWR | O_SYNC;
+    case 'rs+' : // fall through
+    case 'sr+' : return O_RDWR | O_SYNC;
 
     case 'w' : return O_TRUNC | O_CREAT | O_WRONLY;
     case 'wx' : // fall through