Makes sense. Thanks. Ethan (iPhone)
On Aug 7, 2012, at 14:53, Ben Pfaff <b...@nicira.com> wrote: > On Tue, Aug 07, 2012 at 02:49:19PM -0700, Ethan Jackson wrote: >>> I dropped the parameter and return value from >>> ofputil_match_from_ofp10_match(). After some thought, it seemed that >>> I only needed the priority calculation in one place (in >>> ofputil_decode_flow_mod()) so I inlined it there. >> >> The other caller didn't need to calculate it? > > It was in decoding a flow stats reply, IIRC. I think that there it's > actually deceptive to change something else to 65535 because it > misrepresents what the switch actually reported. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
