Skip to content

Commit f7e658f

Browse files
committed
Move the ExceptionDetails type info closer to request_exceptionInfo
1 parent ebcf484 commit f7e658f

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
@@ -1260,46 +1260,6 @@ llvm::json::Value CreateVariable(lldb::SBValue v, int64_t variablesReference,
12601260
return llvm::json::Value(std::move(object));
12611261
}
12621262

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