jingham marked 2 inline comments as done.
jingham added inline comments.

================
Comment at: lldb/source/Commands/CommandObjectType.cpp:879-885
+const char *CommandObjectTypeFormatterDelete::g_short_help_template = 
+    "Delete an existing %s for a type.";
+const char *CommandObjectTypeFormatterDelete::g_long_help_template = 
+    "Delete an existing %s for a type.  Unless you specify a "
+            "specific category or all categories, only the "
+            "'default' category is searched.  The names must be exactly as "
+            "shown in the 'type %s list' output";
----------------
bulbazord wrote:
> You could define these in the class and not out-of-line if you use constexpr. 
> So for example, above in the class you can do:
> 
> ```
>   static constexpr const char *g_short_help_template = "Delete an existing %s 
> for a type.";
> ```
> 
> 
That's neat though I'm not 100% sure why it has to be so ugly...


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148282/new/

https://reviews.llvm.org/D148282

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

Reply via email to