Enable checkstyle rule to catch variables that hide fields
Enable the checkstyle "HiddenFields" rule and fix any issues
that it detects. Suppress these violations for parameters
to setters and constructors.
Added support for the checkstyle comment suppression module
// CHECKSTYLE:OFF and //CHECKSTYLE:ON are now
supported.
Suppressed HiddenField checks for net.onrc.onos.core.packet.*
rather than fix them. These APIs are designed to
require formal parameters and local members having
the same name.
Change-Id: Ibc057603a934842d9c9b9b668f55bc5f5519e7f6
diff --git a/src/main/java/net/onrc/onos/core/flowprogrammer/FlowPusher.java b/src/main/java/net/onrc/onos/core/flowprogrammer/FlowPusher.java
index fe29efd..0be7362 100644
--- a/src/main/java/net/onrc/onos/core/flowprogrammer/FlowPusher.java
+++ b/src/main/java/net/onrc/onos/core/flowprogrammer/FlowPusher.java
@@ -445,20 +445,20 @@
/**
* Set parameters needed for sending messages.
*
- * @param context FloodlightContext used for sending messages.
+ * @param floodlightContext FloodlightContext used for sending messages.
* If null, FlowPusher uses default context.
* @param modContext FloodlightModuleContext used for acquiring
* ThreadPoolService and registering MessageListener.
- * @param factory Factory object to create OFMessage objects.
+ * @param basicFactory Factory object to create OFMessage objects.
* @param damper Message damper used for sending messages.
* If null, FlowPusher creates its own damper object.
*/
- public void init(FloodlightContext context,
+ public void init(FloodlightContext floodlightContext,
FloodlightModuleContext modContext,
- BasicFactory factory,
+ BasicFactory basicFactory,
OFMessageDamper damper) {
- this.context = context;
- this.factory = factory;
+ context = floodlightContext;
+ factory = basicFactory;
this.threadPool = modContext.getServiceImpl(IThreadPoolService.class);
IFloodlightProviderService flservice
= modContext.getServiceImpl(IFloodlightProviderService.class);