llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-lldb
Author: Martin Storsjö (mstorsjo)
<details>
<summary>Changes</summary>
This fixes the following build warnings in a mingw environment:
../../lldb/source/Host/windows/MainLoopWindows.cpp:226:50: warning: format
specifies type 'int' but the argument has type 'IOObject::WaitableHandle' (aka
'void *') [-Wformat]
226 | "File descriptor %d already monitored.", waitable_handle);
| ~~ ^~~~~~~~~~~~~~~
../../lldb/source/Host/windows/MainLoopWindows.cpp:239:49: warning: format
specifies type 'int' but the argument has type 'DWORD' (aka 'unsigned long')
[-Wformat]
238 | error = Status::FromErrorStringWithFormat("Unsupported file
type %d",
|
~~
|
%lu
239 | file_type);
| ^~~~~~~~~
2 warnings generated.
---
Full diff: https://github.com/llvm/llvm-project/pull/150886.diff
1 Files Affected:
- (modified) lldb/source/Host/windows/MainLoopWindows.cpp (+2-2)
``````````diff
diff --git a/lldb/source/Host/windows/MainLoopWindows.cpp
b/lldb/source/Host/windows/MainLoopWindows.cpp
index c1a018238432d..f7f65a82f726f 100644
--- a/lldb/source/Host/windows/MainLoopWindows.cpp
+++ b/lldb/source/Host/windows/MainLoopWindows.cpp
@@ -223,7 +223,7 @@ MainLoopWindows::RegisterReadObject(const IOObjectSP
&object_sp,
if (m_read_fds.find(waitable_handle) != m_read_fds.end()) {
error = Status::FromErrorStringWithFormat(
- "File descriptor %d already monitored.", waitable_handle);
+ "File descriptor %p already monitored.", waitable_handle);
return nullptr;
}
@@ -236,7 +236,7 @@ MainLoopWindows::RegisterReadObject(const IOObjectSP
&object_sp,
DWORD file_type = GetFileType(waitable_handle);
if (file_type != FILE_TYPE_PIPE) {
error = Status::FromErrorStringWithFormat("Unsupported file type %d",
- file_type);
+ static_cast<int>(file_type));
return nullptr;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/150886
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
