On Fri, 20 Dec 2013 20:54:44 -0800
Srini Seetharaman <[email protected]> wrote:

> I noticed in the function to_match in ryu/lib/ofctl_v1_3.py that there is
> an incorrect match_append statement. I've attached a patch to fix that.
> 
> The patch also includes setting the default priority for the rules in
> ofproto_v1_3 and ofproto_v1_3_parser. I think this is missing.

Please fix the following pep8 warning:

ryu/ofproto/ofproto_v1_3_parser.py:2362:80: E501 line too long (92 >
79 characters)
                 idle_timeout=0, hard_timeout=0,
                 priority=ofproto_v1_3.OFP_DEFAULT_PRIORITY,

Please execute 'run_test.sh' script before posting to check pep8.

Also please send a patch with the description and 'Signed-off-by' like:

http://sourceforge.net/mailarchive/message.php?msg_id=31716803

Thanks!

------------------------------------------------------------------------------
Rapidly troubleshoot problems before they affect your business. Most IT 
organizations don't have a clear picture of how application performance 
affects their revenue. With AppDynamics, you get 100% visibility into your 
Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro!
http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to