Re: [Lldb-commits] [PATCH] D13799: [lldb-mi] display summary for simple types + refactor (use lldb formatting for all cases)

[Enrico Granata via lldb-commits]

granata.enrico added inline comments.

================
Comment at: include/lldb/API/SBValue.h:372
@@ +371,3 @@
+    bool
+    IsIntegerType(bool& is_signed);
+
----------------
Greg pointed me to SBType::GetBasicType() - this will return to you one of

```
        eBasicTypeSignedChar,
        eBasicTypeUnsignedChar,

```

such that you don't need to add new API to get this information

Also, even if this were necessary, it would go on SBType, definitely not SBValue


http://reviews.llvm.org/D13799



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits