Re: [ovs-dev] [PATCH] ofproto: remove redundant ofproto class lookup.

2018-02-23 Thread Ben Pfaff
On Fri, Feb 23, 2018 at 01:15:38PM -0600, Mark Michelson wrote: > The ofport_is_internal_or_patch() function called > ofproto_port_open_type() twice. This resulted in the ofproto_class being > looked up twice, when we only needed to look it up once. > > This patch alters

[ovs-dev] [PATCH] ofproto: remove redundant ofproto class lookup.

2018-02-23 Thread Mark Michelson
The ofport_is_internal_or_patch() function called ofproto_port_open_type() twice. This resulted in the ofproto_class being looked up twice, when we only needed to look it up once. This patch alters ofport_is_internal_or_patch() to look up the ofproto_class in-line, and then use the looked up