Don’t require overriding valueSet() anymore, a do-nothing default is fine.
authorDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Sun, 26 May 2013 15:40:15 +0000 (17:40 +0200)
committerDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Mon, 27 May 2013 20:54:38 +0000 (22:54 +0200)
src/main/java/net/pterodactylus/sonitus/data/controller/AbstractController.java
src/main/java/net/pterodactylus/sonitus/data/controller/Fader.java
src/main/java/net/pterodactylus/sonitus/data/controller/Switch.java

index b355a89..d81086c 100644 (file)
@@ -113,10 +113,14 @@ public abstract class AbstractController<V extends Comparable<V>> implements Con
         * Adjusts the controller. This method is called from {@link
         * #value(Comparable)} if the new value is different from the current value.
         * Also, the value is clamped to fit within the range of this controller.
+        * <p/>
+        * This implementation does nothing.
         *
         * @param value
         *              The new value
         */
-       protected abstract void valueSet(V value);
+       protected void valueSet(V value) {
+               /* do nothing. */
+       }
 
 }
index 279da03..c603199 100644 (file)
@@ -24,7 +24,7 @@ import net.pterodactylus.sonitus.data.Controller;
  *
  * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
  */
-public abstract class Fader extends AbstractController<Double> {
+public class Fader extends AbstractController<Double> {
 
        /**
         * Creates a new fader that is at maximum position.
index 1b997d4..0de8832 100644 (file)
@@ -22,7 +22,7 @@ package net.pterodactylus.sonitus.data.controller;
  *
  * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
  */
-public abstract class Switch extends AbstractController<Boolean> {
+public class Switch extends AbstractController<Boolean> {
 
        /**
         * Creates a new switch that is off.