[master] d771942 Hide backends and directors implementation details

Poul-Henning Kamp phk at phk.freebsd.dk
Mon Aug 24 12:04:04 CEST 2015


--------
In message <CAJV_h0Yj-fLdhR7Qv0PgJU1Kg_XZKR_0Cie3aGXKXJWd2rEckw at mail.gmail.com>
, Federico Schwindt writes:

>As I mentioned to Dridi I'm not too thrilled about this.

Personally I'm not to thrilled about it being committed without
any review...

>This pollutes cache.h and causes artificial breakages. I'd prefer, if we
>really want to do this, to take the _int.h approach.

I have refrained from this myself, because I have a really hard time
deciding where the line should be drawn.

>I also had a chat with Martin not so long ago and one of the things that
>came in that discussion is the possibility of installing all the includes.
>This is a step backwards from that.

I'm not too thrilled about that either, because that makes everything
in varnishd an API seen from a VMOD point of view.

(gotta run over at take a tent down...)

-- 
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