> @@ -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.
Looks Good otherwise.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
