[Lldb-commits] [lldb] r223379 - We should not pretend that an int is a valid ObjC pointer. If your number truly is a pointer, po (id)myNumber is a better alternative. Fixes rdar://16585032

Enrico Granata egranata at apple.com
Thu Dec 4 11:31:51 PST 2014


Author: enrico
Date: Thu Dec  4 13:31:51 2014
New Revision: 223379

URL: http://llvm.org/viewvc/llvm-project?rev=223379&view=rev
Log:
We should not pretend that an int is a valid ObjC pointer. If your number truly is a pointer, po (id)myNumber is a better alternative. Fixes rdar://16585032

Modified:
    lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntime.cpp

Modified: lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntime.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntime.cpp?rev=223379&r1=223378&r2=223379&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntime.cpp (original)
+++ lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntime.cpp Thu Dec  4 13:31:51 2014
@@ -42,11 +42,8 @@ using namespace lldb_private;
 bool
 AppleObjCRuntime::GetObjectDescription (Stream &str, ValueObject &valobj)
 {
-    bool is_signed;
-    // ObjC objects can only be pointers, but we extend this to integer types because an expression might just
-    // result in an address, and we should try that to see if the address is an ObjC object.
-    
-    if (!(valobj.IsPointerType() || valobj.IsIntegerType(is_signed)))
+    // ObjC objects can only be pointers
+    if (!valobj.IsPointerType())
         return false;
     
     // Make the argument list: we pass one arg, the address of our pointer, to the print function.





More information about the lldb-commits mailing list