[Lldb-commits] [lldb] [lldb] [Windows] Silence format string warnings (PR #150886)

[via lldb-commits]

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