graceful-fs - npm Package Compare versions

Comparing version 1.1.9 to 1.1.10

graceful-fs.js

@@ -10,9 +10,7 @@ // this keeps a queue of opened file descriptors, and will make
 var queue = []
-  , curOpen = 0
   , constants = require("constants")
 
-
 exports = module.exports = fs
+fs._curOpen = 0
 
-
 fs.MIN_MAX_OPEN = 64
@@ -43,3 +41,3 @@ fs.MAX_OPEN = 1024
 
-  if (curOpen >= fs.MAX_OPEN) {
+  if (fs._curOpen >= fs.MAX_OPEN) {
     queue.push(new OpenReq(path, flags, mode, cb))
@@ -50,7 +48,7 @@ setTimeout(flush)
   open(path, flags, mode, function (er, fd) {
-    if (er && er.code === "EMFILE" && curOpen > fs.MIN_MAX_OPEN) {
+    if (er && er.code === "EMFILE" && fs._curOpen > fs.MIN_MAX_OPEN) {
       // that was too many.  reduce max, get back in queue.
       // this should only happen once in a great while, and only
       // if the ulimit -n is set lower than 1024.
-      fs.MAX_OPEN = curOpen - 1
+      fs.MAX_OPEN = fs._curOpen - 1
       return fs.open(path, flags, mode, cb)
@@ -64,8 +62,5 @@ }
   cb = cb || noop
-  curOpen ++
+  fs._curOpen ++
   originalOpen.call(fs, path, flags, mode, function (er, fd) {
-    if (er) {
-      onclose()
-    }
-
+    if (er) onclose()
     cb(er, fd)
@@ -76,8 +71,12 @@ })
 fs.openSync = function (path, flags, mode) {
-  curOpen ++
-  return originalOpenSync.call(fs, path, flags, mode)
+  var ret
+  try {
+    ret = originalOpenSync.call(fs, path, flags, mode)
+    fs._curOpen ++
+  } finally {}
+  return ret
 }
 
 function onclose () {
-  curOpen --
+  fs._curOpen --
   flush()
@@ -87,8 +86,7 @@ }
 function flush () {
-  while (curOpen < fs.MAX_OPEN) {
+  while (fs._curOpen < fs.MAX_OPEN) {
     var req = queue.shift()
-    if (!req) break
+    if (!req) return
     open(req.path, req.flags || "r", req.mode || 0777, req.cb)
   }
-  if (queue.length === 0) return
 }
@@ -95,0 +93,0 @@

package.json

@@ -5,3 +5,3 @@ {
   "description": "fs monkey-patching to avoid EMFILE and other problems",
-  "version": "1.1.9",
+  "version": "1.1.10",
   "repository": {
@@ -15,3 +15,18 @@ "type": "git",
   },
-  "devDependencies": {}
+  "devDependencies": {},
+  "directories": {
+    "test": "test"
+  },
+  "dependencies": {},
+  "scripts": {
+    "test": "tap test/*.js"
+  },
+  "keywords": [
+    "fs",
+    "EMFILE",
+    "error",
+    "handling",
+    "monkeypatch"
+  ],
+  "license": "BSD"
 }

Fixed alerts

No tests

Quality

Package does not have any tests. This is a strong signal of a poorly maintained or low quality package.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

9864

increased by14.82%

Number of package files

6

increased by20%

Lines of code

275

increased by16.53%

Number of low quality alerts

0

decreased by-100%

No dependency changes