clayborg added inline comments.

================
Comment at: source/API/SBDebugger.cpp:497
+  auto config_up = llvm::make_unique<StructuredData::Dictionary>();
+  config_up->AddBooleanItem("xml", XMLDocument::XMLEnabled());
+
----------------
Do we want a more self documenting format of the JSON? Maybe "xml" is a 
dictionary instead of just a value?

```
{ 
  "xml" : {
    "value" : true, 
    "description" : "A boolean value that indicates if XML support is enabled 
in LLDB" 
  }
}
```


https://reviews.llvm.org/D43333



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

Reply via email to