On Fri, Mar 18, 2011 at 02:21:46PM -0700, Ethan Jackson wrote:
> > @@ -964,9 +964,9 @@ static bool
> > ?fte_version_equals(const struct fte_version *a, const struct fte_version
> > *b)
> > ?{
> > ? ? return (a->cookie == b->cookie
> > - ? ? ? ? ? ?&& a->idle_timeout != b->idle_timeout
> > - ? ? ? ? ? ?&& a->hard_timeout != b->hard_timeout
> > - ? ? ? ? ? ?&& a->n_actions != b->n_actions
> > + ? ? ? ? ? ?&& a->idle_timeout == b->idle_timeout
> > + ? ? ? ? ? ?&& a->hard_timeout == b->hard_timeout
> > + ? ? ? ? ? ?&& a->n_actions == b->n_actions
> > ? ? ? ? ? ? && !memcmp(a->actions, b->actions,
> > ? ? ? ? ? ? ? ? ? ? ? ?a->n_actions * sizeof *a->actions));
> > ?}
>
> Oooh this makes sense now. Looks like you rebased your fix to the
> wrong commit. Please move this change to the previous one.
Thanks for pointing that out. I moved the change.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
