[PATCH 11/13] Generalize the ban reporting to the stevedores using their API.

Martin Blix Grydeland martin at varnish-software.com
Fri Sep 28 16:03:39 CEST 2012


This way any stevedore interested in new  bans can request to be notified
(not just the  persistent).

Review comments:

> I appreciate the generality of STV_NewBan(), but what is the point
> of calling BAN_Spec, rather than passing the ban+len as arguments ?


Reasoning behind BAN_Spec was to preserve the opacity of the ban data
structures. Changed to pass ban+len as arguments.

-- 
Martin Blix Grydeland
Varnish Software AS
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20120928/1569140d/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0011-Generalize-the-ban-reporting-to-the-stevedores-using.diff
Type: application/octet-stream
Size: 4447 bytes
Desc: not available
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20120928/1569140d/attachment.obj>


More information about the varnish-dev mailing list