>
> TXN_UNCOMMITTED should not be possible here (because we just committed
> it). If you want to check for it (I probably wouldn't bother) then
> I'd use an assertion:
> > + /* Sets the 'status_txn_try_again' if the transaction
> fails. */
> > + if (status == TXN_SUCCESS || status == TXN_UNCOMMITTED) {
> > + status_txn_try_again = false;
> > + } else {
> > + status_txn_try_again = true;
> > + }
>
Thanks for catching this! It should be TXN_UNCHANGED.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
