I made that change, thanks for the review. Ethan
On Wed, Sep 4, 2013 at 2:31 PM, Ben Pfaff <b...@nicira.com> wrote: > On Wed, Sep 04, 2013 at 12:37:34PM -0700, Ethan Jackson wrote: >> By hiding struct rule_dpif inside ofproto-dpif, it becomes very clear >> which attributes are accessed by multiple threads and need to be >> protected by locks. >> >> Signed-off-by: Ethan Jackson <et...@nicira.com> > > I'd prefer for the rule_dpif function names to begin with rule_dpif_. > We don't follow a rule like that for most ofproto-dpif.c functions, > but most of those functions are "static", not "extern". > > Acked-by: Ben Pfaff <b...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
