Implement Stricter Warning Logic

mattlyons0 · mattlyons0 · commit 8f8d6e863167 · 2018-03-08T00:27:09.000-07:00
Potentially fixes #21
diff --git a/lib/OutputParser.js b/lib/OutputParser.js
@@ -60,8 +60,6 @@ class OutputParser{
         this.summary.speedup = Number(split[6].replace(/,/g, ''));
       } else if (line.match(/^\s*(rsync|rsync error|rsync warning):/g)) { //Error Message
         this.generator.logger.log('stderr')(line);
-      } else if(line.match(/[^\\]"[^"\/]+"/g)) { //Line with unescaped " set is assumed to be a warning
-        output.push({msgType: 'warning', warning: this.outputStatus});
       } else { //Otherwise, assumed to be a filename/status
         this.outputStatus = line;
       }

Original file line number	Diff line number	Diff line change
`@@ -60,8 +60,6 @@ class OutputParser{`
`60`	`60`	`this.summary.speedup = Number(split[6].replace(/,/g, ''));`
`61`	`61`	`} else if (line.match(/^\s*(rsync\|rsync error\|rsync warning):/g)) { //Error Message`
`62`	`62`	`this.generator.logger.log('stderr')(line);`
`63`		`- } else if(line.match(/[^\\]"[^"\/]+"/g)) { //Line with unescaped " set is assumed to be a warning`
`64`		`- output.push({msgType: 'warning', warning: this.outputStatus});`
`65`	`63`	`} else { //Otherwise, assumed to be a filename/status`
`66`	`64`	`this.outputStatus = line;`
`67`	`65`	`}`