[trace] avoid using <regex>
authorWalter Erquinigo <wallace@fb.com>
Tue, 22 Sep 2020 21:49:16 +0000 (14:49 -0700)
committerWalter Erquinigo <wallace@fb.com>
Tue, 22 Sep 2020 23:08:58 +0000 (16:08 -0700)
Easy fix based on the feedback by maskray on
https://reviews.llvm.org/D85705.

lldb/source/Plugins/Trace/intel-pt/TraceIntelPTSettingsParser.cpp
lldb/source/Target/TraceSettingsParser.cpp

index c8f90c9..2430f77 100644 (file)
@@ -14,14 +14,14 @@ using namespace llvm;
 
 StringRef TraceIntelPTSettingsParser::GetPluginSchema() {
   return R"({
-  "type": "intel-pt",
-  "pt_cpu": {
-    "vendor": "intel" | "unknown",
-    "family": integer,
-    "model": integer,
-    "stepping": integer
-  }
-})";
+    "type": "intel-pt",
+    "pt_cpu": {
+      "vendor": "intel" | "unknown",
+      "family": integer,
+      "model": integer,
+      "stepping": integer
+    }
+  })";
 }
 
 llvm::Error TraceIntelPTSettingsParser::ParsePTCPU(const json::Object &trace) {
index 2c0f337..9021d93 100644 (file)
@@ -8,7 +8,6 @@
 
 #include "lldb/Target/TraceSettingsParser.h"
 
-#include <regex>
 #include <sstream>
 
 #include "Plugins/Process/Utility/HistoryThread.h"
@@ -123,12 +122,7 @@ StringRef TraceSettingsParser::GetSchema() {
   if (schema.empty()) {
     std::ostringstream schema_builder;
     schema_builder << "{\n \"trace\": ";
-
-    // We need to add spaces to indent correctly the plugin schema
-    std::string plugin_schema(GetPluginSchema());
-    plugin_schema = std::regex_replace(plugin_schema, std::regex("\n"), "\n  ");
-    schema_builder << plugin_schema << ",\n";
-
+    schema_builder << GetPluginSchema().str() << ",\n";
     schema_builder << R"(  "processes": [
     {
       "pid": integer,