[Lldb-commits] [PATCH] D11790: Fix ObjC++ types Class and id being defined in C and C++ expressions.

[Zachary Turner via lldb-commits]

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.

If there is going to be a setting, then perhaps the setting should be
something like:

lldb.expression-eval-mode = {ObjC++, C++, current-frame}

and the default value of the setting should be "current-frame".

On Fri, Aug 14, 2015 at 12:49 PM Dawn Perchik via lldb-commits <
lldb-commits at lists.llvm.org> wrote:

> dawn added a comment.
>
> Hi Paul,
>
> Some background...  I basically tried the same thing in my original
> http://reviews.llvm.org/D11102 and http://reviews.llvm.org/D11173.  In
> light of Sean's objections in http://reviews.llvm.org/D11173, I created
> http://reviews.llvm.org/D11482 which used an option to control this
> behavior, but Greg and Jim objected to that, so I took their comments and
> revised http://reviews.llvm.org/D11102 to what I believed the consensus
> was.
>
> You should read the comments in http://reviews.llvm.org/D11482 as they
> apply here as well.
>
> My revision of http://reviews.llvm.org/D11102 is what I believe the
> consensus was, so perhaps you would like to abandon this patch in favor of
> that one?
>
> Thanks,
> -Dawn
>
>
> http://reviews.llvm.org/D11790
>
>
>
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150814/81113cd4/attachment.html>