Javadoc pass and utility cleanup

veinminer-bukkit/src/main/java/wtf/choco/veinminer/ActivationStrategy.java

@@ -7,22 +7,22 @@
 import org.jetbrains.annotations.NotNull;
 
 /**
- * Represents the different methods of activating vein miner.
+ * Represents a different method of activating vein miner.
  */
 public enum ActivationStrategy implements Predicate<VeinMinerPlayer> {
 
     /**
-     * Never activated. Disabled.
+     * Never activated.
      */
     NONE("None", Predicates.alwaysFalse()),
 
     /**
-     * Activated by the client with a client-sided mod.
+     * Activated by the client with the client-sided mod.
      */
     CLIENT("Client", VeinMinerPlayer::isClientKeyPressed),
 
     /**
-     * Activated when a player is holding sneak.
+     * Activated when a player is sneaking.
      */
     SNEAK("Sneak", player -> player.getPlayer().isSneaking()),
 
@@ -45,8 +45,8 @@ private ActivationStrategy(@NotNull String friendlyName, @NotNull Predicate<Vein
     }
 
     /**
-     * Get the friendly name for this activation strategy. In most cases, this is just the name
-     * of the enum with more appropriate capitalization.
+     * Get the friendly name for this activation strategy. This is just the name of the enum
+     * with more appropriate capitalization.
      *
      * @return the friendly name
      */
@@ -58,7 +58,7 @@ public String getFriendlyName() {
     /**
      * Check whether or not the given player has activated this strategy.
      *
-     * @param player the player to check
+     * @param player the player
      *
      * @return true if active, false otherwise
      */
@@ -73,7 +73,7 @@ public boolean test(VeinMinerPlayer player) {
      * This method exists purely for the sake of clarity and simply delegates to
      * {@link #test(VeinMinerPlayer)}.
      *
-     * @param player the player to check
+     * @param player the player
      *
      * @return true if active, false otherwise
      */