<div dir="ltr"><div>Committed.<br><br></div>Thanks for the patch Carlos!<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 10, 2015 at 3:50 PM, Carlos Abalde <span dir="ltr"><<a href="mailto:carlos.abalde@gmail.com" target="_blank">carlos.abalde@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On 10 Sep 2015, at 15:00, Poul-Henning Kamp <<a href="mailto:phk@phk.freebsd.dk">phk@phk.freebsd.dk</a>> wrote:<br>
><br>
> --------<br>
> In message <CABoVN9C=<a href="mailto:f53dbP-KogJ7Cepb1t_vZjDpfTgSaCu6Ptdw8GB05Q@mail.gmail.com">f53dbP-KogJ7Cepb1t_vZjDpfTgSaCu6Ptdw8GB05Q@mail.gmail.com</a>><br>
> , Dridi Boukelmoune writes:<br>
>> On Thu, Sep 10, 2015 at 1:26 PM, Carlos Abalde <<a href="mailto:carlos.abalde@gmail.com">carlos.abalde@gmail.com</a>> wrote:<br>
>>> Hi,<br>
>>><br>
>>> When implementing in VCL complex logic (not necessarily related with caching contents -e.g. access control, rate limiting, etc.-) varnishtest is a powerful tool to automate VCL testing. These type of tests sometimes are complex and they are not easy to split in smaller pieces.<br>
>><br>
>> Hi Carlos,<br>
>><br>
>> One nitpick: I believe it should display the default value in the man<br>
>> (and maybe usage too).<br>
><br>
> Also, please use the VNUM_2bytes() function so one can write "512K"<br>
> rather than a implied "K"<br>
<br>
</span>Hi,<br>
<br>
Thanks! New patch attached: improved docs (default value, alphabetical order) and now using VNUM_2bytes() function.<br>
<br>
<br><br>
<br>
Best,<br>
<br>
--<br>
Carlos Abalde<br>
<br>
<br>_______________________________________________<br>
varnish-dev mailing list<br>
<a href="mailto:varnish-dev@varnish-cache.org">varnish-dev@varnish-cache.org</a><br>
<a href="https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev" rel="noreferrer" target="_blank">https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev</a><br></blockquote></div><br></div>