Adrian Moreno <amore...@redhat.com> writes: > diff --git a/python/ovs/flows/ofp.py b/python/ovs/flows/ofp.py > new file mode 100644 > index 000000000..1af06fa01 > --- /dev/null > +++ b/python/ovs/flows/ofp.py > @@ -0,0 +1,453 @@
[...] > + > +class OFPFlow(Flow): [...] > + def __init__(self, ofp_string, id=None): > + """Create a OFPFlow from a flow string. > + > + The string is expected to have the follwoing format: following [...] > + @staticmethod > + def _field_decoder_args(): > + """Returns the KVDecoder arguments needed to decode match fields > + decoding match fields. Huh? > diff --git a/python/ovs/flows/ofp_act.py b/python/ovs/flows/ofp_act.py > new file mode 100644 > index 000000000..9e04445ba > --- /dev/null > +++ b/python/ovs/flows/ofp_act.py > @@ -0,0 +1,243 @@ > +""" Defines decoders for openflow actions. Leading whitespace, and 'OpenFlow' for consistency? [...] > +def decode_learn(action_decoders): [...] > + For this we need to create a wrapper of field_decoders that, for each > + "field=X" key-value we check if X is a field_name or if it's acually actually -- James _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev