[master] 468a047 Expose is_hit to VCL as resp.is_hit

Nils Goroll slink at schokola.de
Wed Jan 7 10:55:12 CET 2015


Hi,

On 07/01/15 01:51, Federico Schwindt wrote:
> Can we just do:
> 
>     return (ctx->req->is_hit ? ctx->req->objcore->hits : 0);

Sounds like a good idea.

> in VRT_r_obj_hits() and revert this commit altogether?

will do.

> I'm not sure whether this was discussed but besides the macros now having the
> wrong names (REQ_FLAG and VREQ[WR][01] used for other variables than req) adding
> more knobs is not the way to go IMO.

The REQ in REQ_FLAG and VREQ[WR][01] refers to struct req, not the VCL object.

Thanks, Nils



More information about the varnish-dev mailing list