[ovs-dev] [classifier-opt 25/28] classifier: Break cls_rule 'flow' and 'wc' members into new "struct match".

[Ethan Jackson]

Makes sense. Thanks. 

Ethan (iPhone)

On Aug 7, 2012, at 14:53, Ben Pfaff <blp at 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.