Expand Proto functionality

I decided against using doubles to represent protos in shorthand, swhitching to a more complete string.
I also added a range feature, which can be used to furter test Protos.

Author: twizmwazin

src/main/java/in/twizmwaz/cardinal/module/objective/wool/WoolModule.java

@@ -132,11 +132,11 @@ public boolean loadMatch(Match match) {
 
         String locationValue = ParseUtil.getFirstAttribute("location", woolElement, woolsElement);
         Proto proto = match.getMap().getProto();
-        if (locationValue != null && proto.isBefore(1.4)) {
+        if (locationValue != null && proto.isBefore("1.4.0")) {
           errors.add(new ModuleError(this, match.getMap(),
               new String[]{"Attribute \"location\" is supported in proto 1.4.0 or later",
                   "Element at " + located.getLine() + ", " + located.getColumn()}, false));
-        } else if (locationValue == null && proto.isAfterOrAt(1.4)) {
+        } else if (locationValue == null && proto.isAfterOrAt("1.4.0")) {
           errors.add(new ModuleError(this, match.getMap(),
               new String[]{"Attribute \"location\" should be specified for wool in proto 1.4.0 or later",
                   "Element at " + located.getLine() + ", " + located.getColumn()}, false));

src/main/java/in/twizmwaz/cardinal/module/region/RegionModule.java

@@ -130,10 +130,10 @@ public Region getRegion(Match match, Element element, String... alternateAttribu
     if (value != null) {
       String attr = value.getKey();
       Proto proto = match.getMap().getProto();
-      if (attr.equals("id") && proto.isBefore(1.4)) {
+      if (attr.equals("id") && proto.isBefore("1.4.0")) {
         errors.add(new ModuleError(this, match.getMap(),
             new String[]{"Attribute \"id\" should be \"name\" prior to proto 1.4.0"}, false));
-      } else if (attr.equals("name") && proto.isAfterOrAt(1.4)) {
+      } else if (attr.equals("name") && proto.isAfterOrAt("1.4.0")) {
         errors.add(new ModuleError(this, match.getMap(),
             new String[]{"Attribute \"name\" should be \"id\" in proto 1.4.0 or later"}, false));
       }

src/main/java/in/twizmwaz/cardinal/util/Proto.java

@@ -63,16 +63,16 @@ public boolean isAfter(@NonNull Proto proto) {
         || (this.minor == proto.getMinor() && this.patch > proto.getPatch())));
   }
 
-  public boolean isAfter(double proto) {
-    return major + minor * 0.1 + patch * 0.01 > proto;
+  public boolean isAfter(String proto) {
+    return isAfter(parseProto(proto));
   }
 
   public boolean isAfterOrAt(@NonNull Proto proto) {
     return this.equals(proto) || this.isAfter(proto);
   }
 
-  public boolean isAfterOrAt(double proto) {
-    return this.equals(proto) || this.isAfter(proto);
+  public boolean isAfterOrAt(String proto) {
+    return isAfterOrAt(parseProto(proto));
   }
 
   /**
@@ -85,16 +85,28 @@ public boolean isBefore(@NonNull Proto proto) {
         || (this.minor == proto.getMinor() && this.patch < proto.getPatch())));
   }
 
-  public boolean isBefore(double proto) {
-    return major + minor * 0.1 + patch * 0.01 < proto;
+  public boolean isBefore(String proto) {
+    return isBefore(parseProto(proto));
   }
 
   public boolean isBeforeOrAt(@NonNull Proto proto) {
     return this.equals(proto) || this.isBefore(proto);
   }
 
-  public boolean isBeforeOrAt(double proto) {
-    return this.equals(proto) || this.isBefore(proto);
+  public boolean isBeforeOrAt(String proto) {
+    return isBeforeOrAt(parseProto(proto));
+  }
+
+  /**
+   * Returns if a Proto is within a given range.
+   * Note: The range is [min, max), a proto that equals the max will return false.
+   *
+   * @param min The minimum bound, inclusive.
+   * @param max The maximum bound, exclusive.
+   * @return If the proto is within the given range.
+   */
+  public boolean isInRange(Proto min, Proto max) {
+    return isAfterOrAt(min) && isBefore(max);
   }
 
 }