* Eliminated (new) warnings
* Fixed indentation
* Removed excess blank lines
* Improved some varable names
* Removed (unneeded) commented-out code
* Fixed up comments
* Added @Override annotations

ForwardingTest.testForwardDeleteFlowsByDescriptorMultiple() passed in Eclipse, but failed when run by "ant test".
Changed Forwarding.deleteFlowsByDescriptor() to use List<> instead of Set<> to simplify the unit test.
Tests now pass in every configuration I'm able to run them (eclipse single test, eclipse test class, eclipse all tests, ant all tests).

Also general code and comment cleanup.

Firewall cookies

fix LLDP HA role handling

Trivial support for HA role setting in REST API

Fix HA role REST API and add flag to allow us to toggle the controller's behavior when a transition to STANDBY occurs. For now, since we don't have much logic in other modules to handle STANDBY roles, we'll shutdown, but in the future, we should give the ability to stay running and await the next transition to ACTIVE.

allow hex or decimal values for copy_field action parsing

Set version and name when compiling using ant

allow ant to read project version and name from pom.xml. Ideally, we'd put these constants in a separate file, but maven does not allow these tags to contain variable values

Version REST API

Finally! Maven works a lot better. (1) create single jar with all dependencies. This is a much larger jar, but it is directly portable to another system... just use -cf and point to the floodlightdefault.properties file. (2) reduce unit test log verbosity... drastically. (3) include version and name in manifest for new version REST API. Note this also includes the web interface in the jar if the web interface submodule has been fetched prior to building the controller.

Remove old makefile. Add round #1 of version support via REST API. The included code adds a /wm/core/version/json REST API. This will work in Eclipse and in situations where the jar is being run in the build environment with the pom.xml. It will not work when the jar has been compiled and relocated outside of the build environment. For this, we need to come up with a solution to build in the version at compile-time. This will either incorporate it as code or save it somewhere in the classpath.

Conflicts:
	src/main/java/net/floodlightcontroller/forwarding/Forwarding.java

Merged

Moving this to my own branch (I am forking it)

small fixes

fix bug in ACL. UDP is decimal 17, not 11 (is 0x11)

This deletes/add flows from the switch out of forwarding when the firewall rule changes.

Convert any format DPID to hex string DPID for static entry pusher's 'switch' key

convert integer DPIDs specified to the SEP to DatapathId to prevent duplicate DPID flow sets listed just because the DPID was entered in a different format. hexstring is what will be used

OpenFlow 1.5

(1) Fix bug in forwarding where OF1.0 doesn't support tables, so don't set a table ID. (2) Tidy up stats REST API. (3) Remove LLDP info log that should have been debug.

Fixed my delete test case by making sure the message written was correct. (This will be expanded upon in the future).

Added test cases for Forwarding if there is a decision packet.

Updated Firewall.java so that addRule and deleteRule and enableFirewall announce that a rule is being changed in the firewall. Then the IRoutingDecisionChangedListener shall alert Forwarding to update the flowtable accordingly.  None of this is tested and the TopologyManager needs to be fixed.  I might even want to move the change out of TopologyManager altogether since RoutingDecisionChange doesn't concern it.

Add in old URIs for static entry pusher for backwards compatibility. Note this only addresses URI compatibility *not* JSON key:value compatibility. Clean up warnings in LoadBalancer unit test

Finally! :-) OFMessageDamper works. Required creating a custom hashCode() (hashCodeIgnoreXid()) function for all OFMessages we would like to dampen.