Add missing @Override annotations 17/9517/1
authorMarek Gradzki <mgradzki@cisco.com>
Wed, 22 Nov 2017 10:17:15 +0000 (11:17 +0100)
committerMarek Gradzki <mgradzki@cisco.com>
Wed, 22 Nov 2017 10:17:17 +0000 (11:17 +0100)
Fix isses reported by Sonar
thanks to IntelliJ's code inspection.

Change-Id: Ief7b4307d4b90e107f010d7feaa307776afaf947
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
19 files changed:
infra/minimal-distribution-core/src/main/java/io/fd/honeycomb/infra/distro/cfgattrs/CfgAttrsModule.java
infra/minimal-distribution-core/src/main/java/io/fd/honeycomb/infra/distro/data/ConfigAndOperationalPipelineModule.java
infra/minimal-distribution-core/src/main/java/io/fd/honeycomb/infra/distro/data/DataTreeProvider.java
infra/minimal-distribution-core/src/main/java/io/fd/honeycomb/infra/distro/data/HoneycombDOMDataBrokerProvider.java
infra/minimal-distribution-core/src/main/java/io/fd/honeycomb/infra/distro/data/PersistingDataTreeProvider.java
infra/minimal-distribution-core/src/main/java/io/fd/honeycomb/infra/distro/data/context/HoneycombContextDOMDataBrokerProvider.java
infra/minimal-distribution-core/src/main/java/io/fd/honeycomb/infra/distro/initializer/InitializerPipelineModule.java
infra/minimal-distribution-core/src/main/java/io/fd/honeycomb/infra/distro/schema/SchemaModule.java
infra/minimal-distribution-core/src/main/java/io/fd/honeycomb/infra/distro/schema/SchemaServiceProvider.java
infra/minimal-distribution-core/src/main/java/io/fd/honeycomb/infra/distro/schema/YangBindingProviderModule.java
infra/northbound/bgp-extensions/extension-common/src/main/java/io/fd/honeycomb/northbound/bgp/extension/CommonBgpExtensionsModule.java
infra/northbound/bgp/src/main/java/io/fd/honeycomb/infra/bgp/BgpConfigurationModule.java
infra/northbound/bgp/src/main/java/io/fd/honeycomb/infra/bgp/BgpModule.java
infra/northbound/bgp/src/main/java/io/fd/honeycomb/infra/bgp/BgpReadersModule.java
infra/northbound/bgp/src/main/java/io/fd/honeycomb/infra/bgp/BgpWritersModule.java
infra/northbound/netconf/src/main/java/io/fd/honeycomb/northbound/netconf/NetconfReadersModule.java
infra/translate-impl/src/main/java/io/fd/honeycomb/translate/impl/read/registry/CompositeReader.java
infra/translate-utils/src/main/java/io/fd/honeycomb/translate/util/read/DelegatingReader.java
infra/translate-utils/src/main/java/io/fd/honeycomb/translate/util/read/KeepaliveReaderWrapper.java

