GitLab

1. Adding OFGroupFeatures and OFGroupDescription
2. Adjusting qualifier for several variabless to protected
3. Adding convenience functions getValues() in OFActionType and OFMeterBandType
4. Updating iterator for setTypes and setActions to iterate using getValues()

1. Fixing bug where invalid length is used in parsing the meter_mod bands 2. Changing the meter_flags to a enum

-  setNetworkSourceMask(short dataLayerType, int networkDestination, int networkMask)
-  setNetworkDestinationMask(short dataLayerType, int networkDestination, int networkMask)

Also clearing out trailing whitespaces and tabs.

Fixing bug in wire protocol for OFRoleRequest/Reply, and adding setRole() that works with OFControllerRole enum.

Setting default METADATA_MASK to Long.MAX_VALUE, as well as changing variable name from metadata to metaData

Adding OFActionOutput(OFPort port) function for convenience, and updating default MAX_LENGTH value to OFPCML_MAX

Fixing exception caused setDataLayerVirtualLan() a result of the "|" operation that returns an int instead of a short.

Updating the SimpleController example to clear flow entries and install default rule on switch instantiation.

Fixing bug with incorrect switch statement in OFOXMField.updateObjectType(), and updating the toString() to print values in HexString when possible.

- Updating type of OFPCML_* constants - Setting default maximum length to OFPCML_MAX - Adding OFFlowMod.setPort(OFPort) for convenience

Updating toString() for OFTableFeaturesPropertyNextTables class to list all the table IDs, instead of incorrect display if the byte[] address.

Removing FactoryAware interface from OFTableFeaturesPropertyActions and OFTableFeaturesPropertyInstructions because factory is not necessary. Those structs only carry the OFInstruction or OFAction header, and not an actual object.

1. Adding checks to see if value object of a OFOXMField is null. For those cases, avoid writeObject() or isAllZero(), 2. Fixing bug in getHeader() where the mask length was not included.

Adding check to convert the Object supplied for value and mask of a OFMatchField to the actual type expected. Without this fix, we will have an exception during writeTo() operation. This prevents any user oversight in doing appropriate typecasting.

Allowing getMatchFieldMask, getDataLayerType, getNetworkProtocol, getTransportSource, getTransportDestination calls to return zero or null, instead of throwing an exception.

Using the correct length of the OFMatch in computing the remaining length of the data to use for parseInstructions

Handling exception that happens when getInPort() is called without there being an IN_PORT field in the match object

Updating OFMatch.readFrom() to set correct matchLength (length without pad) and length (with pad) fields

Updating OFMatch.readFrom() to use OFMatchField.length as a total length of the OXM header and the payload together

Making the length of the mapping used by different enums as a final variable for better readability and avoiding mistakes of overrunning allocation