diff options
author | gjoranv <gv@yahoo-inc.com> | 2017-06-12 13:10:15 +0200 |
---|---|---|
committer | gjoranv <gv@yahoo-inc.com> | 2017-06-12 13:10:15 +0200 |
commit | 41fdfe00ac0dfeb1aab3a9c2db89b717e9b59558 (patch) | |
tree | 4b5c9e3edee55b8353f5fe9696e80169939e7e93 /configgen | |
parent | 0d9a4d0a7d933fe207082a32d82ada45d301bea2 (diff) |
Set the default constructor on config classes private.
- Default constructors for arrays are still public.
Diffstat (limited to 'configgen')
-rw-r--r-- | configgen/src/main/scala/com/yahoo/config/codegen/ConfigGenerator.scala | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/configgen/src/main/scala/com/yahoo/config/codegen/ConfigGenerator.scala b/configgen/src/main/scala/com/yahoo/config/codegen/ConfigGenerator.scala index 716f2a60c33..1716b831773 100644 --- a/configgen/src/main/scala/com/yahoo/config/codegen/ConfigGenerator.scala +++ b/configgen/src/main/scala/com/yahoo/config/codegen/ConfigGenerator.scala @@ -186,6 +186,10 @@ object ConfigGenerator { // TODO @link gives javadoc warnings, although the syntax seems to be valid //def link = "{@link " + {nodeClass(inner)} + "#" + {nodeClass(inner)} + "(Builder)}" def link = {nodeClass(inner)} + "(Builder)" + def visibility = inner match { + case array: InnerCNode if inner.isArray => "public" + case _ => "private" + } <code> |/** @@ -194,7 +198,7 @@ object ConfigGenerator { | * Replaced by {link} | */ |@Deprecated - |public {nodeClass(inner)}() {{ + |{visibility} {nodeClass(inner)}() {{ | this(new Builder(), false); |}} </code>.text.stripMargin.trim |