index b22e885..29e79a2 100644 (file)
@@ -24,6 +24,7 @@ import net.jmob.guice.conf.core.ConfigurationModule;
  */
 public class CfgAttrsModule extends AbstractModule {
 
+    @Override
     protected void configure() {
         install(ConfigurationModule.create());
         // Inject non-dependency configuration
index c1e67a5..080caaf 100644 (file)
@@ -45,6 +45,7 @@ public class ConfigAndOperationalPipelineModule extends PrivateModule {
     public static final String HONEYCOMB_CONFIG_NONPERSIST = "honeycomb-config-nopersist";
     public static final String HONEYCOMB_CONFIG = "honeycomb-config";
 
+    @Override
     protected void configure() {
         bind(ShutdownHandler.class).to(ShutdownHandlerImpl.class).in(Singleton.class);
         expose(ShutdownHandler.class);
index 225102e..039a0ad 100644 (file)
@@ -32,6 +32,7 @@ public abstract class DataTreeProvider extends ProviderTrait<DataTree> {
     @Inject
     private HoneycombConfiguration config;
 
+    @Override
     public TipProducingDataTree create() {
         TipProducingDataTree delegate = InMemoryDataTreeFactory.getInstance().create(getType());
         delegate.setSchemaContext(schemaService.getGlobalContext());
@@ -41,6 +42,7 @@ public abstract class DataTreeProvider extends ProviderTrait<DataTree> {
     public abstract TreeType getType();
 
     public static class ConfigDataTreeProvider extends DataTreeProvider {
+        @Override
         public TreeType getType() {
             return TreeType.CONFIGURATION;
         }
@@ -48,6 +50,7 @@ public abstract class DataTreeProvider extends ProviderTrait<DataTree> {
     }
 
     public static class ContextDataTreeProvider extends DataTreeProvider {
+        @Override
         public TreeType getType() {
             return TreeType.OPERATIONAL;
         }
index c23530d..ca94350 100644 (file)
@@ -30,6 +30,7 @@ public final class HoneycombDOMDataBrokerProvider extends ProviderTrait<DOMDataB
     @Inject(optional = true)
     private ReadableDataManager readDataManager;
 
+    @Override
     protected DataBroker create() {
         return readDataManager != null
                 ? DataBroker.create(modDataManager, readDataManager)
index 4e940d6..9da3a00 100644 (file)
@@ -34,6 +34,7 @@ public abstract class PersistingDataTreeProvider extends ProviderTrait<DataTree>
     @Inject
     protected HoneycombConfiguration config;
 
+    @Override
     public DataTree create() {
         return isEnabled()
                 ? new PersistingDataTreeAdapter(getDelegate(), schemaService, Paths.get(getPath()))
@@ -54,14 +55,17 @@ public abstract class PersistingDataTreeProvider extends ProviderTrait<DataTree>
         @Named(ConfigAndOperationalPipelineModule.HONEYCOMB_CONFIG_NONPERSIST)
         private DataTree delegate;
 
+        @Override
         public String getPath() {
             return config.peristConfigPath;
         }
 
+        @Override
         public TreeType getType() {
             return TreeType.CONFIGURATION;
         }
 
+        @Override
         public DataTree getDelegate() {
             return delegate;
         }
@@ -78,14 +82,17 @@ public abstract class PersistingDataTreeProvider extends ProviderTrait<DataTree>
         @Named(ContextPipelineModule.HONEYCOMB_CONTEXT_NOPERSIST)
         private DataTree delegate;
 
+        @Override
         public String getPath() {
             return config.peristContextPath;
         }
 
+        @Override
         public TreeType getType() {
             return TreeType.OPERATIONAL;
         }
 
+        @Override
         public DataTree getDelegate() {
             return delegate;
         }
index e0c38f3..0ddcf2e 100644 (file)
@@ -30,6 +30,7 @@ public final class InitializerPipelineModule extends PrivateModule {
 
     public static final String HONEYCOMB_INITIALIZER = "honeycomb-initializer";
 
+    @Override
     protected void configure() {
         // Create data tree manager on top of non-persisting config data tree
         bind(ModifiableDataManager.class).toProvider(ModifiableDTDelegInitProvider.class).in(Singleton.class);
index c84724f..7e6b89d 100644 (file)
@@ -24,6 +24,7 @@ import org.opendaylight.mdsal.binding.generator.impl.ModuleInfoBackedContext;
 
 public class SchemaModule extends AbstractModule {
 
+    @Override
     protected void configure() {
         bind(ModuleInfoBackedContext.class).toProvider(ModuleInfoBackedCtxProvider.class).in(Singleton.class);
         bind(SchemaService.class).toProvider(SchemaServiceProvider.class).in(Singleton.class);
index ec4a303..4ed3698 100644 (file)
@@ -30,6 +30,7 @@ public final class SchemaServiceProvider extends ProviderTrait<SchemaService> {
     @Inject
     private ModuleInfoBackedContext mibCtx;
 
+    @Override
     public StaticSchemaService create() {
         return new StaticSchemaService(mibCtx.getSchemaContext());
     }
@@ -69,8 +70,10 @@ public final class SchemaServiceProvider extends ProviderTrait<SchemaService> {
                 final SchemaContextListener listener) {
             listener.onGlobalContextUpdated(schemaContext);
             return new ListenerRegistration<SchemaContextListener>() {
+                @Override
                 public void close() {}
 
+                @Override
                 public SchemaContextListener getInstance() {
                     return listener;
                 }
index d705226..d43485f 100644 (file)
@@ -23,6 +23,7 @@ import org.slf4j.LoggerFactory;
 public class YangBindingProviderModule extends AbstractModule {
     private static final Logger LOG = LoggerFactory.getLogger(YangBindingProviderModule.class);
 
+    @Override
     protected void configure() {
         LOG.info("Configuring YangBindingProviderModule");
         bind(YangModulesProvider.YangModules.class).toProvider(YangModulesProvider.class).asEagerSingleton();
index e5406dd..2f8d8cc 100644 (file)
@@ -33,6 +33,7 @@ import org.slf4j.LoggerFactory;
 public final class CommonBgpExtensionsModule extends AbstractModule {
     private static final Logger LOG = LoggerFactory.getLogger(CommonBgpExtensionsModule.class);
 
+    @Override
     protected void configure() {
         LOG.debug("Initializing CommonBgpExtensionsModule");
         // This should be part of BgpModule, but that one is Private and Multibinders + private BASE_MODULES
index 0efe7a1..7f94682 100644 (file)
@@ -24,6 +24,7 @@ import net.jmob.guice.conf.core.ConfigurationModule;
  */
 final class BgpConfigurationModule extends AbstractModule {
 
+    @Override
     protected void configure() {
         install(ConfigurationModule.create());
         // Inject non-dependency configuration
index 943a57e..6aab7c6 100644 (file)
@@ -41,6 +41,7 @@ public final class BgpModule extends PrivateModule {
 
     static final String HONEYCOMB_BGP = "honeycomb-bgp";
 
+    @Override
     protected void configure() {
         LOG.debug("Initializing BgpModule");
         // Create BGPDispatcher BGPDispatcher for creating BGP clients
index 1dc1359..5caf315 100644 (file)
@@ -26,6 +26,7 @@ import org.slf4j.LoggerFactory;
 public final class BgpReadersModule extends AbstractModule {
     private static final Logger LOG = LoggerFactory.getLogger(BgpReadersModule.class);
 
+    @Override
     protected void configure() {
         LOG.debug("Initializing BgpReadersModule");
         // This should be part of BgpModule, but that one is Private and Multibinders + private BASE_MODULES
index fca58b6..8a90629 100644 (file)
@@ -34,6 +34,7 @@ import org.slf4j.LoggerFactory;
 public final class BgpWritersModule extends AbstractModule {
     private static final Logger LOG = LoggerFactory.getLogger(BgpWritersModule.class);
 
+    @Override
     protected void configure() {
         LOG.debug("Initializing BgpWritersModule");
         // This should be part of BgpModule, but that one is Private and Multibinders + private BASE_MODULES
index 88d7a1c..377ce6e 100644 (file)
@@ -32,6 +32,7 @@ public class NetconfReadersModule extends NorthboundAbstractModule<NetconfConfig
         super(new NetconfConfigurationModule(), NetconfConfiguration.class);
     }
 
+    @Override
     protected void configure() {
         if (!getConfiguration().isNetconfEnabled()) {
             LOG.debug("NETCONF Northbound disabled, skipping readers initialization");
index 2eca895..0a87d9b 100644 (file)
@@ -187,6 +187,7 @@ class CompositeReader<D extends DataObject, B extends Builder<D>>
             this.delegate = reader;
         }
 
+        @Override
         public ListReader<D, K, B> getDelegate() {
             return delegate;
         }
index 5136396..a0cee92 100644 (file)
@@ -36,23 +36,27 @@ public interface DelegatingReader<D extends DataObject, B extends Builder<D>> ex
 
     Reader<D, B> getDelegate();
 
+    @Override
     @Nonnull
     default Optional<? extends DataObject> read(@Nonnull final InstanceIdentifier<? extends DataObject> id,
                                                 @Nonnull final ReadContext ctx) throws ReadFailedException {
         return getDelegate().read(id, ctx);
     }
 
+    @Override
     default void readCurrentAttributes(@Nonnull final InstanceIdentifier<D> id,
-                                      @Nonnull final B builder,
-                                      @Nonnull final ReadContext ctx) throws ReadFailedException {
+                                       @Nonnull final B builder,
+                                       @Nonnull final ReadContext ctx) throws ReadFailedException {
         getDelegate().readCurrentAttributes(id, builder, ctx);
     }
 
+    @Override
     @Nonnull
     default B getBuilder(final InstanceIdentifier<D> id) {
         return getDelegate().getBuilder(id);
     }
 
+    @Override
     default void merge(@Nonnull final Builder<? extends DataObject> parentBuilder,
                        @Nonnull final D readValue) {
         getDelegate().merge(parentBuilder, readValue);
@@ -63,6 +67,7 @@ public interface DelegatingReader<D extends DataObject, B extends Builder<D>> ex
         return getDelegate().isPresent(id, built, ctx);
     }
 
+    @Override
     @Nonnull
     default InstanceIdentifier<D> getManagedDataObjectType() {
         return getDelegate().getManagedDataObjectType();
@@ -74,6 +79,7 @@ public interface DelegatingReader<D extends DataObject, B extends Builder<D>> ex
     interface DelegatingListReader<D extends DataObject & Identifiable<K>, K extends Identifier<D>, B extends Builder<D>>
             extends DelegatingReader<D, B>, ListReader<D, K, B> {
 
+        @Override
         ListReader<D, K, B> getDelegate();
 
         @Override
index 9a695a5..b4d8be4 100644 (file)
@@ -122,6 +122,7 @@ public final class KeepaliveReaderWrapper<D extends DataObject, B extends Builde
         @Override
         public <T extends DataObject> void put(final InstanceIdentifier<T> path, final T data) {}
 
+        @Override
         public void close() {}
     }