On Fri,  5 Jul 2013 14:04:54 +0900
YAMAMOTO Takashi <yamam...@valinux.co.jp> wrote:

> - use netaddr to convert bin <-> text addresses
> - use bytes instead of int to represent ipv4 addresses
> 
> this is after my of-wire json patches and OXM_OF_ patch.
> (logically independent but only tested with them)
> 
> YAMAMOTO Takashi (3):
>   binary <-> text conversion of mac/ipv4/ipv6 addresses
>   replace homegrown bin<->text routines with addrconv for mac and ipv6
>   use bytes instead of int to represent ipv4 addresses
> 
>  ryu/lib/addrconv.py                  | 69 
> ++++++++++++++++++++++++++++++++++++
>  ryu/lib/ip.py                        | 44 +++--------------------
>  ryu/lib/mac.py                       | 21 ++++++-----
>  ryu/lib/packet/arp.py                |  2 +-
>  ryu/lib/packet/ipv4.py               |  4 +--
>  ryu/lib/packet/packet_utils.py       |  2 +-
>  ryu/lib/packet/vrrp.py               | 12 +++----
>  ryu/tests/unit/lib/test_ip.py        |  4 +--
>  ryu/tests/unit/packet/test_arp.py    |  4 +--
>  ryu/tests/unit/packet/test_ipv4.py   |  4 +--
>  ryu/tests/unit/packet/test_packet.py | 10 +++---
>  ryu/tests/unit/packet/test_tcp.py    | 10 +++---
>  ryu/tests/unit/packet/test_udp.py    |  6 ++--
>  ryu/tests/unit/packet/test_vlan.py   |  4 +--
>  ryu/tests/unit/packet/test_vrrp.py   | 31 ++++++++--------
>  15 files changed, 130 insertions(+), 97 deletions(-)
>  create mode 100644 ryu/lib/addrconv.py

Can you describe why we need this change in the first patch in a series?

------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
Ryu-devel mailing list
Ryu-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to