git.opendaylight Code Review - openflowplugin.git/commit

Code Review / openflowplugin.git / commit

author	Michael Vorburger <vorburger@redhat.com>
	Wed, 14 Nov 2018 14:48:00 +0000 (15:48 +0100)
committer	Michael Vorburger <vorburger@redhat.com>
	Fri, 16 Nov 2018 12:39:19 +0000 (13:39 +0100)
commit	c5f66e01bcc7908e234e502ad172202620f294a8
tree	2229baae2f0edc704e91f4ac68c52dcf53f0eade	tree \| snapshot
parent	04715569cb90858df9aa2e58d1e619e5d152f8fb	commit \| diff

re-activate FindBugs

and fix a lot of logging format bugs which we now automatically detect

and also addresses a few other mistakes identified by FB

Change-Id: I37f0ab54736ba55f23cd3a1fa23c8263a4f3e212
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

46 files changed:

applications/arbitratorreconciliation/api/pom.xml		diff \| blob \| history
applications/arbitratorreconciliation/impl/pom.xml		diff \| blob \| history
applications/forwardingrules-manager/pom.xml		diff \| blob \| history
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/impl/BundleFlowForwarder.java		diff \| blob \| history
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/impl/BundleGroupForwarder.java		diff \| blob \| history
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/impl/FlowForwarder.java		diff \| blob \| history
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/impl/FlowNodeReconciliationImpl.java		diff \| blob \| history
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/impl/GroupForwarder.java		diff \| blob \| history
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/impl/TableForwarder.java		diff \| blob \| history
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/nodeconfigurator/JobEntry.java		diff \| blob \| history
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/util/FrmUtil.java		diff \| blob \| history
applications/forwardingrules-sync/pom.xml		diff \| blob \| history
applications/forwardingrules-sync/src/main/java/org/opendaylight/openflowplugin/applications/frsync/dao/FlowCapableNodeOdlDao.java		diff \| blob \| history
applications/forwardingrules-sync/src/main/java/org/opendaylight/openflowplugin/applications/frsync/impl/AbstractFrmSyncListener.java		diff \| blob \| history
applications/forwardingrules-sync/src/main/java/org/opendaylight/openflowplugin/applications/frsync/impl/clustering/DeviceMastership.java		diff \| blob \| history
applications/forwardingrules-sync/src/main/java/org/opendaylight/openflowplugin/applications/frsync/util/FxChainUtil.java		diff \| blob \| history
applications/forwardingrules-sync/src/main/java/org/opendaylight/openflowplugin/applications/frsync/util/SemaphoreKeeperGuavaImpl.java		diff \| blob \| history
applications/reconciliation-framework/pom.xml		diff \| blob \| history
openflowjava/openflow-protocol-impl/pom.xml		diff \| blob \| history
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/OFDatagramPacketDecoder.java		diff \| blob \| history
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/OFDecoder.java		diff \| blob \| history
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/SslKeyStore.java		diff \| blob \| history
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/connection/OutboundQueueEntry.java		diff \| blob \| history
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/MeterModInputMessageFactory.java		diff \| blob \| history
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/util/ListDeserializer.java		diff \| blob \| history
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/statistics/StatisticsCounters.java		diff \| blob \| history
openflowplugin-common/pom.xml		diff \| blob \| history
openflowplugin-common/src/main/java/org/opendaylight/openflowplugin/common/wait/SimpleTaskRetryLooper.java		diff \| blob \| history
openflowplugin-impl/pom.xml		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/OpenFlowPluginProviderImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/configuration/ConfigurationServiceFactoryImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/configuration/ConfigurationServiceFactoryOsgiImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/connection/ErrorHandlerSimpleImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/connection/listener/ConnectionReadyListenerImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/connection/listener/SystemNotificationsListenerImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/datastore/multipart/AbstractMultipartWriter.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/DeviceContextImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/protocol/serialization/messages/MeterMessageSerializer.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/services/RoleService.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/services/sal/SalFlowServiceImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/services/sal/SalGroupServiceImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/services/sal/SalMeterServiceImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/statistics/StatisticsGatheringUtils.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/statistics/ofpspecific/MessageIntelligenceAgencyImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/translator/FlowRemovedTranslator.java		diff \| blob \| history
parent/pom.xml		diff \| blob \| history

OpenFlow Plugin

RSS Atom