From 85b81988facc919908d522bd5a7b9ff99cef8f2d Mon Sep 17 00:00:00 2001 From: "Benjamin J. Culkin" Date: Wed, 5 Sep 2018 17:19:53 -0300 Subject: Comment --- src/main/java/bjc/rgens/parser/RGrammarParser.java | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) (limited to 'src/main/java/bjc/rgens/parser') diff --git a/src/main/java/bjc/rgens/parser/RGrammarParser.java b/src/main/java/bjc/rgens/parser/RGrammarParser.java index 3fe4886..4014baa 100755 --- a/src/main/java/bjc/rgens/parser/RGrammarParser.java +++ b/src/main/java/bjc/rgens/parser/RGrammarParser.java @@ -619,6 +619,23 @@ public class RGrammarParser { doChance = true; } else if (partToAdd.matches("\\<\\<\\>")) { + /* + * @NOTE + * + * One, am I even using this feature anywhere? + * As far as I can tell, this says to apply the + * current set of case part rules to the + * previous case part. This may be useful in + * certain cases, but none come to mind at the + * moment. + * + * @PERF + * + * For performance reasons, we may want to + * consider setting the chance/serial values as + * a setting on CaseElement, instead of having + * their own CaseElement type. + */ CaseElement elm = caseParts.popLast(); if(repCount == 0) { -- cgit v1.2.3