Pushed with this change: - Another variable was already named “match_len”, so I had to rename it as well. The following patch removes this other variable.
Jarno On Jul 17, 2014, at 10:40 AM, Ben Pfaff <b...@nicira.com> wrote: > On Thu, Jul 17, 2014 at 07:15:17AM -0700, Jarno Rajahalme wrote: >> >> On Jul 15, 2014, at 1:55 PM, Ben Pfaff <b...@nicira.com> wrote: >> >>> On Fri, Jul 11, 2014 at 04:55:55AM -0700, Jarno Rajahalme wrote: >>>> This stylistic change makes the following patch a bit simpler. >>>> >>>> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com> >>> >>> It's hard for me to see the difference between m_bits and n_bits in >>> some of the fonts I use, and a 1-character difference isn't ideal >>> anyway. Could it be named something more explicit like "match_bits" >>> or just "match"? >> >> How about ?match_len? (it is the length of the prefix matching so far)? > > match_len sounds good. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
