Reduce tracing verbosity a bit
authorAndre Heinecke <aheinecke@intevation.de>
Tue, 25 Sep 2018 07:31:36 +0000 (09:31 +0200)
committerAndre Heinecke <aheinecke@intevation.de>
Tue, 25 Sep 2018 07:31:36 +0000 (09:31 +0200)
--
This removes tracing from very often called functions
like message_cb in mimedataprovider or lookup_oom_dispid.

src/debug.cpp
src/mimedataprovider.cpp
src/oomhelp.cpp
src/windowmessages.cpp

index 454de19..13a33bb 100644 (file)
@@ -347,6 +347,5 @@ const char *anonstr (const char *data)
 
   gpgrt_lock_unlock (&anon_str_lock);
 
-  TRACEPOINT;
   return it->second.c_str();
 }
index 9f57f17..cf66ddf 100644 (file)
@@ -425,7 +425,6 @@ static int
 message_cb (void *opaque, rfc822parse_event_t event,
             rfc822parse_t msg)
 {
-  TSTART;
   int retval = 0;
 
   MimeDataProvider *provider = static_cast<MimeDataProvider*> (opaque);
@@ -490,7 +489,7 @@ message_cb (void *opaque, rfc822parse_event_t event,
       break;
     }
 
-  TRETURN retval;
+  return retval;
 }
 
 MimeDataProvider::MimeDataProvider(bool no_headers) :
@@ -566,8 +565,7 @@ MimeDataProvider::~MimeDataProvider()
 bool
 MimeDataProvider::isSupported(GpgME::DataProvider::Operation op) const
 {
-  TSTART;
-  TRETURN op == GpgME::DataProvider::Read ||
+  return op == GpgME::DataProvider::Read ||
          op == GpgME::DataProvider::Seek ||
          op == GpgME::DataProvider::Write ||
          op == GpgME::DataProvider::Release;
@@ -1010,8 +1008,7 @@ ssize_t MimeDataProvider::write(const void *buffer, size_t bufSize)
 off_t
 MimeDataProvider::seek(off_t offset, int whence)
 {
-  TSTART;
-  TRETURN m_crypto_data.seek (offset, whence);
+  return m_crypto_data.seek (offset, whence);
 }
 
 GpgME::Data *
index 851b7ba..383960c 100644 (file)
 HRESULT
 gpgol_queryInterface (LPUNKNOWN pObj, REFIID riid, LPVOID FAR *ppvObj)
 {
-  TSTART;
   HRESULT ret = pObj->QueryInterface (riid, ppvObj);
   if ((opt.enable_debug & DBG_OOM_EXTRA) && *ppvObj)
     {
       memdbg_addRef (*ppvObj);
     }
-  TRETURN ret;
+  return ret;
 }
 
 HRESULT
@@ -51,7 +50,6 @@ gpgol_openProperty (LPMAPIPROP obj, ULONG ulPropTag, LPCIID lpiid,
                     ULONG ulInterfaceOptions, ULONG ulFlags,
                     LPUNKNOWN FAR * lppUnk)
 {
-  TSTART;
   HRESULT ret = obj->OpenProperty (ulPropTag, lpiid,
                                    ulInterfaceOptions, ulFlags,
                                    lppUnk);
@@ -61,7 +59,7 @@ gpgol_openProperty (LPMAPIPROP obj, ULONG ulPropTag, LPCIID lpiid,
       log_debug ("%s:%s: OpenProperty on %p prop %lx result %p",
                  SRCNAME, __func__, obj,  ulPropTag, *lppUnk);
     }
-  TRETURN ret;
+  return ret;
 }
 /* Return a malloced string with the utf-8 encoded name of the object
    or NULL if not available.  */
@@ -117,7 +115,6 @@ get_object_name (LPUNKNOWN obj)
 DISPID
 lookup_oom_dispid (LPDISPATCH pDisp, const char *name)
 {
-  TSTART;
   HRESULT hr;
   DISPID dispid;
   wchar_t *wname;
@@ -142,13 +139,12 @@ lookup_oom_dispid (LPDISPATCH pDisp, const char *name)
   if (hr != S_OK)
     dispid = DISPID_UNKNOWN;
 
-  TRETURN dispid;
+  return dispid;
 }
 
 static void
 init_excepinfo (EXCEPINFO *err)
 {
-  TSTART;
   if (!err)
     {
       TRETURN;
index d20655b..0b233e9 100644 (file)
@@ -39,7 +39,6 @@ static int invalidation_blocked = 0;
 LONG_PTR WINAPI
 gpgol_window_proc (HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
 {
-  TSTART;
 //  log_debug ("WMG: %x", (unsigned int) message);
   if (message == WM_USER + 42)
     {