passing options through to readers is still a good idea, so not a ful…

…l revert. #72

README.md

@@ -59,8 +59,6 @@ fs.src(['*.js', '!b*.js'])
   - read - `true` or `false` if you want the file to be read or not. Useful for stuff like `rm`ing files.
     - Default value is `true`
     - `false` will disable writing the file to disk via `.dest()`
-  - stripBOM - `true` or `false` if you want UTF-8 BOM stripped from files as they are read.
-    - Default value is `true`
   - since - `Date` or `number` if you only want files that have been modified since the time specified.
   - passthrough - `true` or `false` if you want a duplex stream which passes items through and emits globbed files.
     - Default is `false`.

lib/src/getContents/bufferFile.js

@@ -8,7 +8,7 @@ function bufferFile(file, opt, cb) {
     if (err) {
       return cb(err);
     }
-    file.contents = opt.stripBOM ? stripBom(data) : data;
+    file.contents = stripBom(data);
     cb(null, file);
   });
 }

lib/src/getContents/streamFile.js

@@ -4,10 +4,8 @@ var fs = require('graceful-fs');
 var stripBom = require('strip-bom');
 
 function streamFile(file, opt, cb) {
-  file.contents = fs.createReadStream(file.path);
-  if (opt.stripBOM) {
-    file.contents = file.contents.pipe(stripBom.stream());
-  }
+  file.contents = fs.createReadStream(file.path)
+    .pipe(stripBom.stream());
   cb(null, file);
 }
 

lib/src/index.js

@@ -18,7 +18,6 @@ function createFile(globFile, enc, cb) {
 
 function src(glob, opt) {
   var options = assign({
-    stripBOM: true,
     read: true,
     buffer: true,
     sourcemaps: false,

test/src.js

@@ -106,28 +106,6 @@ describe('source stream', function() {
     stream.pipe(bufferStream);
   });
 
-  it('should not strip BOM from UTF-8-encoded files when stripBOM is false', function(done) {
-    var expectedPath = path.join(__dirname, './fixtures/bom-utf8.txt');
-    var expectedContent = fs.readFileSync(expectedPath);
-
-    var onEnd = function(){
-      should.exist(buffered[0].stat);
-      buffered[0].path.should.equal(expectedPath);
-      buffered[0].isBuffer().should.equal(true);
-      buffered[0].contents.length.should.equal(expectedContent.length);
-      done();
-    };
-
-    var stream = vfs.src('./fixtures/bom-utf8.txt', {
-      cwd: __dirname,
-      stripBOM: false
-    });
-
-    var buffered = [];
-    bufferStream = through.obj(dataWrap(buffered.push.bind(buffered)), onEnd);
-    stream.pipe(bufferStream);
-  });
-
   it('should not strip anything that looks like a UTF-8-encoded BOM from UTF-16-BE-encoded files', function(done) {
     // Note: this goes for any non-UTF-8 encoding, but testing for UTF-16-BE
     // and UTF-16-LE is enough to demonstrate this is done properly.