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 --- .../bjc/rgens/parser/elements/ExpVariableCaseElement.java | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) (limited to 'src/main/java/bjc/rgens/parser/elements/ExpVariableCaseElement.java') diff --git a/src/main/java/bjc/rgens/parser/elements/ExpVariableCaseElement.java b/src/main/java/bjc/rgens/parser/elements/ExpVariableCaseElement.java index 3972e7a..198b0db 100755 --- a/src/main/java/bjc/rgens/parser/elements/ExpVariableCaseElement.java +++ b/src/main/java/bjc/rgens/parser/elements/ExpVariableCaseElement.java @@ -23,16 +23,12 @@ public class ExpVariableCaseElement extends VariableCaseElement { if(rl != null) { RGrammar destGrammar = rl.belongsTo; newState.swapGrammar(destGrammar); - String res = destGrammar.generate(varDef, state); - - /* - * @NOTE - * - * :Postprocessing - * - * This is because generate() returns a processed - * string, but modifies the passed in StringBuilder. + /* + * Don't post-process the string, we should only do that + * once. */ + String res = destGrammar.generate(varDef, state, false); + newState.contents = new StringBuilder(res); } else { String msg = String.format("No rule '%s' defined", varDef); -- cgit v1.2.3