Details
-
Improvement
-
Status: Confirmed
-
Medium
-
Resolution: Unresolved
-
None
-
None
-
None
Description
Examination of the use of this class shows we can reduce code duplication by sharing simple action case serialize. Furthermore we can reuse objects across deserializeHeader() invocations. Similar is true for others, hence audit and refactor them from performance.
Attachments
Gerrit Reviews
| # | Subject | Branch | Project | Status | CR | V |
|---|---|---|---|---|---|---|
| 92381,12 | Refactor AbstractActionDeserializer | master | openflowplugin | Status: MERGED | +2 | +1 |
| 92382,12 | Refactor AbstractInstructionSerializer | master | openflowplugin | Status: MERGED | +2 | +1 |
| 92383,12 | Refactor AbstractActionSerializer | master | openflowplugin | Status: MERGED | +2 | +1 |
| 92384,12 | Make AbstractActionSerializer.serialize() final | master | openflowplugin | Status: MERGED | +2 | +1 |
| 92385,12 | Add AbstractEmptyActionSerializer | master | openflowplugin | Status: MERGED | +2 | +1 |
| 92386,13 | Eliminate AbstractEmptyActionSerializer subclasses | master | openflowplugin | Status: MERGED | +2 | +1 |
| 92387,13 | Use ActionContstants in ActionDeserializerInitializer | master | openflowplugin | Status: MERGED | +2 | +1 |
| 92388,13 | Eliminate AbstractEmptyActionDeserializer subclasses | master | openflowplugin | Status: MERGED | +2 | +1 |
| 92391,12 | Make VersionAssignableFactory implement OFDeserializer | master | openflowplugin | Status: MERGED | +2 | +1 |
| 92393,12 | AbstractOxmMatchEntryDeserializer should be an OFDeserializer | master | openflowplugin | Status: MERGED | +2 | +1 |
| 92394,14 | Refactor AbstractOxmMatchEntryDeserializer | master | openflowplugin | Status: MERGED | +2 | +1 |