On Mon, 8 Apr 2013 19:39:28 +0900 Isaku Yamahata <[email protected]> wrote:
> On Mon, Apr 08, 2013 at 06:08:57PM +0800, can. wrote: >> FYI, I initially found this bug when parse the packets: >> >> pkt = packet.Packet(data) >> for p in pkt.protocols: >> print p >> something_else() >> >> if remove __str__() entirely, would the print work correctly? > > Yes, it works. It prints something like > <lldp object at 0x7fac93cb1090> Can someone send a patch to remove __str__() entirely? The current code is pointless. > Hmm, does it make sense to add __str__ method to all other packet parser > for human readability? Yes. I think that I said so in the past. Especally, I would like to have such on OFP matches and actions. ------------------------------------------------------------------------------ Minimize network downtime and maximize team effectiveness. Reduce network management and security costs.Learn how to hire the most talented Cisco Certified professionals. Visit the Employer Resources Portal http://www.cisco.com/web/learning/employer_resources/index.html _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
