Forge config cleanup

Author: Robotgiggle

Forge/src/main/java/at/petrak/hexcasting/forge/ForgeHexConfig.java

@@ -151,11 +151,6 @@ public static class Server implements HexConfig.ServerConfigAccess {
         private static ForgeConfigSpec.BooleanValue doesTrueNameHaveAmbit;
         private static ForgeConfigSpec.DoubleValue traderScrollChance;
 
-        private static ForgeConfigSpec.ConfigValue<List<? extends String>> fewScrollTables;
-        private static ForgeConfigSpec.ConfigValue<List<? extends String>> someScrollTables;
-        private static ForgeConfigSpec.ConfigValue<List<? extends String>> manyScrollTables;
-
-
         public Server(ForgeConfigSpec.Builder builder) {
             builder.push("Spells");
             maxOpCount = builder.comment("The maximum number of actions that can be executed in one tick, to avoid " +
@@ -181,6 +176,10 @@ public Server(ForgeConfigSpec.Builder builder) {
             traderScrollChance = builder.comment("The chance for wandering traders to sell an Ancient Scroll")
                 .defineInRange("traderScrollChance", DEFAULT_TRADER_SCROLL_CHANCE, 0.0, 1.0);
 
+            // builders for loot (eg. scroll/lore/cypher pools and chances) should go here
+
+            builder.pop();
+
             actionDenyList = builder.comment(
                     "Resource locations of disallowed actions. Trying to cast one of these will result in a mishap.")
                 .defineList("actionDenyList", List.of(), Server::isValidReslocArg);
@@ -193,7 +192,7 @@ public Server(ForgeConfigSpec.Builder builder) {
                 .defineList("tpDimDenyList", DEFAULT_DIM_TP_DENYLIST, Server::isValidReslocArg);
 
             doesTrueNameHaveAmbit = builder.comment(
-                    "when false makes player reference iotas behave as normal entity reference iotas")
+                    "When false, makes player reference iotas behave as normal entity reference iotas")
                 .define("doesTrueNameHaveAmbit", DEFAULT_TRUE_NAME_HAS_AMBIT);
         }