From 054e27ccd0196b197f55532d154fe53940f396f0 Mon Sep 17 00:00:00 2001 From: Havoc Pennington Date: Wed, 16 Nov 2011 09:36:06 -0500 Subject: [PATCH] add comment about not having ConfigValue.parent() --- .../java/com/typesafe/config/impl/AbstractConfigValue.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/main/java/com/typesafe/config/impl/AbstractConfigValue.java b/src/main/java/com/typesafe/config/impl/AbstractConfigValue.java index 0fc7e7e4..3c018191 100644 --- a/src/main/java/com/typesafe/config/impl/AbstractConfigValue.java +++ b/src/main/java/com/typesafe/config/impl/AbstractConfigValue.java @@ -4,6 +4,13 @@ import com.typesafe.config.ConfigOrigin; import com.typesafe.config.ConfigResolveOptions; import com.typesafe.config.ConfigValue; +/** + * + * Trying very hard to avoid a parent reference in config values; when you have + * a tree like this, the availability of parent() tends to result in a lot of + * improperly-factored and non-modular code. Please don't add parent(). + * + */ abstract class AbstractConfigValue implements ConfigValue { final private ConfigOrigin origin;