<div dir="ltr">I agree, it'd be better and if it's possible in the future we can switch to it but splitting rollback is a one way road.<br><br>There are other places checking ctx->method so it should be fine.<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jul 22, 2014 at 12:38 PM, Nils Goroll <span dir="ltr"><<a href="mailto:slink@schokola.de" target="_blank">slink@schokola.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Federico,<br>
<div class=""><br>
On 22/07/14 02:26, Federico Schwindt wrote:<br>
> I've changed your diff to avoid splitting rollback. I don't think we should make<br>
> that distinction.<br>
<br>
</div>I had considered this approach also (see yesterday's irc log) but ditched it<br>
because I thought checking context in VCC would be better than checking at<br>
runtime - and VCC would have required more attention to become "context sensitive".<br>
<br>
Other than that I am fine with your change in principle - but I have not tested it.<br>
<span class="HOEnZb"><font color="#888888"><br>
Nils<br>
</font></span></blockquote></div><br></div>