Move onSessionUp()/onMessage() method declarations

There is a completely superfluous suppression here, co-locate methods
to fix the reported violation.

Change-Id: I7aa045870b1b0539ad36de0c0e8d267e97de5400
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/pcep/topology/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/AbstractTopologySessionListener.java b/pcep/topology/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/AbstractTopologySessionListener.java
index 838714885c0ff475094d63505b9bbfb0daff0399..2fa800fd934468f5e14868feec6fb523442540de 100644 (file)
--- a/pcep/topology/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/AbstractTopologySessionListener.java
+++ b/pcep/topology/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/AbstractTopologySessionListener.java
@@ -188,6 +188,8 @@ public abstract class AbstractTopologySessionListener<S, L> implements TopologyS
         }
     }
 
+    protected abstract void onSessionUp(PCEPSession session, PathComputationClientBuilder pccBuilder);
+
     synchronized void updatePccState(final PccSyncState pccSyncState) {
         if (this.nodeState == null) {
             LOG.info("Server Session Manager is closed.");
@@ -329,6 +331,15 @@ public abstract class AbstractTopologySessionListener<S, L> implements TopologyS
         }, MoreExecutors.directExecutor());
     }
 
+    /**
+     * Perform revision-specific message processing when a message arrives.
+     *
+     * @param ctx     Message processing context
+     * @param message Protocol message
+     * @return True if the message type is not handle.
+     */
+    protected abstract boolean onMessage(MessageContext ctx, Message message);
+
     @Override
     public void close() {
         synchronized (this.serverSessionManager) {
@@ -548,19 +559,6 @@ public abstract class AbstractTopologySessionListener<S, L> implements TopologyS
         this.lspData.remove(name);
     }
 
-    @SuppressWarnings("checkstyle:OverloadMethodsDeclarationOrder")
-    protected abstract void onSessionUp(PCEPSession session, PathComputationClientBuilder pccBuilder);
-
-    /**
-     * Perform revision-specific message processing when a message arrives.
-     *
-     * @param ctx     Message processing context
-     * @param message Protocol message
-     * @return True if the message type is not handle.
-     */
-    @SuppressWarnings("checkstyle:OverloadMethodsDeclarationOrder")
-    protected abstract boolean onMessage(MessageContext ctx, Message message);
-
     @Holding("this")
     final String lookupLspName(final L id) {
         return this.lsps.get(requireNonNull(id, "ID parameter null."));