<div dir="ltr">Thank you a lot for explanations and commit!</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 19, 2018 at 4:40 PM Pavel Labath <<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks.<br>
<br>
I agree you're not in the best position to make these kinds of changes<br>
(and I don't think I would have asked you to do them). In fact, I was<br>
already considering just changing that function myself, so I went<br>
ahead and did that now in r337452. As far as I can tell, no tests need<br>
to be updated as a result of this, though I haven't tried running the<br>
test suite on non-linux platforms.<br>
On Thu, 19 Jul 2018 at 12:49, Aleksandr Urakov<br>
<<a href="mailto:aleksandr.urakov@jetbrains.com" target="_blank">aleksandr.urakov@jetbrains.com</a>> wrote:<br>
><br>
> On Thu, Jul 19, 2018 at 2:14 PM Pavel Labath <<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>> wrote:<br>
>><br>
>> I knew I should have stayed quiet :P, but now that I am in, here's my reasoning:<br>
><br>
> Thank you for not staying quiet, I think it's the only way to have a dialog and solve the problems :)<br>
><br>
> I find your argumentation convincing. The problem is that I'm very new in lldb, and I admit that my misgivings may be in vain. So I have no objections if some more experienced lldb developer will commit this (I have no commit access). Or, if you want that it to be exactly my patch, I can make it (test it, find another such places etc.) some later, after current work, and will send a review.<br>
><br>
> What do you think about it?<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="m_5871561292775001576gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Aleksandr Urakov</div><div><span>Software Developer</span></div><div><span>JetBrains</span></div><div><span><a href="http://www.jetbrains.com" target="_blank">http://www.jetbrains.com</a></span></div><div><span>The Drive to Develop</span></div></div></div>