inline unnecessary method.

artasparks · Jun 8, 2016 · e463993 · e463993
1 parent 1d31f96
commit e463993
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/src/rules/goban.js b/src/rules/goban.js
@@ -251,7 +251,11 @@ glift.rules.Goban.prototype = {
    *    the placement was successful.
    */
   addStone: function(pt, color) {
-    if (!glift.util.colors.isLegalColor(color)) throw "Unknown color: " + color;
+    if (!(color === glift.enums.states.BLACK ||
+        color === glift.enums.states.WHITE ||
+        color === glift.enums.states.EMPTY)) {
+      throw "Unknown color: " + color;
+    }
 
     // Add stone fail.  Return a failed StoneResult.
     if (!this.placeable(pt)) {

diff --git a/src/util/colors.js b/src/util/colors.js
@@ -4,12 +4,10 @@ goog.require('glift.util');
 goog.require('glift.enums');
 
 glift.util.colors = {
-  isLegalColor: function(color) {
-    return color === glift.enums.states.BLACK ||
-        color === glift.enums.states.WHITE ||
-        color === glift.enums.states.EMPTY;
-  },
-
+  /**
+   * @param {glift.enums.states} color
+   * @return {glift.enums.states} The opposite color
+   */
   oppositeColor: function(color) {
     if (color === glift.enums.states.BLACK) return glift.enums.states.WHITE;
     if (color === glift.enums.states.WHITE) return glift.enums.states.BLACK;