Remove DOMMountPointServiceImpl.registerMountPoint() 82/77582/7
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 7 Nov 2018 12:58:13 +0000 (13:58 +0100)
committerTom Pantelis <tompantelis@gmail.com>
Fri, 9 Nov 2018 11:55:16 +0000 (11:55 +0000)
This method is a leak and was deprecated in Fluorine, remove it.

Change-Id: I46e2a0eaa1d8baef5c35b6664daa0c606b7ad401
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
opendaylight/md-sal/sal-dom-broker/src/main/java/org/opendaylight/controller/md/sal/dom/broker/impl/mount/DOMMountPointServiceImpl.java

index b73f177cdae0da26584476cee52a5270dec4d00c..bc46b6db3b0ad990ce4e179982cbe172b0651ce7 100644 (file)
@@ -89,21 +89,8 @@ public class DOMMountPointServiceImpl implements DOMMountPointService {
         return delegate.registerProvisionListener(listener);
     }
 
-    /**
-     * Deprecated.
-
-     * @deprecated this method should never have been exposed publicly - registration should be done via the
-     *         public {@link #createMountPoint} interface. As such, this method expects the {@code mountPoint} param
-     *         to be of type {@link SimpleDOMMountPoint}.
-     */
-    @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) {
+    ObjectRegistration<DOMMountPoint> registerMountPoint(final SimpleDOMMountPoint mountPoint) {
         final org.opendaylight.mdsal.dom.api.DOMMountPointService.DOMMountPointBuilder delegateBuilder =
                 delegate.createMountPoint(mountPoint.getIdentifier());
 
@@ -155,7 +142,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 doRegisterMountPoint(mountPoint);
+            return registerMountPoint(mountPoint);
         }
     }
 }