Skip to content

Commit fca2cc0

Browse files
committed
Move the ExceptionDetails type info closer to request_exceptionInfo
1 parent 7641ce0 commit fca2cc0

File tree

3 files changed

+35
-43
lines changed

3 files changed

+35
-43
lines changed

lldb/tools/lldb-dap/JSONUtils.cpp

Lines changed: 0 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -1257,46 +1257,6 @@ llvm::json::Value CreateVariable(lldb::SBValue v, int64_t variablesReference,
12571257
return llvm::json::Value(std::move(object));
12581258
}
12591259

1260-
// "ExceptionDetails": {
1261-
// "type": "object",
1262-
// "description": "Detailed information about an exception that has
1263-
// occurred.", "properties": {
1264-
// "message": {
1265-
// "type": "string",
1266-
// "description": "Message contained in the exception."
1267-
// },
1268-
// "typeName": {
1269-
// "type": "string",
1270-
// "description": "Short type name of the exception object."
1271-
// },
1272-
// "fullTypeName": {
1273-
// "type": "string",
1274-
// "description": "Fully-qualified type name of the exception object."
1275-
// },
1276-
// "evaluateName": {
1277-
// "type": "string",
1278-
// "description": "An expression that can be evaluated in the current
1279-
// scope to obtain the exception object."
1280-
// },
1281-
// "stackTrace": {
1282-
// "type": "string",
1283-
// "description": "Stack trace at the time the exception was thrown."
1284-
// },
1285-
// "innerException": {
1286-
// "type": "array",
1287-
// "items": {
1288-
// "$ref": "#/definitions/ExceptionDetails"
1289-
// },
1290-
// "description": "Details of the exception contained by this exception,
1291-
// if any."
1292-
// }
1293-
// }
1294-
// },
1295-
llvm::json::Value CreateExceptionDetails() {
1296-
llvm::json::Object object;
1297-
return llvm::json::Value(std::move(object));
1298-
}
1299-
13001260
llvm::json::Value CreateCompileUnit(lldb::SBCompileUnit unit) {
13011261
llvm::json::Object object;
13021262
char unit_path_arr[PATH_MAX];

lldb/tools/lldb-dap/JSONUtils.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -467,9 +467,6 @@ llvm::json::Value CreateVariable(lldb::SBValue v, int64_t variablesReference,
467467
bool is_name_duplicated = false,
468468
std::optional<std::string> custom_name = {});
469469

470-
/// Create a "ExceptionDetail" object for a LLDB
471-
llvm::json::Value CreateExceptionDetails();
472-
473470
llvm::json::Value CreateCompileUnit(lldb::SBCompileUnit unit);
474471

475472
/// Create a runInTerminal reverse request object

lldb/tools/lldb-dap/lldb-dap.cpp

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1080,6 +1080,41 @@ void request_disconnect(const llvm::json::Object &request) {
10801080
// "required": [ "body" ]
10811081
// }]
10821082
// }
1083+
// "ExceptionDetails": {
1084+
// "type": "object",
1085+
// "description": "Detailed information about an exception that has
1086+
// occurred.", "properties": {
1087+
// "message": {
1088+
// "type": "string",
1089+
// "description": "Message contained in the exception."
1090+
// },
1091+
// "typeName": {
1092+
// "type": "string",
1093+
// "description": "Short type name of the exception object."
1094+
// },
1095+
// "fullTypeName": {
1096+
// "type": "string",
1097+
// "description": "Fully-qualified type name of the exception object."
1098+
// },
1099+
// "evaluateName": {
1100+
// "type": "string",
1101+
// "description": "An expression that can be evaluated in the current
1102+
// scope to obtain the exception object."
1103+
// },
1104+
// "stackTrace": {
1105+
// "type": "string",
1106+
// "description": "Stack trace at the time the exception was thrown."
1107+
// },
1108+
// "innerException": {
1109+
// "type": "array",
1110+
// "items": {
1111+
// "$ref": "#/definitions/ExceptionDetails"
1112+
// },
1113+
// "description": "Details of the exception contained by this exception,
1114+
// if any."
1115+
// }
1116+
// }
1117+
// },
10831118
void request_exceptionInfo(const llvm::json::Object &request) {
10841119
llvm::json::Object response;
10851120
FillResponse(request, response);

0 commit comments

Comments
 (0)