diff --git a/common/src/main/java/org/opendaylight/transportpce/common/catalog/PenaltiesComparator.java b/common/src/main/java/org/opendaylight/transportpce/common/catalog/PenaltiesComparator.java index 55f805d023..52c917a89b 100644 --- a/common/src/main/java/org/opendaylight/transportpce/common/catalog/PenaltiesComparator.java +++ b/common/src/main/java/org/opendaylight/transportpce/common/catalog/PenaltiesComparator.java @@ -7,12 +7,13 @@ */ package org.opendaylight.transportpce.common.catalog; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.io.Serializable; import java.util.Comparator; import org.opendaylight.yang.gen.v1.http.org.openroadm.operational.mode.catalog.rev211210.operational.mode.transponder.parameters.Penalties; @SuppressWarnings("serial") -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SE_NO_SERIALVERSIONID", justification = "https://github.com/rzwitserloot/lombok/wiki/WHY-NOT:-serialVersionUID") public class PenaltiesComparator implements Comparator, Serializable { diff --git a/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion121.java b/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion121.java index 37016acb0a..fe9d0ff067 100644 --- a/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion121.java +++ b/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion121.java @@ -9,6 +9,7 @@ package org.opendaylight.transportpce.common.mapping; import com.google.common.util.concurrent.FluentFuture; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.util.ArrayList; import java.util.Collections; import java.util.Comparator; @@ -421,7 +422,7 @@ private String createLogicalConnectionPort(Ports port, int index, int portIndex) return null; } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "SLF4J_UNKNOWN_ARRAY", justification = "False positive") private Map getDegreesMap(String deviceId, Info ordmInfo) { diff --git a/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion221.java b/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion221.java index 2c0386d3ad..a1e24eaab4 100644 --- a/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion221.java +++ b/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion221.java @@ -9,6 +9,7 @@ package org.opendaylight.transportpce.common.mapping; import com.google.common.util.concurrent.FluentFuture; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.util.ArrayList; import java.util.Collections; import java.util.Comparator; @@ -574,7 +575,7 @@ private String createLogicalConnectionPort(Ports port, int index, int portIndex) return null; } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "SLF4J_UNKNOWN_ARRAY", justification = "False positive") private Map getDegreesMap(String deviceId, Info ordmInfo) { diff --git a/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion710.java b/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion710.java index 46b1047bba..607701f7c0 100644 --- a/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion710.java +++ b/common/src/main/java/org/opendaylight/transportpce/common/mapping/PortMappingVersion710.java @@ -9,6 +9,7 @@ package org.opendaylight.transportpce.common.mapping; import com.google.common.util.concurrent.FluentFuture; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.util.ArrayList; import java.util.Collection; import java.util.Collections; @@ -654,7 +655,7 @@ private Map getMcCapabilityProfiles return mcCapabilityProfiles; } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "SLF4J_UNKNOWN_ARRAY", justification = "False positive") private Map getDegreesMap(String deviceId, Info ordmInfo) { diff --git a/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort121ByName.java b/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort121ByName.java index 21166d099f..76b0d8d97e 100644 --- a/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort121ByName.java +++ b/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort121ByName.java @@ -7,6 +7,7 @@ */ package org.opendaylight.transportpce.common.mapping; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.io.Serializable; import java.util.Comparator; import org.opendaylight.yang.gen.v1.http.org.openroadm.device.rev170206.Port; @@ -19,7 +20,7 @@ * */ @SuppressWarnings("serial") -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SE_NO_SERIALVERSIONID", justification = "https://github.com/rzwitserloot/lombok/wiki/WHY-NOT:-serialVersionUID") public class SortPort121ByName implements Comparator, Serializable { diff --git a/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort221ByName.java b/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort221ByName.java index de16646ff3..565d980008 100644 --- a/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort221ByName.java +++ b/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort221ByName.java @@ -7,6 +7,7 @@ */ package org.opendaylight.transportpce.common.mapping; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.io.Serializable; import java.util.Comparator; import org.opendaylight.yang.gen.v1.http.org.openroadm.device.rev181019.Port; @@ -19,7 +20,7 @@ * */ @SuppressWarnings("serial") -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SE_NO_SERIALVERSIONID", justification = "https://github.com/rzwitserloot/lombok/wiki/WHY-NOT:-serialVersionUID") public class SortPort221ByName implements Comparator, Serializable { diff --git a/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort710ByName.java b/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort710ByName.java index 82fae03f9e..b9f8a88417 100644 --- a/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort710ByName.java +++ b/common/src/main/java/org/opendaylight/transportpce/common/mapping/SortPort710ByName.java @@ -7,6 +7,7 @@ */ package org.opendaylight.transportpce.common.mapping; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.io.Serializable; import java.util.Comparator; import org.opendaylight.yang.gen.v1.http.org.openroadm.device.rev200529.Port; @@ -18,7 +19,7 @@ * */ @SuppressWarnings("serial") -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SE_NO_SERIALVERSIONID", justification = "https://github.com/rzwitserloot/lombok/wiki/WHY-NOT:-serialVersionUID") public class SortPort710ByName implements Comparator, Serializable { diff --git a/common/src/main/java/org/opendaylight/transportpce/common/network/RequestProcessor.java b/common/src/main/java/org/opendaylight/transportpce/common/network/RequestProcessor.java index d3a61a69b7..8a4780ca10 100644 --- a/common/src/main/java/org/opendaylight/transportpce/common/network/RequestProcessor.java +++ b/common/src/main/java/org/opendaylight/transportpce/common/network/RequestProcessor.java @@ -11,6 +11,7 @@ import com.google.common.util.concurrent.FluentFuture; import com.google.common.util.concurrent.ListenableFuture; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.util.HashMap; import java.util.Map; import java.util.Optional; @@ -28,7 +29,7 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings(value = "UL_UNRELEASED_LOCK_EXCEPTION_PATH", +@SuppressFBWarnings(value = "UL_UNRELEASED_LOCK_EXCEPTION_PATH", justification = "This appears to be doing exactly the right thing with the finally-clause to release the lock") public class RequestProcessor { diff --git a/common/src/main/java/org/opendaylight/transportpce/common/openroadminterfaces/OpenRoadmInterfaceException.java b/common/src/main/java/org/opendaylight/transportpce/common/openroadminterfaces/OpenRoadmInterfaceException.java index 6f77b56544..2f476fcdd4 100644 --- a/common/src/main/java/org/opendaylight/transportpce/common/openroadminterfaces/OpenRoadmInterfaceException.java +++ b/common/src/main/java/org/opendaylight/transportpce/common/openroadminterfaces/OpenRoadmInterfaceException.java @@ -8,8 +8,10 @@ package org.opendaylight.transportpce.common.openroadminterfaces; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; + @SuppressWarnings("serial") -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SE_NO_SERIALVERSIONID", justification = "https://github.com/rzwitserloot/lombok/wiki/WHY-NOT:-serialVersionUID") public class OpenRoadmInterfaceException extends Exception { diff --git a/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode.java b/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode.java index 368f402030..e1fd3cfe92 100644 --- a/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode.java +++ b/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode.java @@ -8,6 +8,7 @@ package org.opendaylight.transportpce.inventory; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.sql.Connection; import java.sql.PreparedStatement; import java.sql.ResultSet; @@ -16,7 +17,7 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SQL_PREPARED_STATEMENT_GENERATED_FROM_NONCONSTANT_STRING", justification = "TODO review the SQL statement generation process") public class INode { diff --git a/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode121.java b/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode121.java index bce48a2da0..def34d0e9e 100644 --- a/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode121.java +++ b/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode121.java @@ -13,6 +13,7 @@ import static org.opendaylight.transportpce.inventory.utils.StringUtils.prepareDashString; import static org.opendaylight.transportpce.inventory.utils.StringUtils.prepareEmptyString; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.sql.Connection; import java.sql.PreparedStatement; import java.sql.ResultSet; @@ -83,7 +84,7 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SQL_PREPARED_STATEMENT_GENERATED_FROM_NONCONSTANT_STRING", justification = "TODO review the SQL statement generation process") public class INode121 { diff --git a/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode221.java b/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode221.java index 5827dc25aa..f42261759d 100644 --- a/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode221.java +++ b/inventory/src/main/java/org/opendaylight/transportpce/inventory/INode221.java @@ -12,6 +12,7 @@ import static org.opendaylight.transportpce.inventory.utils.StringUtils.prepareDashString; import static org.opendaylight.transportpce.inventory.utils.StringUtils.prepareEmptyString; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.sql.Connection; import java.sql.PreparedStatement; import java.sql.ResultSet; @@ -79,7 +80,7 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SQL_PREPARED_STATEMENT_GENERATED_FROM_NONCONSTANT_STRING", justification = "TODO review the SQL statement generation process") public class INode221 { diff --git a/lighty/src/main/java/io/lighty/controllers/tpce/exception/TechnicalException.java b/lighty/src/main/java/io/lighty/controllers/tpce/exception/TechnicalException.java index 44aa827ecb..29391fdf2d 100644 --- a/lighty/src/main/java/io/lighty/controllers/tpce/exception/TechnicalException.java +++ b/lighty/src/main/java/io/lighty/controllers/tpce/exception/TechnicalException.java @@ -7,7 +7,9 @@ */ package io.lighty.controllers.tpce.exception; -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; + +@SuppressFBWarnings( value = "SE_NO_SERIALVERSIONID", justification = "https://github.com/rzwitserloot/lombok/wiki/WHY-NOT:-serialVersionUID") public class TechnicalException extends RuntimeException { diff --git a/pce/src/main/java/org/opendaylight/transportpce/pce/SortPortsByName.java b/pce/src/main/java/org/opendaylight/transportpce/pce/SortPortsByName.java index a793e6d584..c04aec4adf 100644 --- a/pce/src/main/java/org/opendaylight/transportpce/pce/SortPortsByName.java +++ b/pce/src/main/java/org/opendaylight/transportpce/pce/SortPortsByName.java @@ -7,6 +7,7 @@ */ package org.opendaylight.transportpce.pce; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.io.Serializable; import java.util.Comparator; @@ -17,7 +18,7 @@ * */ @SuppressWarnings("serial") -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SE_NO_SERIALVERSIONID", justification = "https://github.com/rzwitserloot/lombok/wiki/WHY-NOT:-serialVersionUID") public class SortPortsByName implements Comparator, Serializable { diff --git a/pce/src/main/java/org/opendaylight/transportpce/pce/gnpy/utils/AToZComparator.java b/pce/src/main/java/org/opendaylight/transportpce/pce/gnpy/utils/AToZComparator.java index 0f1b32aa1e..44e2f6cb32 100644 --- a/pce/src/main/java/org/opendaylight/transportpce/pce/gnpy/utils/AToZComparator.java +++ b/pce/src/main/java/org/opendaylight/transportpce/pce/gnpy/utils/AToZComparator.java @@ -8,12 +8,13 @@ package org.opendaylight.transportpce.pce.gnpy.utils; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.io.Serializable; import java.util.Comparator; import org.opendaylight.yang.gen.v1.http.org.transportpce.b.c._interface.pathdescription.rev210705.path.description.atoz.direction.AToZ; @SuppressWarnings("serial") -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SE_NO_SERIALVERSIONID", justification = "https://github.com/rzwitserloot/lombok/wiki/WHY-NOT:-serialVersionUID") public class AToZComparator implements Comparator, Serializable { diff --git a/pce/src/main/java/org/opendaylight/transportpce/pce/gnpy/utils/ZToAComparator.java b/pce/src/main/java/org/opendaylight/transportpce/pce/gnpy/utils/ZToAComparator.java index 4571c63801..0fa4885e96 100644 --- a/pce/src/main/java/org/opendaylight/transportpce/pce/gnpy/utils/ZToAComparator.java +++ b/pce/src/main/java/org/opendaylight/transportpce/pce/gnpy/utils/ZToAComparator.java @@ -8,12 +8,13 @@ package org.opendaylight.transportpce.pce.gnpy.utils; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.io.Serializable; import java.util.Comparator; import org.opendaylight.yang.gen.v1.http.org.transportpce.b.c._interface.pathdescription.rev210705.path.description.ztoa.direction.ZToA; @SuppressWarnings("serial") -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SE_NO_SERIALVERSIONID", justification = "https://github.com/rzwitserloot/lombok/wiki/WHY-NOT:-serialVersionUID") public class ZToAComparator implements Comparator, Serializable { diff --git a/pce/src/main/java/org/opendaylight/transportpce/pce/graph/PceGraphEdge.java b/pce/src/main/java/org/opendaylight/transportpce/pce/graph/PceGraphEdge.java index 6983d3f18e..e5a4f251f8 100644 --- a/pce/src/main/java/org/opendaylight/transportpce/pce/graph/PceGraphEdge.java +++ b/pce/src/main/java/org/opendaylight/transportpce/pce/graph/PceGraphEdge.java @@ -7,11 +7,12 @@ */ package org.opendaylight.transportpce.pce.graph; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import org.jgrapht.graph.DefaultWeightedEdge; import org.opendaylight.transportpce.pce.networkanalyzer.PceLink; @SuppressWarnings("serial") -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SE_NO_SERIALVERSIONID", justification = "https://github.com/rzwitserloot/lombok/wiki/WHY-NOT:-serialVersionUID") public class PceGraphEdge extends DefaultWeightedEdge { diff --git a/pce/src/main/java/org/opendaylight/transportpce/pce/networkanalyzer/PceLink.java b/pce/src/main/java/org/opendaylight/transportpce/pce/networkanalyzer/PceLink.java index 408c6c822e..96fc226e05 100644 --- a/pce/src/main/java/org/opendaylight/transportpce/pce/networkanalyzer/PceLink.java +++ b/pce/src/main/java/org/opendaylight/transportpce/pce/networkanalyzer/PceLink.java @@ -8,6 +8,7 @@ package org.opendaylight.transportpce.pce.networkanalyzer; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.io.Serializable; import java.util.Collection; import java.util.HashMap; @@ -32,7 +33,7 @@ import org.slf4j.LoggerFactory; @SuppressWarnings("serial") -@edu.umd.cs.findbugs.annotations.SuppressFBWarnings( +@SuppressFBWarnings( value = "SE_NO_SERIALVERSIONID", justification = "https://github.com/rzwitserloot/lombok/wiki/WHY-NOT:-serialVersionUID") public class PceLink implements Serializable { diff --git a/renderer/src/main/java/org/opendaylight/transportpce/renderer/ModelMappingUtils.java b/renderer/src/main/java/org/opendaylight/transportpce/renderer/ModelMappingUtils.java index fafbd4e57c..b4af7a19a0 100644 --- a/renderer/src/main/java/org/opendaylight/transportpce/renderer/ModelMappingUtils.java +++ b/renderer/src/main/java/org/opendaylight/transportpce/renderer/ModelMappingUtils.java @@ -8,6 +8,7 @@ package org.opendaylight.transportpce.renderer; import com.google.common.util.concurrent.ListenableFuture; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.math.BigDecimal; import java.util.ArrayList; import java.util.HashSet; @@ -390,7 +391,7 @@ private static void populateTreeMap(Map treeMap, Resource r } } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = {"NP_LOAD_OF_KNOWN_NULL_VALUE","RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE"}, justification = "loop when value is not always null - " + "TODO: check if something exists in Java lib") diff --git a/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/DeviceRendererServiceImpl.java b/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/DeviceRendererServiceImpl.java index 5cc550c064..032c474105 100644 --- a/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/DeviceRendererServiceImpl.java +++ b/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/DeviceRendererServiceImpl.java @@ -9,6 +9,7 @@ import com.google.common.collect.Sets; import com.google.common.util.concurrent.FluentFuture; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.util.ArrayList; import java.util.Arrays; import java.util.HashMap; @@ -640,7 +641,7 @@ private void setTopologyForService(String name, Topology topo) } } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "SLF4J_FORMAT_SHOULD_BE_CONST", justification = "Log messages content needs to be formatted before" + "since they are used in the returned object") diff --git a/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/RendererServiceOperationsImpl.java b/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/RendererServiceOperationsImpl.java index fdad346c21..7827b5ca2f 100644 --- a/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/RendererServiceOperationsImpl.java +++ b/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/RendererServiceOperationsImpl.java @@ -11,6 +11,7 @@ import com.google.common.util.concurrent.ListenableFuture; import com.google.common.util.concurrent.ListeningExecutorService; import com.google.common.util.concurrent.MoreExecutors; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import java.util.ArrayList; import java.util.List; import java.util.Map; @@ -271,7 +272,7 @@ public ServiceDeleteOutput call() throws Exception { }); } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "UPM_UNCALLED_PRIVATE_METHOD", justification = "call in call() method") private Uint32 getServiceRate(ServiceImplementationRequestInput input) { @@ -315,7 +316,7 @@ private Uint32 getServiceRate(ServiceImplementationRequestInput input) { .get(serviceName); } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "UPM_UNCALLED_PRIVATE_METHOD", justification = "call in call() method") private ServicePowerTurndownOutput olmPowerTurndown(ServicePathInputData servicePathInputData) @@ -328,7 +329,7 @@ private ServicePowerTurndownOutput olmPowerTurndown(ServicePathInputData service .getResult(); } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "UPM_UNCALLED_PRIVATE_METHOD", justification = "call in call() method") private Optional deviceRendering( @@ -413,7 +414,7 @@ private List deviceRendering( return renderingResults; } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "UPM_UNCALLED_PRIVATE_METHOD", justification = "call in call() method") private List otnDeviceRendering( @@ -471,7 +472,7 @@ private List otnDeviceRendering( return otnRenderingResults; } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "UPM_UNCALLED_PRIVATE_METHOD", justification = "call in call() method") private void olmPowerSetup( @@ -535,7 +536,7 @@ private void olmPowerSetup( powerSetupInputZtoA)); } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "UPM_UNCALLED_PRIVATE_METHOD", justification = "call in call() method") private boolean isServiceActivated(String nodeId, String tpId) { @@ -618,7 +619,7 @@ private boolean verifyPreFecBer(List measurements) { return result <= threshold; } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "UPM_UNCALLED_PRIVATE_METHOD", justification = "call in call() method") private boolean manageServicePathCreation(ServiceImplementationRequestInput input, String serviceType) { @@ -697,7 +698,7 @@ private boolean manageServicePathCreation(ServiceImplementationRequestInput inpu return true; } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "UPM_UNCALLED_PRIVATE_METHOD", justification = "call in call() method") private boolean manageServicePathDeletion(String serviceName, PathDescription pathDescription, String serviceType) @@ -774,7 +775,7 @@ private boolean manageServicePathDeletion(String serviceName, PathDescription pa return true; } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "UPM_UNCALLED_PRIVATE_METHOD", justification = "call in call() method") private boolean manageOtnServicePathCreation( @@ -830,7 +831,7 @@ private boolean manageOtnServicePathCreation( return true; } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "UPM_UNCALLED_PRIVATE_METHOD", justification = "call in call() method") private boolean manageOtnServicePathDeletion( diff --git a/servicehandler/src/main/java/org/opendaylight/transportpce/servicehandler/ModelMappingUtils.java b/servicehandler/src/main/java/org/opendaylight/transportpce/servicehandler/ModelMappingUtils.java index 302227797c..f81dcbd8c2 100644 --- a/servicehandler/src/main/java/org/opendaylight/transportpce/servicehandler/ModelMappingUtils.java +++ b/servicehandler/src/main/java/org/opendaylight/transportpce/servicehandler/ModelMappingUtils.java @@ -9,6 +9,7 @@ import com.google.common.collect.ImmutableMap; import com.google.common.util.concurrent.ListenableFuture; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.pce.rev220808.PathComputationRequestOutput; import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.pce.rev220808.path.computation.request.input.ServiceAEnd; import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.pce.rev220808.path.computation.request.input.ServiceAEndBuilder; @@ -621,7 +622,7 @@ public static ServicePaths mappingServicePaths(ServiceInput serviceInput, PathCo } - @edu.umd.cs.findbugs.annotations.SuppressFBWarnings( + @SuppressFBWarnings( value = "PZLA_PREFER_ZERO_LENGTH_ARRAYS", justification = "not relevant to return and zero length array as we need real pos") public static int[] findTheLongestSubstring(String s1, String s2) {