Use regex for BOM, always strip. #46, #18

index.js

@@ -101,10 +101,9 @@ function hbsfy(file, opts) {
     var compiled = "// hbsfy compiled Handlebars template\n";
     var parsed = null;
     var partials = null;
-
-    if (opts && (opts.b || opts.bom) && buffer.indexOf('') === 0) {
-      buffer = buffer.substring(1);
-    }
+
+    // Kill BOM
+    buffer = buffer.replace(/^\uFEFF/, '');
 
     try {
       if (traverse) {

test/bom_test.js

@@ -2,27 +2,25 @@
 var assert = require("assert");
 var concat = require("concat-stream");
 var fs = require("fs");
-var hbsfyBom = require("hbsfy");
-var hbsfyNoBom = require("hbsfy").configure({
-  bom: true
-});
+var hbsfy = require("hbsfy");
 
 var templatePath = __dirname + "/bom.hbs";
+var reBOM = /^\uFEFF/;
 
+// Ensure our fixture actually contains a BOM
 fs.createReadStream(templatePath)
-.pipe(hbsfyBom(templatePath))
 .pipe(concat(function(data) {
   assert(
-    //.test(data.toString()),
+    reBOM.test(data.toString()),
     "The template should contain a bom"
   );
 }));
 
 fs.createReadStream(templatePath)
-.pipe(hbsfyNoBom(templatePath))
+.pipe(hbsfy(templatePath))
 .pipe(concat(function(data) {
   assert(
-    //.test(data.toString()) === false,
+    reBOM.test(data.toString()) === false,
     "The template should not contain a bom"
   );
 }));