This is the direction that I want to move in anyhow, for supporting OpenFlow 1.1 and later. Thanks for doing this.
One comment: other ofputil_decode_*() functions call ofputil_decode_msg_type() and check the result because they can actually decode more than one message format. But this function only decodes a single message format, so I think that it is better for it to simply assume that the caller provided the correct message format. It could even take a "struct ofp_packet_in *" as parameter (instead of a generic ofp_header). _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
