[master] 7da6220df Assert
Dridi Boukelmoune
dridi at varni.sh
Mon Dec 16 13:26:35 UTC 2019
On Mon, Dec 16, 2019 at 1:05 PM Nils Goroll <nils.goroll at uplex.de> wrote:
>
> my thinking was that two WS_Asserts were overkill and I think a
> CHECK_OBJ_NOTNULL would actually make more sense.
I think checking we should check have a valid workspace before we
start fiddling with it, and then check that we left a valid workspace
once we're done.
This is what most functions do, and I happen to see that WS_ReserveAll
missed that step, I don't see a point arguing further :-/
It's not like the sanity check has a huge cost.
Dridi
More information about the varnish-commit
mailing list