fix for crappy 0.10.X node

Author: nmccready

src/utils.js

@@ -12,7 +12,17 @@ var urlRegex = /^(https?|webpack(-[^:]+)?):\/\//;
 
 var debug = require('debug-fabulous')()(PLUGIN_NAME + ':utils');
 
-var sourceMapUrlRegEx = /\/\/\# sourceMappingURL\=.*/g
+/*
+TODO: DROP SUPPORT FOR node <= 0.12.X
+So reusing the same ref for a regex (with global (g))
+is a major bug in node 0.10.X apparently.
+See http://stackoverflow.com/questions/10229144/bug-with-regexp-in-javascript-when-do-global-search
+
+So we either need to use a new instance of a regex everywhere
+to make it compatible for all nodes 0.10.X + or drop support for 0.10.X.
+For now we will continue to support it I guess. But this sucks.
+*/
+var sourceMapUrlRegEx = function(){ return /\/\/\# sourceMappingURL\=.*/g;}
 
 
 var getCommentFormatter = function (file) {
@@ -52,9 +62,9 @@ var getCommentFormatter = function (file) {
 }
 
 var getPreExisting = function(fileContent){
-  if(sourceMapUrlRegEx.test(fileContent)){
+  if(sourceMapUrlRegEx().test(fileContent)){
     debug('has preExisting');
-    return fileContent.match(sourceMapUrlRegEx)[0];
+    return fileContent.match(sourceMapUrlRegEx())[0];
   }
 }
 

src/write.js

@@ -161,13 +161,10 @@ function write(destPath, options) {
       }
     }
 
+    var preExisting = options.preExisting && utils.getPreExisting(String(file.contents));
     // append source map comment
-    if (options.addComment){
-      if(options.preExisting && utils.getPreExisting(String(file.contents))){
-        //normally the comment is pre-stripped but this option lets us know it is not
-      }
-      else
-        file.contents = Buffer.concat([file.contents, new Buffer(comment)]);
+    if (options.addComment && !preExisting){
+      file.contents = Buffer.concat([file.contents, new Buffer(comment)]);
     }
 
     this.push(file);