bulbazord added inline comments.

================
Comment at: lldb/source/Commands/CommandObjectType.cpp:112-113
+    return "format";
+  }
+};
+
----------------
maybe add an `llvm_unreachable` after the switch statement? 


================
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";
----------------
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.";
```




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