[Lldb-commits] [lldb] [lldb-dap] Adding support for well typed events. (PR #130104)

[Adrian Vogelsgesang via lldb-commits]

================
@@ -316,6 +316,36 @@ struct Source {
 bool fromJSON(const llvm::json::Value &, Source &, llvm::json::Path);
 llvm::json::Value toJSON(const Source &);
 
+// MARK: Events
+
+// "ExitedEvent": {
+//   "allOf": [ { "$ref": "#/definitions/Event" }, {
+//     "type": "object",
+//     "description": "The event indicates that the debuggee has exited and
+//     returns its exit code.", "properties": {
+//       "event": {
+//         "type": "string",
+//         "enum": [ "exited" ]
+//       },
+//       "body": {
+//         "type": "object",
+//         "properties": {
+//           "exitCode": {
+//             "type": "integer",
+//             "description": "The exit code returned from the debuggee."
+//           }
+//         },
+//         "required": [ "exitCode" ]
+//       }
+//     },
+//     "required": [ "event", "body" ]
+//   }]
+// }
+struct ExitedEventBody {
+  int exitCode;
----------------
vogelsgesang wrote:

I think we should package the "event" property in here.
After all, the comment just above

```
//       "event": {
//         "type": "string",
//         "enum": [ "exited" ]
//       },
```

also provides this piece of information.
That way, `RegisterEvent(llvm::StringLiteral Event)` would no longer need the `Event` parameter

```
  static const char* eventType = "exited";
  int exitCode;
```

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