From 3b00ba54a294da5d2c2e0e5547f8603f5fc144d2 Mon Sep 17 00:00:00 2001
From: Robert Varga <robert.varga@pantheon.tech>
Date: Sun, 5 May 2019 19:31:33 +0200
Subject: [PATCH] Hide CodecDataObjectGenerator subclasses

These classes should not be visible, hide them.

JIRA: MDSAL-443
Change-Id: I598c50d5de6e700e76988052d3e6747de933deb0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
---
 .../binding/dom/codec/impl/CodecDataObjectGenerator.java | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/binding/mdsal-binding-dom-codec/src/main/java/org/opendaylight/mdsal/binding/dom/codec/impl/CodecDataObjectGenerator.java b/binding/mdsal-binding-dom-codec/src/main/java/org/opendaylight/mdsal/binding/dom/codec/impl/CodecDataObjectGenerator.java
index c97ca3c254..fd62b1be79 100644
--- a/binding/mdsal-binding-dom-codec/src/main/java/org/opendaylight/mdsal/binding/dom/codec/impl/CodecDataObjectGenerator.java
+++ b/binding/mdsal-binding-dom-codec/src/main/java/org/opendaylight/mdsal/binding/dom/codec/impl/CodecDataObjectGenerator.java
@@ -159,11 +159,11 @@ import org.slf4j.LoggerFactory;
  */
 abstract class CodecDataObjectGenerator<T extends CodecDataObject<?>> implements ClassGenerator<T>, BridgeProvider {
     // Not reusable defintion: we can inline NodeContextSuppliers without a problem
-    static final class Fixed<T extends CodecDataObject<?>> extends CodecDataObjectGenerator<T>
+    private static final class Fixed<T extends CodecDataObject<?>> extends CodecDataObjectGenerator<T>
             implements NodeContextSupplierProvider {
         private final ImmutableMap<Method, NodeContextSupplier> properties;
 
-        private Fixed(final Builder<?> template, final ImmutableMap<Method, NodeContextSupplier> properties,
+        Fixed(final Builder<?> template, final ImmutableMap<Method, NodeContextSupplier> properties,
                 final @Nullable Method keyMethod) {
             super(template, keyMethod);
             this.properties = requireNonNull(properties);
@@ -197,13 +197,12 @@ abstract class CodecDataObjectGenerator<T extends CodecDataObject<?>> implements
     }
 
     // Reusable definition: we have to rely on context lookups
-    static final class Reusable<T extends CodecDataObject<?>> extends CodecDataObjectGenerator<T>
+    private static final class Reusable<T extends CodecDataObject<?>> extends CodecDataObjectGenerator<T>
             implements LocalNameProvider {
         private final ImmutableMap<Method, ValueNodeCodecContext> simpleProperties;
         private final Map<Method, Class<?>> daoProperties;
 
-        private Reusable(final Builder<?> template,
-                final ImmutableMap<Method, ValueNodeCodecContext> simpleProperties,
+        Reusable(final Builder<?> template, final ImmutableMap<Method, ValueNodeCodecContext> simpleProperties,
                 final Map<Method, Class<?>> daoProperties, final @Nullable Method keyMethod) {
             super(template, keyMethod);
             this.simpleProperties = requireNonNull(simpleProperties);
-- 
2.36.6