<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 21, 2014 at 8:09 PM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank" class="cremed">alp@nuanti.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><br>
On 22/04/2014 00:31, Alp Toker wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'm not really opposed to the idea, though. I just found the current version easier to use.<br>
</blockquote>
<br>
I recommend not modifying the Diag() signature because the feature will be unavailable to other modules until they each update their own Diag() signatures introducing latent churn.<br>
</blockquote>
<br></div>
To be clear, my concern is that other developers will propagate this over time to other Diag() functions as they would have to to be able to use it -- a problem that a fluent extension to DiagnosticBuilder wouldn't suffer from.<br>
</blockquote><div><br></div><div>No problem. I don't really care what it uses, as long as I can get the behaviour I want from the system.  I'll change it to use a manipulator, as you folks agreed downthread.  Patch coming up.</div>
<div><br></div><div><br></div><div>Diego.</div></div></div></div>