[Lldb-commits] [lldb] [lldb-dap] fix config value comparison (PR #142017)
via lldb-commits
lldb-commits at lists.llvm.org
Thu May 29 12:28:24 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-lldb
Author: Ely Ronnen (eronnen)
<details>
<summary>Changes</summary>
Fix TypeSCript comparison (when defining an empty config string like `commandEscapePrefix=""` it would skip it)
---
Full diff: https://github.com/llvm/llvm-project/pull/142017.diff
1 Files Affected:
- (modified) lldb/tools/lldb-dap/src-ts/debug-configuration-provider.ts (+1-1)
``````````diff
diff --git a/lldb/tools/lldb-dap/src-ts/debug-configuration-provider.ts b/lldb/tools/lldb-dap/src-ts/debug-configuration-provider.ts
index 957bc5e1eb956..316ffaf47c3d2 100644
--- a/lldb/tools/lldb-dap/src-ts/debug-configuration-provider.ts
+++ b/lldb/tools/lldb-dap/src-ts/debug-configuration-provider.ts
@@ -84,7 +84,7 @@ export class LLDBDapConfigurationProvider
continue;
}
const value = config.get(key);
- if (!value || value === cfg.default) {
+ if (value === undefined || value === cfg.default) {
continue;
}
switch (cfg.type) {
``````````
</details>
https://github.com/llvm/llvm-project/pull/142017
More information about the lldb-commits
mailing list