1. Added more match fields for ICMPv6 and MPLS
2. Fixes to avoid unnecessary warnings in generated code
3. Added the general method Match getMatch() to factories and made unit tests use it
diff --git a/java_gen/templates/of_class.java b/java_gen/templates/of_class.java
index 2dc88fb..93ab558 100644
--- a/java_gen/templates/of_class.java
+++ b/java_gen/templates/of_class.java
@@ -145,7 +145,12 @@
static class Reader implements OFMessageReader<${msg.interface.name}> {
@Override
public ${msg.interface.name} readFrom(ChannelBuffer bb) throws OFParseError {
+//:: for prop in msg.members:
+//:: if not prop.is_virtual and (prop.is_length_value or prop.is_field_length_value):
int start = bb.readerIndex();
+//:: break
+//:: #endif
+//:: #endfor
//:: fields_with_length_member = {}
//:: for prop in msg.members:
//:: if prop.is_virtual:
@@ -203,7 +208,9 @@
static class Writer implements OFMessageWriter<${impl_class}> {
@Override
public void write(ChannelBuffer bb, ${impl_class} message) {
+//:: if not msg.is_fixed_length:
int startIndex = bb.writerIndex();
+//:: #endif
//:: fields_with_length_member = {}
//:: for prop in msg.members:
//:: if prop.c_name in fields_with_length_member:
@@ -278,7 +285,9 @@
return false;
if (getClass() != obj.getClass())
return false;
+ //:: if len(msg.data_members) > 0:
${msg.name} other = (${msg.name}) obj;
+ //:: #endif
//:: for prop in msg.data_members:
//:: if prop.java_type.is_primitive:
@@ -300,7 +309,9 @@
@Override
public int hashCode() {
+ //:: if len(msg.data_members) > 0:
final int prime = 31;
+ //:: #endif
int result = 1;
//:: for prop in msg.data_members:
diff --git a/java_gen/templates/of_factory_class.java b/java_gen/templates/of_factory_class.java
index adf5df1..86b092a 100644
--- a/java_gen/templates/of_factory_class.java
+++ b/java_gen/templates/of_factory_class.java
@@ -27,12 +27,15 @@
//::
//:: import itertools
//:: import of_g
+//:: import re
//:: include('_copyright.java')
//:: include('_autogen.java')
package ${factory.package};
+import org.projectfloodlight.openflow.protocol.OFOxmList;
+
//:: include("_imports.java")
public class ${factory.name} implements ${factory.interface.name} {
@@ -45,10 +48,13 @@
}
//:: #endfor
+//:: general_get_match_func_written = False
//:: for i in factory.interface.members:
//:: if i.is_virtual:
//:: continue
//:: #endif
+ //:: is_match_object = re.match('OFMatch.*', i.name) # i.has_version(factory.version) and model.generate_class(i.versioned_class(factory.version)) and i.versioned_class(factory.version).interface.parent_interface == 'Match'
+ //:: unsupported_match_object = is_match_object and not i.has_version(factory.version)
//:: if len(i.writeable_members) > 0:
public ${i.name}.Builder ${factory.interface.method_name(i, builder=True)}() {
@@ -59,6 +65,12 @@
//:: #endif
}
//:: #endif
+ //:: if not general_get_match_func_written and is_match_object and not unsupported_match_object:
+ public Match.Builder buildMatch() {
+ return new ${i.versioned_class(factory.version).name}.Builder();
+ }
+ //:: general_get_match_func_written = True
+ //:: #endif
//:: if len(i.writeable_members) <= 2:
public ${i.name} ${factory.interface.method_name(i, builder=False)}(${", ".join("%s %s" % (p.java_type.public_type, p.name) for p in i.writeable_members)}) {
//:: if i.has_version(factory.version) and model.generate_class(i.versioned_class(factory.version)):
diff --git a/java_gen/templates/of_factory_interface.java b/java_gen/templates/of_factory_interface.java
index 329539f..634362d 100644
--- a/java_gen/templates/of_factory_interface.java
+++ b/java_gen/templates/of_factory_interface.java
@@ -27,6 +27,7 @@
//::
//:: import itertools
//:: import of_g
+//:: import re
//:: include('_copyright.java')
//:: include('_autogen.java')
@@ -52,6 +53,9 @@
${i.name} ${factory.method_name(i, builder=False )}(${", ".join("%s %s" % (p.java_type.public_type, p.name) for p in i.writeable_members)});
//:: #endif
//:: #endfor
+//:: if factory.name == 'OFFactory':
+ Match.Builder buildMatch();
+//:: #endif
OFMessageReader<${factory.base_class}> getReader();
diff --git a/java_gen/templates/of_virtual_class.java b/java_gen/templates/of_virtual_class.java
index 6237b8c..fc6a6c0 100644
--- a/java_gen/templates/of_virtual_class.java
+++ b/java_gen/templates/of_virtual_class.java
@@ -51,7 +51,7 @@
static class Reader implements OFMessageReader<${msg.interface.inherited_declaration()}> {
@Override
- public ${msg.interface.name} readFrom(ChannelBuffer bb) throws OFParseError {
+ public ${msg.interface.inherited_declaration()} readFrom(ChannelBuffer bb) throws OFParseError {
//:: if msg.is_fixed_length:
if(bb.readableBytes() < LENGTH)
//:: else: