summaryrefslogtreecommitdiffstats
path: root/configgen
diff options
context:
space:
mode:
authorgjoranv <gv@yahoo-inc.com>2017-06-12 13:10:15 +0200
committergjoranv <gv@yahoo-inc.com>2017-06-12 13:10:15 +0200
commit41fdfe00ac0dfeb1aab3a9c2db89b717e9b59558 (patch)
tree4b5c9e3edee55b8353f5fe9696e80169939e7e93 /configgen
parent0d9a4d0a7d933fe207082a32d82ada45d301bea2 (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.scala6
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