[Libreoffice-commits] core.git: include/comphelper

[Tor Lillqvist (via logerrit)]

 include/comphelper/profilezone.hxx |   39 ++++++++++++++++++++++---------------
 include/comphelper/traceevent.hxx  |   20 ++++++++++++++----
 2 files changed, 39 insertions(+), 20 deletions(-)

New commits:
commit 0fa2bb31b53c2b9114480ab77e0db4e011691440
Author:     Tor Lillqvist <tml at collabora.com>
AuthorDate: Mon May 31 15:55:20 2021 +0300
Commit:     Tor Lillqvist <tml at collabora.com>
CommitDate: Wed Jun 9 16:52:15 2021 +0200

    Avoid empty std::map constructor
    
    Change-Id: Ie1bc333409fb201d82dd2cff7597e281600f01db
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116449
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
    Reviewed-by: Tor Lillqvist <tml at collabora.com>
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116849
    Tested-by: Jenkins

diff --git a/include/comphelper/profilezone.hxx b/include/comphelper/profilezone.hxx
index babd11f5b93b..08e6a8b031e8 100644
--- a/include/comphelper/profilezone.hxx
+++ b/include/comphelper/profilezone.hxx
@@ -31,6 +31,24 @@ class COMPHELPER_DLLPUBLIC ProfileZone : public NamedEvent
 
     void addRecording();
 
+    ProfileZone(const char* sName, const OUString& sArgs, bool bConsole)
+        : NamedEvent(sName, sArgs)
+        , m_bConsole(bConsole)
+        , m_nNesting(-1)
+    {
+        if (s_bRecording || m_bConsole)
+        {
+            TimeValue systemTime;
+            osl_getSystemTime(&systemTime);
+            m_nCreateTime
+                = static_cast<long long>(systemTime.Seconds) * 1000000 + systemTime.Nanosec / 1000;
+
+            m_nNesting = s_nNesting++;
+        }
+        else
+            m_nCreateTime = 0;
+    }
+
 public:
     /**
      * Starts measuring the cost of a C++ scope.
@@ -47,23 +65,14 @@ public:
      * Similar to the DEBUG macro in sal/log.hxx, don't forget to remove these lines before
      * committing.
      */
-    ProfileZone(const char* sName, bool bConsole = false,
-                const std::map<OUString, OUString>& args = std::map<OUString, OUString>())
-        : NamedEvent(sName, args)
-        , m_bConsole(bConsole)
-        , m_nNesting(-1)
+    ProfileZone(const char* sName, const std::map<OUString, OUString>& aArgs, bool bConsole = false)
+        : ProfileZone(sName, createArgsString(aArgs), bConsole)
     {
-        if (s_bRecording || m_bConsole)
-        {
-            TimeValue systemTime;
-            osl_getSystemTime(&systemTime);
-            m_nCreateTime
-                = static_cast<long long>(systemTime.Seconds) * 1000000 + systemTime.Nanosec / 1000;
+    }
 
-            m_nNesting = s_nNesting++;
-        }
-        else
-            m_nCreateTime = 0;
+    ProfileZone(const char* sName, bool bConsole = false)
+        : ProfileZone(sName, OUString(), bConsole)
+    {
     }
 
     ~ProfileZone()
diff --git a/include/comphelper/traceevent.hxx b/include/comphelper/traceevent.hxx
index 6e614ec0552e..5e2502de72a1 100644
--- a/include/comphelper/traceevent.hxx
+++ b/include/comphelper/traceevent.hxx
@@ -82,9 +82,14 @@ protected:
     const int m_nPid;
     const OUString m_sArgs;
 
-    TraceEvent(std::map<OUString, OUString> args)
+    TraceEvent(const OUString& sArgs)
         : m_nPid(getPid())
-        , m_sArgs(createArgsString(args))
+        , m_sArgs(sArgs)
+    {
+    }
+
+    TraceEvent(std::map<OUString, OUString> aArgs)
+        : TraceEvent(createArgsString(aArgs))
     {
     }
 
@@ -105,9 +110,14 @@ class COMPHELPER_DLLPUBLIC NamedEvent : public TraceEvent
 protected:
     const char* m_sName;
 
-    NamedEvent(const char* sName,
-               const std::map<OUString, OUString>& args = std::map<OUString, OUString>())
-        : TraceEvent(args)
+    NamedEvent(const char* sName, const OUString& sArgs)
+        : TraceEvent(sArgs)
+        , m_sName(sName ? sName : "(null)")
+    {
+    }
+
+    NamedEvent(const char* sName, const std::map<OUString, OUString>& aArgs)
+        : TraceEvent(aArgs)
         , m_sName(sName ? sName : "(null)")
     {
     }