Update CodeGenPipeline.process to check for errors before writing out…

…put files
deklanowski · Aug 9, 2014 · 8e69a5b · 8e69a5b
1 parent 4a57ed6
commit 8e69a5b
Showing 1 changed file with 27 additions and 6 deletions.
diff --git a/tool/src/org/antlr/v4/codegen/CodeGenPipeline.java b/tool/src/org/antlr/v4/codegen/CodeGenPipeline.java
@@ -64,23 +64,44 @@ public void process() {
 			}
 		}
 
+		// all templates are generated in memory to report the most complete
+		// error information possible, but actually writing output files stops
+		// after the first error is reported
+		int errorCount = g.tool.errMgr.getNumErrors();
+
 		if ( g.isLexer() ) {
 			ST lexer = gen.generateLexer();
-			writeRecognizer(lexer, gen);
+			if (g.tool.errMgr.getNumErrors() == errorCount) {
+				writeRecognizer(lexer, gen);
+			}
 		}
 		else {
 			ST parser = gen.generateParser();
-			writeRecognizer(parser, gen);
+			if (g.tool.errMgr.getNumErrors() == errorCount) {
+				writeRecognizer(parser, gen);
+			}
 			if ( g.tool.gen_listener ) {
-				gen.writeListener(gen.generateListener());
+				ST listener = gen.generateListener();
+				if (g.tool.errMgr.getNumErrors() == errorCount) {
+					gen.writeListener(listener);
+				}
 				if (gen.getTarget().wantsBaseListener()) {
-					gen.writeBaseListener(gen.generateBaseListener());
+					ST baseListener = gen.generateBaseListener();
+					if (g.tool.errMgr.getNumErrors() == errorCount) {
+						gen.writeBaseListener(baseListener);
+					}
 				}
 			}
 			if ( g.tool.gen_visitor ) {
-				gen.writeVisitor(gen.generateVisitor());
+				ST visitor = gen.generateVisitor();
+				if (g.tool.errMgr.getNumErrors() == errorCount) {
+					gen.writeVisitor(visitor);
+				}
 				if (gen.getTarget().wantsBaseVisitor()) {
-					gen.writeBaseVisitor(gen.generateBaseVisitor());
+					ST baseVisitor = gen.generateBaseVisitor();
+					if (g.tool.errMgr.getNumErrors() == errorCount) {
+						gen.writeBaseVisitor(baseVisitor);
+					}
 				}
 			}
 			gen.writeHeaderFile();