Fixes issue #32

ryanmcgrath · Nov 9, 2012 · 676e210 · 676e210
1 parent 04d6db1
commit 676e210
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/wrench.js b/lib/wrench.js
@@ -369,6 +369,10 @@ exports.LineReader = function(filename, bufferSize) {
 };
 
 exports.LineReader.prototype = {
+ close: function() {
+ return fs.closeSync(this.fd);
+ },
+
  getBufferAndSetCurrentPosition: function(position) {
  var res = fs.readSync(this.fd, this.bufferSize, position, "ascii");
 

diff --git a/readme.md b/readme.md
@@ -45,6 +45,10 @@ var f = new wrench.LineReader('x.txt');
 while(f.hasNextLine()) {
  util.puts(x.getNextLine());
 }
+
+// Note: You will need to close that above line reader at some point, otherwise
+// you will run into a "too many open files" error. f.close() or fs.closeSync(f.fd) are
+// your friends, as only you know when it is safe to close.
 ```
 
 ### Asynchronous operations
@@ -54,7 +58,6 @@ var files = [];
 wrench.readdirRecursive('my_directory_name', function(error, curFiles) {
  // curFiles is what you want
 });
-
 ```
 
 Questions, comments? Hit me up. (ryan [at] venodesigns.net | http:https://twitter.com/ryanmcgrath)