From 44a8d9d2d56a311293ec86ea40df7126748300a1 Mon Sep 17 00:00:00 2001 From: "Benjamin J. Culkin" Date: Thu, 7 Jun 2018 20:37:51 -0300 Subject: Refactoring The main refactoring here is removing the type field from the various classes, but there are a few other smaller ones. This also contains the grounds for a refactoring on variable use --- .../java/bjc/rgens/parser/elements/RuleCaseElement.java | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'src/main/java/bjc/rgens/parser/elements/RuleCaseElement.java') diff --git a/src/main/java/bjc/rgens/parser/elements/RuleCaseElement.java b/src/main/java/bjc/rgens/parser/elements/RuleCaseElement.java index e0c847a..6aa50fc 100755 --- a/src/main/java/bjc/rgens/parser/elements/RuleCaseElement.java +++ b/src/main/java/bjc/rgens/parser/elements/RuleCaseElement.java @@ -39,16 +39,11 @@ public abstract class RuleCaseElement extends StringCaseElement { if(rl != null) { RGrammar destGrammar = rl.belongsTo; newState.swapGrammar(destGrammar); - String res = destGrammar.generate(actName, newState); - - /* - * @NOTE - * - * :Postprocessing - * - * This is because generate() returns a processed - * string, but modifies the passed in StringBuilder. + /* + * Don't postprocess the string, we should only do that + * once. */ + String res = destGrammar.generate(actName, newState, false); newState.contents = new StringBuilder(res); } else { /* -- cgit v1.2.3