When printing warnings, the repeat_key should be

const void * because the data is never accessed,
the pointer is the only useful piece of data.

llvm-svn: 255090

diff --git a/lldb/include/lldb/Target/Process.h b/lldb/include/lldb/Target/Process.h
index ae33d30..2e063c5 100644 (file)
--- a/lldb/include/lldb/Target/Process.h
+++ b/lldb/include/lldb/Target/Process.h
@@ -3201,7 +3201,7 @@ protected:
     ///     printf style format string
     //------------------------------------------------------------------
     void
-    PrintWarning (uint64_t warning_type, void *repeat_key, const char *fmt, ...) __attribute__((format(printf, 4, 5)));
+    PrintWarning (uint64_t warning_type, const void *repeat_key, const char *fmt, ...) __attribute__((format(printf, 4, 5)));
     
     //------------------------------------------------------------------
     // NextEventAction provides a way to register an action on the next
@@ -3286,7 +3286,7 @@ protected:
     // Type definitions
     //------------------------------------------------------------------
     typedef std::map<lldb::LanguageType, lldb::LanguageRuntimeSP> LanguageRuntimeCollection;
-    typedef std::unordered_set<void *> WarningsPointerSet;
+    typedef std::unordered_set<const void *> WarningsPointerSet;
     typedef std::map<uint64_t, WarningsPointerSet> WarningsCollection;
 
     struct PreResumeCallbackAndBaton

diff --git a/lldb/source/Target/Process.cpp b/lldb/source/Target/Process.cpp
index cbe2d58..3297add 100644 (file)
--- a/lldb/source/Target/Process.cpp
+++ b/lldb/source/Target/Process.cpp
@@ -6400,7 +6400,7 @@ Process::ModulesDidLoad (ModuleList &module_list)
 }
 
 void
-Process::PrintWarning (uint64_t warning_type, void *repeat_key, const char *fmt, ...)
+Process::PrintWarning (uint64_t warning_type, const void *repeat_key, const char *fmt, ...)
 {
     bool print_warning = true;