On Thu, Feb 27, 2014 at 09:01:08AM +0900, FUJITA Tomonori wrote: > On Thu, 27 Feb 2014 09:01:33 +0900 > Simon Horman <[email protected]> wrote: > > > On Thu, Feb 27, 2014 at 08:35:56AM +0900, FUJITA Tomonori wrote: > >> On Wed, 26 Feb 2014 15:58:01 +0900 > >> Simon Horman <[email protected]> wrote: > >> > >> > --- > >> > ryu/ofproto/ofproto_v1_3_parser.py | 16 ++++-------- > >> > .../of13/4-55-ofp_table_features_request.packet | Bin 11072 -> 11128 > >> > bytes > >> > ryu/tests/packet_data_generator/src/x4.erl | 14 +++++++++- > >> > .../4-55-ofp_table_features_request.packet.json | 29 > >> > ++++++++++++++++++++- > >> > 4 files changed, 46 insertions(+), 13 deletions(-) > >> > > >> > diff --git a/ryu/ofproto/ofproto_v1_3_parser.py > >> > b/ryu/ofproto/ofproto_v1_3_parser.py > >> > index 0ca4769..c05f90c 100644 > >> > --- a/ryu/ofproto/ofproto_v1_3_parser.py > >> > +++ b/ryu/ofproto/ofproto_v1_3_parser.py > >> > @@ -5126,7 +5126,7 @@ class OFPTableFeaturePropOxm(OFPTableFeatureProp): > >> > @OFPTableFeatureProp.register_type(ofproto.OFPTFPT_EXPERIMENTER_MISS) > >> > class OFPTableFeaturePropExperimenter(OFPTableFeatureProp): > >> > _DATA_ELEMENT_PACK_STR = '!I' > >> > - _PACK_STR = '!II' > >> > + _BODY_PACK_STR = '!II' > >> > > >> > def __init__(self, type_, experimenter=None, exp_type=None, > >> > data=None, length=None): > >> > @@ -5138,13 +5138,13 @@ class > >> > OFPTableFeaturePropExperimenter(OFPTableFeatureProp): > >> > > >> > @classmethod > >> > def _parse_prop(cls, buf): > >> > - (experimenter, exp_type) = struct.unpack_from(cls._PACK_STR, > >> > buf, 0) > >> > + (experimenter, exp_type) = > >> > struct.unpack_from(cls._BODY_PACK_STR, buf, 0) > >> > >> I got a pep8 warning: > >> > >> ryu/ofproto/ofproto_v1_3_parser.py:5141:80: E501 line too long (81 > > >> 79 characters) > >> (experimenter, exp_type) = > >> struct.unpack_from(cls._BODY_PACK_STR, buf, 0) > > > > Sorry for letting that slip through. > > I'll repost the series. > > Only 2/3 patch is fine by me. Thanks.
I read this just after I had reposted the entire series. Let me know if you want 2/3 separately. ------------------------------------------------------------------------------ Flow-based real-time traffic analytics software. Cisco certified tool. Monitor traffic, SLAs, QoS, Medianet, WAAS etc. with NetFlow Analyzer Customize your own dashboards, set traffic alerts and generate reports. Network behavioral analysis & security monitoring. All-in-one tool. http://pubads.g.doubleclick.net/gampad/clk?id=126839071&iu=/4140/ostg.clktrk _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
