feat: gcc 9.2 output support

merged from particle-iot/gcc-output-parser commit c9a6665

Author: urish

lib/main.js

@@ -12,10 +12,11 @@ function parseString(stdout) {
 function parseGcc(stdout) {
   const messages = [];
   let match = null;
-  const deepRegex = /([^:^\r?\n]+):(\d+):(\d+):\s(\w+\s*\w*):\s(.+)\r?\n(\s+)(.*)\s+(~*\^~*)/gm;
-  //                 ^             ^     ^       ^             ^        ^    ^      ^
-  //                 |             |     |       |             |        |    |      +- token marker
-  //                 |             |     |       |             |        |    +- affected code
+  const deepRegex = /([^:^\r?\n]+):(\d+):(\d+):\s(\w+\s*\w*):\s(.+)\r?\n(\s+)\d*\s*[|]*\s*(.*)\s+[|]*\s*(~*\^~*)/gm;
+  //                 ^             ^     ^       ^             ^        ^    |            ^             ^
+  //                 |             |     |       |             |        |    |            |             +- token marker
+  //                 |             |     |       |             |        |    |            +- affected code
+  //                 |             |     |       |             |        |    +- optional gcc 9.2 markup
   //                 |             |     |       |             |        +- whitespace before code
   //                 |             |     |       |             +- message text
   //                 |             |     |       +- type (error|warning|note)

spec/9_gcc_9.2.txt

@@ -0,0 +1,48 @@
+201204_fc_recv_1.ino:561:9: warning: "/*" within comment [-Wcomment]
+  561 |         /*
+      |          
+201204_fc_recv_1.ino:64:25: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
+   64 |     char *rStates[] = { "NO Run State", "SYNC", "RUN" } ;
+      |                         ^~~~~~~~~~~~~~
+201204_fc_recv_1.ino:64:41: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
+   64 |     char *rStates[] = { "NO Run State", "SYNC", "RUN" } ;
+      |                                         ^~~~~~
+201204_fc_recv_1.ino:64:49: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
+   64 |     char *rStates[] = { "NO Run State", "SYNC", "RUN" } ;
+      |                                                 ^~~~~
+201204_fc_recv_1.ino:65:25: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
+   65 |     char *rSyncs[]  = { "No Sync State","LOOKing for SYNC"  };
+      |                         ^~~~~~~~~~~~~~~
+201204_fc_recv_1.ino:65:41: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
+   65 |     char *rSyncs[]  = { "No Sync State","LOOKing for SYNC"  };
+      |                                         ^~~~~~~~~~~~~~~~~~
+201204_fc_recv_1.ino:76:23: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
+   76 | char *workNames[] = { "1-On ","1-Off","2-On ","2-Off","PAF","  nc " };
+      |                       ^~~~~~~
+201204_fc_recv_1.ino:76:31: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
+   76 | char *workNames[] = { "1-On ","1-Off","2-On ","2-Off","PAF","  nc " };
+      |                               ^~~~~~~
+201204_fc_recv_1.ino:76:39: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
+   76 | char *workNames[] = { "1-On ","1-Off","2-On ","2-Off","PAF","  nc " };
+      |                                       ^~~~~~~
+201204_fc_recv_1.ino:76:47: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
+   76 | char *workNames[] = { "1-On ","1-Off","2-On ","2-Off","PAF","  nc " };
+      |                                               ^~~~~~~
+201204_fc_recv_1.ino:76:55: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
+   76 | char *workNames[] = { "1-On ","1-Off","2-On ","2-Off","PAF","  nc " };
+      |                                                       ^~~~~
+201204_fc_recv_1.ino:76:61: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
+   76 | char *workNames[] = { "1-On ","1-Off","2-On ","2-Off","PAF","  nc " };
+      |                                                             ^~~~~~~
+201204_fc_recv_1.ino:583:5: error: expected declaration before '}' token
+  583 |     }
+      |     ^
+201204_fc_recv_1.ino: In function 'void announce()':
+201204_fc_recv_1.ino:639:9: warning: unused variable 'averagePulseTime' [-Wunused-variable]
+  639 |     int averagePulseTime;
+      |         ^~~~~~~~~~~~~~~~
+201204_fc_recv_1.ino:640:9: warning: unused variable 'averageNonPulseTime' [-Wunused-variable]
+  640 |     int averageNonPulseTime;
+      |         ^~~~~~~~~~~~~~~~~~~
+../build/module.mk:274: recipe for target '../build/target/user/platform-6-m201204_fc_recv_1.o' failed
+make[2]: *** [../build/target/user/platform-6-m201204_fc_recv_1.o] Error 1

spec/errors.spec.js

@@ -196,3 +196,20 @@ describe('error marker prefixed by ~', () => {
     output[0].tokenLength.should.equal(26);
   });
 });
+
+describe('parsing gcc 9.2 errors', () => {
+  let stdout = null;
+  before(() => {
+    stdout = fs.readFileSync(__dirname + '/9_gcc_9.2.txt');
+  });
+  it('should return an object', () => {
+    const output = parser.parseString(stdout);
+
+    output.length.should.equal(14);
+    output[11].type.should.equal('error');
+    output[11].filename.should.equal('201204_fc_recv_1.ino');
+    output[11].line.should.equal(583);
+    output[11].column.should.equal(5);
+    output[11].text.should.equal(`expected declaration before '}' token`);
+  });
+});