portmap: don't use Boolean object when primitive is required

Acked-by: Paul Millar Target: master
dCache · Feb 3, 2020 · 22fdabb · 22fdabb
1 parent c66adbf
commit 22fdabb
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/oncrpc4j-core/src/main/java/org/dcache/oncrpc4j/portmap/OncRpcbindServer.java b/oncrpc4j-core/src/main/java/org/dcache/oncrpc4j/portmap/OncRpcbindServer.java
@@ -89,14 +89,15 @@ private void processV2Call(RpcCall call) throws OncRpcException, IOException {
                         netid.toString(newMapping.getPort()),
                         SERVICE_OWNER_UNSPECIFIED
                 );
-				Boolean found = false;
+				boolean found = false;
                 synchronized(_services) {
 					for ( rpcb c : _services ) {
 						if ( c.getProg() == rpcbMapping.getProg() &&  c.getVers() == rpcbMapping.getVers() && c.getNetid().equals(rpcbMapping.getNetid()) ) {
 							found = true;
+							break;
 						}
 					}
-					if ( found == false) { // only add if not found already
+					if (!found) { // only add if not found already
 						_services.add(rpcbMapping);
 					}
                 }
@@ -112,7 +113,7 @@ private void processV2Call(RpcCall call) throws OncRpcException, IOException {
                         netid.toString(unsetMapping.getPort()),
                         getOwner(call)
                 );
-				Boolean removed = false;
+				boolean removed = false;
                 synchronized(_services) {
 					Set<rpcb> target = new HashSet<>();
 					// lookup entries