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 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
