Move trailing comments 92/71892/1
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 9 May 2018 12:33:10 +0000 (14:33 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 9 May 2018 12:33:36 +0000 (14:33 +0200)
This is to make sonar happy.

Change-Id: Iadb582329c5774c6e302317a70d718869444d2f2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
common/util/src/main/java/org/opendaylight/yangtools/util/concurrent/CachedThreadPoolExecutor.java
common/util/src/main/java/org/opendaylight/yangtools/util/concurrent/FastThreadPoolExecutor.java

index 0db9a83767d9bf0680a42d6844235b97201fc6a4..47fcb03d70333ac671c657d173ec5b7e472fdf91 100644 (file)
@@ -54,9 +54,10 @@ public class CachedThreadPoolExecutor extends ThreadPoolExecutor {
      * @param loggerIdentity
      *               the class to use as logger name for logging uncaught exceptions from the threads.
      */
-    @SuppressWarnings("checkstyle:LoggerFactoryClassParameter") // due to loggerIdentity argument usage
+    // due to loggerIdentity argument usage
+    @SuppressWarnings("checkstyle:LoggerFactoryClassParameter")
     public CachedThreadPoolExecutor(final int maximumPoolSize, final int maximumQueueSize, final String threadPrefix,
-            Class<?> loggerIdentity) {
+            final Class<?> loggerIdentity) {
         // We're using a custom SynchronousQueue that has a backing bounded LinkedBlockingQueue.
         // We don't specify any core threads (first parameter) so, when a task is submitted,
         // the base class will always try to offer to the queue. If there is an existing waiting
@@ -142,7 +143,8 @@ public class CachedThreadPoolExecutor extends ThreadPoolExecutor {
 
         private static final long POLL_WAIT_TIME_IN_MS = 300;
 
-        @SuppressFBWarnings("SE_BAD_FIELD") // Runnable is not Serializable
+        @SuppressFBWarnings("SE_BAD_FIELD")
+        // Runnable is not Serializable
         private final TrackingLinkedBlockingQueue<Runnable> backingQueue;
 
         ExecutorQueue(final int maxBackingQueueSize) {
index b2054936471afe9299bb3a334b476c7c0d0caf83..24159770ff7a7ca34e888f075b68f046508af389 100644 (file)
@@ -43,7 +43,7 @@ public class FastThreadPoolExecutor extends ThreadPoolExecutor {
      *               the class to use as logger name for logging uncaught exceptions from the threads.
      */
     public FastThreadPoolExecutor(final int maximumPoolSize, final int maximumQueueSize, final String threadPrefix,
-            Class<?> loggerIdentity) {
+            final Class<?> loggerIdentity) {
         this(maximumPoolSize, maximumQueueSize, DEFAULT_IDLE_TIMEOUT_IN_SEC, TimeUnit.SECONDS,
               threadPrefix, loggerIdentity);
     }
@@ -90,9 +90,10 @@ public class FastThreadPoolExecutor extends ThreadPoolExecutor {
      * @param loggerIdentity
      *               the class to use as logger name for logging uncaught exceptions from the threads.
      */
-    @SuppressWarnings("checkstyle:LoggerFactoryClassParameter") // due to loggerIdentity argument usage
+    // due to loggerIdentity argument usage
+    @SuppressWarnings("checkstyle:LoggerFactoryClassParameter")
     public FastThreadPoolExecutor(final int maximumPoolSize, final int maximumQueueSize, final long keepAliveTime,
-            final TimeUnit unit, final String threadPrefix, Class<?> loggerIdentity) {
+            final TimeUnit unit, final String threadPrefix, final Class<?> loggerIdentity) {
         // We use all core threads (the first 2 parameters below equal) so, when a task is submitted,
         // if the thread limit hasn't been reached, a new thread will be spawned to execute
         // the task even if there is an existing idle thread in the pool. This is faster than