make ConfigTransformer private, don't think it's useful

src/main/java/com/typesafe/config/Config.java

@@ -10,7 +10,7 @@ public final class Config {
     }
 
     public static ConfigObject load(String rootPath) {
-        return ConfigImpl.loadConfig(new ConfigConfig(rootPath, null));
+        return ConfigImpl.loadConfig(new ConfigConfig(rootPath));
     }
 
     private static String getUnits(String s) {

src/main/java/com/typesafe/config/ConfigConfig.java

@@ -6,17 +6,12 @@ package com.typesafe.config;
 public final class ConfigConfig {
 
     private String rootPath;
-    private ConfigTransformer extraTransformer;
 
-    public ConfigConfig(String rootPath, ConfigTransformer extraTransformer) {
+    public ConfigConfig(String rootPath) {
         this.rootPath = rootPath;
     }
 
     public String rootPath() {
         return rootPath;
     }
-
-    public ConfigTransformer extraTransformer() {
-        return extraTransformer;
-    }
 }

src/main/java/com/typesafe/config/ConfigTransformer.java

@@ -1,11 +0,0 @@
-package com.typesafe.config;
-
-/**
- * A ConfigTransformer converts values in the config to other values, most often
- * it's used to parse strings and treat them as some other kind of value.
- * There's no implementation of ConfigValue included in the public API of this
- * library, you have to just create one.
- */
-public interface ConfigTransformer {
-    ConfigValue transform(ConfigValue value, ConfigValueType requested);
-}

src/main/java/com/typesafe/config/impl/AbstractConfigObject.java

@@ -11,7 +11,6 @@ import com.typesafe.config.Config;
 import com.typesafe.config.ConfigException;
 import com.typesafe.config.ConfigObject;
 import com.typesafe.config.ConfigOrigin;
-import com.typesafe.config.ConfigTransformer;
 import com.typesafe.config.ConfigValue;
 import com.typesafe.config.ConfigValueType;
 

src/main/java/com/typesafe/config/impl/ConfigImpl.java

@@ -11,7 +11,6 @@ import java.util.Properties;
 import com.typesafe.config.ConfigConfig;
 import com.typesafe.config.ConfigException;
 import com.typesafe.config.ConfigObject;
-import com.typesafe.config.ConfigTransformer;
 import com.typesafe.config.ConfigValue;
 
 /** This is public but is only supposed to be used by the "config" package */
@@ -30,8 +29,7 @@ public class ConfigImpl {
         if (system != null)
             stack.add(system);
 
-        ConfigTransformer transformer = withExtraTransformer(configConfig
+        ConfigTransformer transformer = withExtraTransformer(null);
-                .extraTransformer());
 
         AbstractConfigObject merged = AbstractConfigObject
                 .merge(new SimpleConfigOrigin("config for "
@@ -40,20 +38,18 @@ public class ConfigImpl {
         return merged;
     }
 
-    public static ConfigObject getEnvironmentAsConfig(
+    public static ConfigObject getEnvironmentAsConfig() {
-            ConfigTransformer extraTransformer) {
         // This should not need to create a new config object
         // as long as the transformer is just the default transformer.
         return AbstractConfigObject.transformed(envVariablesConfig(),
-                withExtraTransformer(extraTransformer));
+                withExtraTransformer(null));
     }
 
-    public static ConfigObject getSystemPropertiesAsConfig(
+    public static ConfigObject getSystemPropertiesAsConfig() {
-            ConfigTransformer extraTransformer) {
         // This should not need to create a new config object
         // as long as the transformer is just the default transformer.
         return AbstractConfigObject.transformed(systemPropertiesConfig(),
-                withExtraTransformer(extraTransformer));
+                withExtraTransformer(null));
     }
 
     private static ConfigTransformer withExtraTransformer(

src/main/java/com/typesafe/config/impl/ConfigTransformer.java

@@ -0,0 +1,17 @@
+package com.typesafe.config.impl;
+
+import com.typesafe.config.ConfigValue;
+import com.typesafe.config.ConfigValueType;
+
+/**
+ * A ConfigTransformer converts values in the config to other values, most often
+ * it's used to parse strings and treat them as some other kind of value.
+ * 
+ * This was originally in the public API but I'm now thinking it is not useful
+ * to customize, so it's not public for now. It's still used internally, but
+ * probably the code could be cleaned up by just hard-coding the equivalent of
+ * the DefaultTransformer.
+ */
+interface ConfigTransformer {
+    ConfigValue transform(ConfigValue value, ConfigValueType requested);
+}

src/main/java/com/typesafe/config/impl/DefaultTransformer.java

@@ -1,9 +1,11 @@
 package com.typesafe.config.impl;
 
-import com.typesafe.config.ConfigTransformer;
 import com.typesafe.config.ConfigValue;
 import com.typesafe.config.ConfigValueType;
 
+/**
+ * Default automatic type transformations.
+ */
 class DefaultTransformer implements ConfigTransformer {
 
     @Override

src/main/java/com/typesafe/config/impl/SimpleConfigObject.java

@@ -7,7 +7,6 @@ import java.util.Set;
 import com.typesafe.config.ConfigException;
 import com.typesafe.config.ConfigObject;
 import com.typesafe.config.ConfigOrigin;
-import com.typesafe.config.ConfigTransformer;
 import com.typesafe.config.ConfigValue;
 
 class SimpleConfigObject extends AbstractConfigObject {

src/main/java/com/typesafe/config/impl/StackConfigObject.java

@@ -8,7 +8,6 @@ import java.util.Map;
 import java.util.Set;
 
 import com.typesafe.config.ConfigOrigin;
-import com.typesafe.config.ConfigTransformer;
 import com.typesafe.config.ConfigValue;
 
 /**

src/main/java/com/typesafe/config/impl/StackTransformer.java

@@ -2,7 +2,6 @@ package com.typesafe.config.impl;
 
 import java.util.List;
 
-import com.typesafe.config.ConfigTransformer;
 import com.typesafe.config.ConfigValue;
 import com.typesafe.config.ConfigValueType;
 

src/main/java/com/typesafe/config/impl/TransformedConfigObject.java

@@ -2,7 +2,6 @@ package com.typesafe.config.impl;
 
 import java.util.Set;
 
-import com.typesafe.config.ConfigTransformer;
 import com.typesafe.config.ConfigValue;
 
 class TransformedConfigObject extends AbstractConfigObject {

src/test/scala/com/typesafe/config/impl/ConfParserTest.scala

@@ -9,10 +9,6 @@ import java.util.HashMap
 
 class ConfParserTest extends TestUtils {
 
-    @org.junit.Before
-    def setup() {
-    }
-
     def parse(s: String): ConfigValue = {
         Parser.parse(SyntaxFlavor.CONF, new SimpleConfigOrigin("test conf string"), s)
     }

src/test/scala/com/typesafe/config/impl/ConfigValueTest.scala

@@ -6,10 +6,6 @@ import com.typesafe.config.ConfigValue
 
 class ConfigValueTest extends TestUtils {
 
-    @org.junit.Before
-    def setup() {
-    }
-
     @Test
     def configIntEquality() {
         val a = new ConfigInt(fakeOrigin(), 42)

src/test/scala/com/typesafe/config/impl/JsonTest.scala

@@ -10,10 +10,6 @@ import java.util.HashMap
 
 class ParseTest extends TestUtils {
 
-    @org.junit.Before
-    def setup() {
-    }
-
     def parse(s: String): ConfigValue = {
         Parser.parse(SyntaxFlavor.JSON, new SimpleConfigOrigin("test json string"), s)
     }

src/test/scala/com/typesafe/config/impl/TokenTest.scala

@@ -5,10 +5,6 @@ import org.junit._
 
 class TokenTest extends TestUtils {
 
-    @org.junit.Before
-    def setup() {
-    }
-
     @Test
     def tokenEquality() {
         checkEqualObjects(Tokens.START, Tokens.START)

src/test/scala/com/typesafe/config/impl/TokenizerTest.scala

@@ -10,10 +10,6 @@ import java.util.HashMap
 
 class TokenizerTest extends TestUtils {
 
-    @org.junit.Before
-    def setup() {
-    }
-
     def tokenTrue = Tokens.newBoolean(fakeOrigin(), true)
     def tokenFalse = Tokens.newBoolean(fakeOrigin(), false)
     def tokenNull = Tokens.newNull(fakeOrigin())