<div dir="ltr">Is Objective C++ even a thing on non-Apple platforms?  I dont' think we should be forcing a default *anything* to a language that only exists on a subset of supported platforms.  But even ignoring that, I don't think preferential treatment should be given to any particular language.<div><br></div><div>If there is going to be a setting, then perhaps the setting should be something like:</div><div><br></div><div>lldb.expression-eval-mode = {ObjC++, C++, current-frame}</div><div><br></div><div>and the default value of the setting should be "current-frame".</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Aug 14, 2015 at 12:49 PM Dawn Perchik via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">dawn added a comment.<br>
<br>
Hi Paul,<br>
<br>
Some background...  I basically tried the same thing in my original <a href="http://reviews.llvm.org/D11102" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11102</a> and <a href="http://reviews.llvm.org/D11173" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11173</a>.  In light of Sean's objections in <a href="http://reviews.llvm.org/D11173" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11173</a>, I created  <a href="http://reviews.llvm.org/D11482" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11482</a> which used an option to control this behavior, but Greg and Jim objected to that, so I took their comments and revised <a href="http://reviews.llvm.org/D11102" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11102</a> to what I believed the consensus was.<br>
<br>
You should read the comments in <a href="http://reviews.llvm.org/D11482" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11482</a> as they apply here as well.<br>
<br>
My revision of <a href="http://reviews.llvm.org/D11102" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11102</a> is what I believe the consensus was, so perhaps you would like to abandon this patch in favor of that one?<br>
<br>
Thanks,<br>
-Dawn<br>
<br>
<br>
<a href="http://reviews.llvm.org/D11790" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11790</a><br>
<br>
<br>
<br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
</blockquote></div>