[Lldb-commits] [PATCH] D17616: Add a set of new plugins to handle Java debugging

Enrico Granata via lldb-commits lldb-commits at lists.llvm.org
Thu Feb 25 16:46:12 PST 2016

granata.enrico added a subscriber: granata.enrico.

Comment at: source/Plugins/Language/Java/JavaLanguage.cpp:81
@@ +80,3 @@
Is there any reason to use hardcoded summaries here?

It looks like IsJavaString is a trivial textual match on the string "java.lang.String". If so, could you please avoid hardcoding this formatter?

The rationale is that an hardcoded formatter can never be turned off by the user, whereas for a normal typename --> formatter match, the user has a way to delete the individual formatter should they desire to do so.

Not a big deal, but would be great to fix if possible.


More information about the lldb-commits mailing list