[Lldb-commits] [lldb] [lldb] update the listener logs (PR #195850)

[Jonas Devlieghere via lldb-commits]

================
@@ -258,7 +258,10 @@ bool Listener::GetEventInternal(
     Broadcaster *broadcaster, // nullptr for any broadcaster
     uint32_t event_type_mask, EventSP &event_sp) {
   Log *log = GetLog(LLDBLog::Events);
-  LLDB_LOG(log, "this = {0}, timeout = {1} for {2}", this, timeout, m_name);
+
+  LLDB_LOG(log,
+           "{0} Listener('{1}')::{2}, timeout = {3}, event_type_mask = {4:x}",
+           this, m_name, __FUNCTION__, timeout, event_type_mask);
----------------
JDevlieghere wrote:

`log enable` has a `--file-function/-F` option which prepends the name of the file and function to a log statement. There's (almost) never a reason to include `__FUNCTION__` in a log message. Furthermore, if you do need the function macro, you should use `LLVM_PRETTY_FUNCTION` which is a wrapper over the different macros.

https://github.com/llvm/llvm-project/pull/195850