[Lldb-commits] [lldb] [lldb] NFC: Use standard comment for lldb-python.h include (PR #181498)
via lldb-commits
lldb-commits at lists.llvm.org
Sat Feb 14 10:55:49 PST 2026
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-lldb
Author: None (rchamala)
<details>
<summary>Changes</summary>
## Summary
Use the standard `// LLDB Python header must be included first.` comment to match every other Python interface `.cpp` file in this directory, as suggested by @<!-- -->JDevlieghere.
## Test plan
NFC - comment only change.
---
Full diff: https://github.com/llvm/llvm-project/pull/181498.diff
1 Files Affected:
- (modified) lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptInterpreterPythonInterfaces.cpp (+1-2)
``````````diff
diff --git a/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptInterpreterPythonInterfaces.cpp b/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptInterpreterPythonInterfaces.cpp
index 7cb5c90f8e14b..faa293c23c83a 100644
--- a/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptInterpreterPythonInterfaces.cpp
+++ b/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptInterpreterPythonInterfaces.cpp
@@ -11,8 +11,7 @@
#if LLDB_ENABLE_PYTHON
-// Include lldb-python.h first to define NO_PID_T on Windows before any
-// LLDB header transitively pulls in PosixApi.h.
+// LLDB Python header must be included first.
#include "../lldb-python.h"
#include "ScriptInterpreterPythonInterfaces.h"
``````````
</details>
https://github.com/llvm/llvm-project/pull/181498
More information about the lldb-commits
mailing list