Quoting Bojan Smojver <bojan@rexursive.com>:

> I would stick to the current meaning, mostly because of this:
>
> ---------------------------
>     if (!trans)
>         return APR_DBD_TRANSACTION_COMMIT;
> ---------------------------
>
> In essence, the function returns "the mode in force after the call"
> (actually, I should change the documentation to say that explicitly).

Chris, are you OK with this explanation? Can I go ahead and commit this?

-- 
Bojan