[Lldb-commits] [lldb] [lldb] Adding A new Binding helper for JSONTransport. (PR #159160)

[John Harrison via lldb-commits]

================
@@ -50,17 +56,88 @@ class TransportUnhandledContentsError
   std::string m_unhandled_contents;
 };
 
+class InvalidParams : public llvm::ErrorInfo<InvalidParams> {
+public:
+  static char ID;
+
+  explicit InvalidParams(std::string method, std::string context)
+      : m_method(std::move(method)), m_context(std::move(context)) {}
+
+  void log(llvm::raw_ostream &OS) const override;
+  std::error_code convertToErrorCode() const override;
+
+private:
+  std::string m_method;
----------------
ashgti wrote:

I added a comment about this, this comes from the Binder. In JSONRPC the refer to the remote calls as 'methods'. In the DAP, which isn't precisely a JSONRPC transport, its a 'command' or an 'event', so the terminology isn't consistent. I went with the JSONRPC based name.

https://github.com/llvm/llvm-project/pull/159160