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/MathDie.java | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'dice-lang/src/bjc/dicelang/dice/MathDie.java') diff --git a/dice-lang/src/bjc/dicelang/dice/MathDie.java b/dice-lang/src/bjc/dicelang/dice/MathDie.java index 1984581..e4f2953 100644 --- a/dice-lang/src/bjc/dicelang/dice/MathDie.java +++ b/dice-lang/src/bjc/dicelang/dice/MathDie.java @@ -7,6 +7,12 @@ package bjc.dicelang.dice; * */ public class MathDie implements Die { + /* + * @TODO 10/08/17 Ben Culkin :MathGeneralize + * Why do we have the operator types hardcoded, instead of just + * having a general thing for applying a binary operator to dice? + * Fix this by changing it to the more general form. + */ /** * The types of a math operator. * @@ -14,17 +20,11 @@ public class MathDie implements Die { * */ public static enum MathOp { - /** - * Add two dice. - */ + /** Add two dice. */ ADD, - /** - * Subtract two dice. - */ + /** Subtract two dice. */ SUBTRACT, - /** - * Multiply two dice. - */ + /** Multiply two dice. */ MULTIPLY; @Override @@ -118,4 +118,4 @@ public class MathDie implements Die { public String toString() { return left.toString() + " " + type.toString() + " " + right.toString(); } -} \ No newline at end of file +} -- cgit v1.2.3