Fix eclipse warnings and nits (https://github.com/bigswitch/bigswitchcontroller/pull/2921)

Handle code review from Gregor (https://github.com/bigswitch/bigswitchcontroller/pull/2910)

     removed dead code
     fixed bug on handling display of ethtypes with MSB set
     replaced counterkey string processing on every packet-in with tuple keys
     removed collection of traffic type per switch-port

[BSC-3362] If the flowModRemoved message from a switch has a reason that's beyond the limit, round it back to the maximum valid value.  This change will avoid creating a MessageParseException for invalid flowmod removal reasons.

BT-168 - process all flow-mod failure cases and cleanup the policy that has this flow that failed to be pushed.
BT-169 - set xid in default flow-mods in the bigtap flow cache.
BSC-3361 - suppress spurious warning syslog when switch sends barrier-reply, as it is unhandled. - changes to src/main/java/net/floodlightcontroller/core/internal/Controller.java

Generate launch targets with ant eclipse (https://github.com/bigswitch/bigswitchcontroller/pull/2847)

OF Cluster from a L3 NOF domain.
o VRS detects traffic destined to TEP's MAC and !TEP IP and applies the
  virtual routing logic to the device corresponding to dstIp. It marks the orig dst device
  in the floodlight context
o Forwarding checks for the orig dst device in floodlight ctxt being a TunnelEndPoint.
  If so, it ensures that the packet is received on the designated port for the TEP device and
  computes the route to the destination device as follows:
      o Intra Cluster SourceSwitch --> DestSwitch
      o Prepend {Source Switch,Port}
      o Append {Dest Switch,Port}
o Reconcile changes in VRS and Forwarding to handle such flows.
o Junit tests

Conflicts:
	bigfloodlight/src/main/resources/bigfloodlight.properties

removing unused imports in ControllerTest

Change buildRoute() to use only RouteId as the only parameter as srcId and dstId are included in it.