Avoid unnecessary list access in generated checkResult calls

src/main/java/org/sablecc/sablecc/GenerateAlternativeCodeForParser.java

@@ -130,13 +130,18 @@ public class GenerateAlternativeCodeForParser extends DepthFirstAdapter
   {
     try
     {
+      PTerm firstTerm = null;
       for(PTerm term : node.getTerms())
       {
+        if(firstTerm == null)
+        {
+          firstTerm = term;
+        }
         macros.apply(file, "ParserNewBodyListAdd", new String[] {getVariableName(term)});
       }
-      if (popCount > 0) {
+      if (firstTerm != null && popCount > 0) {
         // The nodeArrayList variables are numbered starting at 1, so the first popped variable has the number popCount and not popCount-1.
-        macros.apply(file, "ParserNewCheck", new String[] {String.valueOf(popCount)});
+        macros.apply(file, "ParserNewCheck", new String[] {getVariableName(firstTerm), String.valueOf(popCount)});
       }
       macros.apply(file, "ParserNewTail");
     }

src/main/resources/org/sablecc/sablecc/parser.txt

@@ -396,7 +396,7 @@ Macro:ParserNewBodyListAdd
 $
 
 Macro:ParserNewCheck
-        checkResult(nodeList.get(0), nodeArrayList1, nodeArrayList$0$);
+        checkResult($0$, nodeArrayList1, nodeArrayList$1$);
 
 $