godotengine/godot
1
0
Fork 0
mirror of https://github.com/godotengine/godot.git synced 2025-12-06 17:25:19 +00:00
Code Releases Wiki Activity

Fix DAP path mismatch on Windows

Browse Source
This commit is contained in:
Ricardo Subtil
2023-04-08 19:59:01 +01:00
parent cfab3d2f57
commit b8ae1c218c
2 changed files with 12 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
editor/debugger/debug_adapter/debug_adapter_parser.cpp
View File

@@ -341,6 +341,12 @@ Dictionary DebugAdapterParser::req_setBreakpoints(const Dictionary &p_params) co
return prepare_error_response(p_params, DAP::ErrorType::WRONG_PATH, variables); return prepare_error_response(p_params, DAP::ErrorType::WRONG_PATH, variables);
} }
// If path contains \, it's a Windows path, so we need to convert it to /, and make the drive letter uppercase
if (source.path.find("\\") != -1) {
source.path = source.path.replace("\\", "/");
source.path = source.path.substr(0, 1).to_upper() + source.path.substr(1);
}
Array breakpoints = args["breakpoints"], lines; Array breakpoints = args["breakpoints"], lines;
for (int i = 0; i < breakpoints.size(); i++) { for (int i = 0; i < breakpoints.size(); i++) {
DAP::SourceBreakpoint breakpoint; DAP::SourceBreakpoint breakpoint;

6
editor/debugger/debug_adapter/debug_adapter_parser.h
View File

@@ -45,6 +45,12 @@ private:
friend DebugAdapterProtocol; friend DebugAdapterProtocol;
_FORCE_INLINE_ bool is_valid_path(const String &p_path) const { _FORCE_INLINE_ bool is_valid_path(const String &p_path) const {
// If path contains \, it's a Windows path, so we need to convert it to /, and check as case-insensitive.
if (p_path.contains("\\")) {
String project_path = ProjectSettings::get_singleton()->get_resource_path();
String path = p_path.replace("\\", "/");
return path.findn(project_path) != -1;
}
return p_path.begins_with(ProjectSettings::get_singleton()->get_resource_path()); return p_path.begins_with(ProjectSettings::get_singleton()->get_resource_path());
} }