[PATCH] Add a miniobj macro for pointer acquisitions

Poul-Henning Kamp phk at phk.freebsd.dk
Thu Mar 17 12:50:39 CET 2016


--------
In message <CABoVN9COqLsa72X4Qa-=ft7eqNUbsDiBY2e3fBbzLQQkSFUEDA at mail.gmail.com>, Dridi Boukelmoune writes:

>In this evening's offline self-hackathon in the train I came up with a
>macro for a recurring pattern in Varnish Cache that I also happen to
>use in at least one VMOD so it might benefit others too.

I agree that we should beautify this pattern, but I can't get used
to the word "ACQUIRE" for it, particular because we are inside a
"release" functionality.

Could we call it "TAKE_OBJ_NOTMULL()" ?

That matches what we are doing to the object reference much better.

I think TAKEOVER_OBJ_NOTNULL() would be too clunky?


-- 
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
phk at FreeBSD.ORG         | TCP/IP since RFC 956
FreeBSD committer       | BSD since 4.3-tahoe    
Never attribute to malice what can adequately be explained by incompetence.



More information about the varnish-dev mailing list