From f028ea6dc555fc5192a96b00b8e96e90dbf6de55 Mon Sep 17 00:00:00 2001 From: "Benjamin J. Culkin" Date: Mon, 9 Oct 2017 16:02:10 -0300 Subject: TODO tagging --- dice-lang/src/bjc/dicelang/dice/DieExpression.java | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) (limited to 'dice-lang/src/bjc/dicelang/dice/DieExpression.java') diff --git a/dice-lang/src/bjc/dicelang/dice/DieExpression.java b/dice-lang/src/bjc/dicelang/dice/DieExpression.java index 71b7a68..b8faeda 100644 --- a/dice-lang/src/bjc/dicelang/dice/DieExpression.java +++ b/dice-lang/src/bjc/dicelang/dice/DieExpression.java @@ -2,24 +2,25 @@ package bjc.dicelang.dice; import java.util.Arrays; +/* + * @NOTE + * I'm not a particularly large fan of sticking everything on this class + * and just documenting which fields are tied together in a non-obvious + * way. I think a class hierarchy might be better, but I am unsure of the + * details. + */ /** * Represents either a die or a die list. * * @author Ben Culkin */ public class DieExpression { - /** - * Is this expression a list? - */ + /** Is this expression a list? */ public final boolean isList; - /** - * The scalar value in this expression, if there is one. - */ + /** The scalar value in this expression, if there is one. */ public Die scalar; - /** - * The list value in this expression, if there is one. - */ + /** The list value in this expression, if there is one. */ public DieList list; /** -- cgit v1.2.3