Hide DOMMountPointServiceImpl.registerMountPoint implementation 79/77579/3
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 7 Nov 2018 11:39:29 +0000 (12:39 +0100)
committerRobert Varga <nite@hq.sk>
Wed, 21 Nov 2018 17:05:24 +0000 (17:05 +0000)
This adds a trivial package-protected method which holds the
implementation, allowing us to short-circuit some casts.

Change-Id: I41d792c50d87638fc6ba51433cd26bf68980fca2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit e5bd339c5b81732da4496010b03533ec247455fa)

opendaylight/md-sal/sal-dom-broker/src/main/java/org/opendaylight/controller/md/sal/dom/broker/impl/mount/DOMMountPointServiceImpl.java

index 90d513aa9cfdada07e7a2ede5433fddfeca1de7e..ce18d54261390e2c589ac574b4015f2787e46edd 100644 (file)
@@ -97,19 +97,22 @@ public class DOMMountPointServiceImpl implements DOMMountPointService {
      *         public {@link #createMountPoint} interface. As such, this method expects the {@code mountPoint} param
      *         to be of type {@link SimpleDOMMountPoint}.
      */
-    @SuppressWarnings("unchecked")
     @Deprecated
     public ObjectRegistration<DOMMountPoint> registerMountPoint(final DOMMountPoint mountPoint) {
         Preconditions.checkArgument(mountPoint instanceof SimpleDOMMountPoint, "Expected SimpleDOMMountPoint");
+        return doRegisterMountPoint((SimpleDOMMountPoint) mountPoint);
+    }
 
+    @SuppressWarnings("unchecked")
+    ObjectRegistration<DOMMountPoint> doRegisterMountPoint(final SimpleDOMMountPoint mountPoint) {
         final org.opendaylight.mdsal.dom.api.DOMMountPointService.DOMMountPointBuilder delegateBuilder =
-            delegate.createMountPoint(mountPoint.getIdentifier());
+                delegate.createMountPoint(mountPoint.getIdentifier());
 
         if (mountPoint.getSchemaContext() != null) {
             delegateBuilder.addInitialSchemaContext(mountPoint.getSchemaContext());
         }
 
-        ((SimpleDOMMountPoint)mountPoint).getAllServices().forEach(
+        mountPoint.getAllServices().forEach(
             entry -> delegateBuilder.addService((Class<DOMService>)entry.getKey(), entry.getValue()));
 
         final ObjectRegistration<org.opendaylight.mdsal.dom.api.DOMMountPoint> delegateReg = delegateBuilder.register();
@@ -153,7 +156,7 @@ public class DOMMountPointServiceImpl implements DOMMountPointService {
         public ObjectRegistration<DOMMountPoint> register() {
             Preconditions.checkState(mountPoint == null, "Mount point is already built.");
             mountPoint = SimpleDOMMountPoint.create(path, services, schemaContext);
-            return registerMountPoint(mountPoint);
+            return doRegisterMountPoint(mountPoint);
         }
     }
 }