Mechanical code cleanup (sal-dom-api) 85/46085/3
authorStephen Kitt <skitt@redhat.com>
Thu, 22 Sep 2016 16:22:29 +0000 (18:22 +0200)
committerTom Pantelis <tpanteli@brocade.com>
Fri, 23 Sep 2016 18:41:17 +0000 (18:41 +0000)
* Remove unnecessary type specifiers (use Java 7 <>)
* Remove unnecessary "extends Object" declarations
* Remove unnecessary semi-colons
* Merge identical catch blocks
* Remove redundant modifiers:
  - enum constructors are private by default
  - interface properties are public static final by default
  - interface methods are public abstract by default
  - interfaces are abstract by default
  - inner interfaces are static by default
  - inner classes in interfaces are public static by default

Change-Id: I88c96965215d2f5dbf60967afe4a4ddbaed0072f
Signed-off-by: Stephen Kitt <skitt@redhat.com>
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMMountPointService.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/md/sal/dom/api/DOMNotificationPublishService.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/sal/core/api/Broker.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/sal/core/api/Consumer.java
opendaylight/md-sal/sal-dom-api/src/main/java/org/opendaylight/controller/sal/core/api/Provider.java

index 9286f06..5972273 100644 (file)
@@ -26,7 +26,7 @@ public interface DOMMountPointService extends BrokerService {
 
     ListenerRegistration<MountProvisionListener> registerProvisionListener(MountProvisionListener listener);
 
-    public interface DOMMountPointBuilder {
+    interface DOMMountPointBuilder {
 
         <T extends DOMService> DOMMountPointBuilder addService(Class<T> type,T impl);
 
index a7d2770..b4df6b4 100644 (file)
@@ -70,7 +70,7 @@ public interface DOMNotificationPublishService extends DOMService, BrokerService
      * @throws InterruptedException if interrupted while waiting
      * @throws NullPointerException if notification is null.
      */
-    @Nonnull ListenableFuture<? extends Object> putNotification(@Nonnull DOMNotification notification) throws InterruptedException;
+    @Nonnull ListenableFuture<?> putNotification(@Nonnull DOMNotification notification) throws InterruptedException;
 
     /**
      * Attempt to publish a notification. The result of this method is a {@link ListenableFuture}
@@ -87,7 +87,7 @@ public interface DOMNotificationPublishService extends DOMService, BrokerService
      *         the implementation from accepting the notification for delivery.
      * @throws NullPointerException if notification is null.
      */
-    @Nonnull ListenableFuture<? extends Object> offerNotification(@Nonnull DOMNotification notification);
+    @Nonnull ListenableFuture<?> offerNotification(@Nonnull DOMNotification notification);
 
     /**
      * Attempt to publish a notification. The result of this method is a {@link ListenableFuture}
@@ -108,6 +108,6 @@ public interface DOMNotificationPublishService extends DOMService, BrokerService
      * @throws NullPointerException if notification or unit is null.
      * @throws IllegalArgumentException if timeout is negative.
      */
-    @Nonnull ListenableFuture<? extends Object> offerNotification(@Nonnull DOMNotification notification,
+    @Nonnull ListenableFuture<?> offerNotification(@Nonnull DOMNotification notification,
         @Nonnegative long timeout, @Nonnull TimeUnit unit) throws InterruptedException;
 }
index 500dce3..01f78cd 100644 (file)
@@ -116,7 +116,7 @@ public interface Broker {
      *
      *
      */
-    public interface ConsumerSession {
+    interface ConsumerSession {
 
         boolean isClosed();
 
@@ -156,7 +156,7 @@ public interface Broker {
      * functionality provided by other {@link Provider}s.
      *
      */
-    public interface ProviderSession extends ConsumerSession {
+    interface ProviderSession extends ConsumerSession {
         /**
          * Closes a session between provider and SAL.
          *
index a5ab8ac..f0c4be2 100644 (file)
@@ -35,7 +35,7 @@ public interface Consumer {
      * @param session
      *            Unique session between consumer and SAL.
      */
-    public void onSessionInitiated(ConsumerSession session);
+    void onSessionInitiated(ConsumerSession session);
 
     /**
      * @deprecated - no longer used or needed
@@ -48,13 +48,13 @@ public interface Consumer {
      * }
      */
     @Deprecated
-    public Collection<ConsumerFunctionality> getConsumerFunctionality();
+    Collection<ConsumerFunctionality> getConsumerFunctionality();
 
     /**
      * @deprecated - no longer used or needed
      */
     @Deprecated
-    public interface ConsumerFunctionality {
+    interface ConsumerFunctionality {
 
     }
 }
index 4f32983..4895d10 100644 (file)
@@ -39,7 +39,7 @@ public interface Provider {
      * @param session
      *            Unique session between provider and SAL.
      */
-    public void onSessionInitiated(ProviderSession session);
+    void onSessionInitiated(ProviderSession session);
 
     /**
      * @deprecated - No longer used or needed
@@ -52,13 +52,13 @@ public interface Provider {
      * }
      */
     @Deprecated
-    public Collection<ProviderFunctionality> getProviderFunctionality();
+    Collection<ProviderFunctionality> getProviderFunctionality();
 
     /**
      * @deprecated - no longer used or needed
      */
     @Deprecated
-    public interface ProviderFunctionality {
+    interface ProviderFunctionality {
 
     }
 }

©2013 OpenDaylight, A Linux Foundation Collaborative Project. All Rights Reserved.
OpenDaylight is a registered trademark of The OpenDaylight Project, Inc.
Linux Foundation and OpenDaylight are registered trademarks of the Linux Foundation.
Linux is a registered trademark of Linus Torvalds.