Bump org.jenkins-ci:jenkins from 1.140 to 1.141 (#826)

Author: dependabot[bot]

pom.xml

@@ -28,7 +28,7 @@ THE SOFTWARE.
   <parent>
     <groupId>org.jenkins-ci</groupId>
     <artifactId>jenkins</artifactId>
-    <version>1.140</version>
+    <version>1.141</version>
     <relativePath />
   </parent>
 

src/main/java/hudson/remoting/AbstractByteBufferCommandTransport.java

@@ -25,7 +25,6 @@
 
 import edu.umd.cs.findbugs.annotations.NonNull;
 import edu.umd.cs.findbugs.annotations.Nullable;
-import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import java.io.IOException;
 import java.io.ObjectOutputStream;
 import java.nio.Buffer;
@@ -167,9 +166,6 @@ protected void write(ByteBuffer headerAndData) throws IOException {
      * @throws IOException          if something goes wrong during the receive.
      * @throws InterruptedException if interrupted during the receive.
      */
-    @SuppressFBWarnings(
-            value = {"SF_DEAD_STORE_DUE_TO_SWITCH_FALLTHROUGH", "SF_SWITCH_FALLTHROUGH"},
-            justification = "TODO: Remove when https://github.com/spotbugs/spotbugs/issues/3617 is fixed")
     public final void receive(@NonNull ByteBuffer data) throws IOException, InterruptedException {
         while (receiver != null && readCommandIndex > 0) {
             processCommand();

src/main/java/org/jenkinsci/remoting/protocol/ProtocolStack.java

@@ -294,9 +294,6 @@ public void close() throws IOException {
      *
      * @param listener the listener.
      */
-    @SuppressFBWarnings(
-            value = "CWO_CLOSED_WITHOUT_OPENED",
-            justification = "TODO: Remove when https://github.com/spotbugs/spotbugs/issues/3616 is fixed")
     public void addListener(Listener listener) {
         stackLock.writeLock().lock();
         try {
@@ -311,9 +308,6 @@ public void addListener(Listener listener) {
      *
      * @param listener the listener.
      */
-    @SuppressFBWarnings(
-            value = "CWO_CLOSED_WITHOUT_OPENED",
-            justification = "TODO: Remove when https://github.com/spotbugs/spotbugs/issues/3616 is fixed")
     public void removeListener(Listener listener) {
         stackLock.writeLock().lock();
         try {

src/main/java/org/jenkinsci/remoting/protocol/impl/NIONetworkLayer.java

@@ -24,7 +24,6 @@
 package org.jenkinsci.remoting.protocol.impl;
 
 import edu.umd.cs.findbugs.annotations.NonNull;
-import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import java.io.IOException;
 import java.nio.Buffer;
 import java.nio.ByteBuffer;
@@ -123,9 +122,6 @@ public NIONetworkLayer(IOHub ioHub, ReadableByteChannel in, WritableByteChannel
      * {@inheritDoc}
      */
     @Override
-    @SuppressFBWarnings(
-            value = "CWO_CLOSED_WITHOUT_OPENED",
-            justification = "TODO: Remove when https://github.com/spotbugs/spotbugs/issues/3616 is fixed")
     public void ready(boolean accept, boolean connect, boolean read, boolean write) {
         if (LOGGER.isLoggable(Level.FINEST)) {
             LOGGER.log(Level.FINEST, "{0} - entering ready({1}, {2}, {3}, {4})", new Object[] {

src/main/java/org/jenkinsci/remoting/util/VersionNumber.java

@@ -231,8 +231,7 @@ public StringItem(String value, boolean followedByDigit) {
                     case 'm':
                         value = "milestone";
                         break;
-                    default:
-                        /* fall through? */
+                    default: /* fall through? */
                 }
             }
             this.value = ALIASES.getProperty(value, value);