<div dir="ltr">Hi,<div><br></div><div>This patch has now been merged. Thanks for the contribution!</div><div><br></div><div>If you'd like to have a look at varnishhist too that would be much appreciated. Have a look at the commit history for some pointers as to missing items in the original patch.</div>
<div><br></div><div>Regards,</div><div>Martin Blix Grydeland</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 13 January 2014 09:55, Martin Blix Grydeland <span dir="ltr"><<a href="mailto:martin@varnish-software.com" target="_blank">martin@varnish-software.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Hi Guillaume,</p>
<p dir="ltr">I'll come back to you with a proper review and reply to this some time next week. This week I am away on a business trip all week and connectivity will be mostly offline.</p>
<p dir="ltr">Martin</p>
<div class="gmail_quote">9. jan. 2014 16:44 skrev "Guillaume Quintard" <<a href="mailto:guillaume.quintard@smartjog.com" target="_blank">guillaume.quintard@smartjog.com</a>> følgende:<div><div class="h5">
<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF">
<div>Ah, I was wondering about it too.
Martin, feel free to give me pointer about what you want changed,
I see a few spots that could be more elegant or consistent, but
you'll have probably more to say.<br>
<br>
On 01/09/2014 05:41 PM, Martin Blix Grydeland wrote:<br>
</div>
<blockquote type="cite">
<p dir="ltr">Not looked at it in detail yet, though first
impression looks good. Will be going in with some tweaks. Thanks
for the reminder though :-) </p>
<p dir="ltr">Main</p>
<div class="gmail_quote">9. jan. 2014 17:23 skrev "Poul-Henning
Kamp" <<a href="mailto:phk@phk.freebsd.dk" target="_blank">phk@phk.freebsd.dk</a>>
følgende:<br type="attribution">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
>As said on IRC, here's a shot at porting varnishtop on the
new logging<br>
>API. I didn't change much, maybe not enough (RB tree
freeing?), but it<br>
>seems to work.<br>
<br>
I'm wading through my inbox, did Martin look at this patch ?<br>
<br>
--<br>
Poul-Henning Kamp | UNIX since Zilog Zeus 3.20<br>
<a href="mailto:phk@FreeBSD.ORG" target="_blank">phk@FreeBSD.ORG</a> | TCP/IP since RFC 956<br>
FreeBSD committer | BSD since 4.3-tahoe<br>
Never attribute to malice what can adequately be explained by
incompetence.<br>
<br>
_______________________________________________<br>
varnish-dev mailing list<br>
<a href="mailto:varnish-dev@varnish-cache.org" target="_blank">varnish-dev@varnish-cache.org</a><br>
<a href="https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev" target="_blank">https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev</a><br>
</blockquote>
</div>
</blockquote>
<br>
<br>
<pre cols="72">--
Guillaume Quintard</pre>
</div>
</blockquote></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div><table border="0" cellpadding="0" cellspacing="0" style="font-size:12px;line-height:1.5em;font-family:'Helvetica Neue',Arial,sans-serif;color:rgb(102,102,102);width:550px;border-top-width:1px;border-top-style:solid;border-top-color:rgb(238,238,238);border-bottom-width:1px;border-bottom-style:solid;border-bottom-color:rgb(238,238,238);margin-top:20px;padding-top:5px;padding-bottom:5px">
<tbody><tr><td width="100"><a href="http://varnish-software.com" target="_blank"><img src="http://www.varnish-software.com/static/media/logo-email.png"></a><span></span><span></span></td><td><strong style="font-size:14px;color:rgb(34,34,34)">Martin Blix Grydeland</strong><br>
Senior Developer | Varnish Software AS<br>Cell: +47 21 98 92 60<br><span style="font-weight:bold">We Make Websites Fly!</span></td></tr></tbody></table></div>
</div>