[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:18 PST 2026
https://github.com/rchamala created https://github.com/llvm/llvm-project/pull/181498
## 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.
>From e6ce1d60ce9213aa844f31d13bd2464a252c5dab Mon Sep 17 00:00:00 2001
From: Rahul Reddy Chamala <rachamal at meta.com>
Date: Sat, 14 Feb 2026 10:54:57 -0800
Subject: [PATCH] [lldb] NFC: Use standard comment for lldb-python.h include
---
.../Python/Interfaces/ScriptInterpreterPythonInterfaces.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
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"
More information about the lldb-commits
mailing list