Replace log_oom_extra with log_oom
authorAndre Heinecke <aheinecke@intevation.de>
Mon, 24 Sep 2018 08:57:19 +0000 (10:57 +0200)
committerAndre Heinecke <aheinecke@intevation.de>
Mon, 24 Sep 2018 08:57:19 +0000 (10:57 +0200)
--

src/application-events.cpp
src/debug.h
src/explorer-events.cpp
src/folder-events.cpp
src/gpgoladdin.cpp
src/mail.cpp
src/mailitem-events.cpp
src/memdbg.h
src/oomhelp.cpp
src/windowmessages.cpp

index 1100934..d882e50 100644 (file)
@@ -101,7 +101,7 @@ EVENT_SINK_INVOKE(ApplicationEvents)
           break;
         }
       default:
-        log_oom_extra ("%s:%s: Unhandled Event: %lx \n",
+        log_oom ("%s:%s: Unhandled Event: %lx \n",
                        SRCNAME, __func__, dispid);
     }
   /* We always return S_OK even on error so that outlook
index 523b540..b5b5e49 100644 (file)
@@ -72,7 +72,7 @@ void log_hexdump (const void *buf, size_t buflen, const char *fmt,
                   ...)  __attribute__ ((format (printf,3,4)));
 
 #define log_oom if (opt.enable_debug & DBG_OOM) log_debug
-#define log_oom_extra if (opt.enable_debug & DBG_OOM) log_debug
+#define log_oom if (opt.enable_debug & DBG_OOM) log_debug
 #define log_mime_parser if (opt.enable_debug & DBG_DATA) log_debug
 #define log_data if (opt.enable_debug & DBG_DATA) log_debug
 #define log_mime_data if (opt.enable_debug & DBG_DATA) log_debug
index e9dd776..7ccedfd 100644 (file)
@@ -139,7 +139,7 @@ start_watchdog (LPVOID arg)
 
   if ((it->second & SelectSeen))
     {
-      log_oom_extra ("%s:%s: Cancel watchdog as we have seen a select %p",
+      log_oom ("%s:%s: Cancel watchdog as we have seen a select %p",
                      SRCNAME, __func__, explorer);
       it->second = SelectSeen;
     }
@@ -175,14 +175,14 @@ changeSeen (LPDISPATCH explorer)
   if (has_selection)
     {
       it->second = (state & WatchDogActive) + SelectSeen;
-      log_oom_extra ("%s:%s: Seen select for %p",
+      log_oom ("%s:%s: Seen select for %p",
                      SRCNAME, __func__, explorer);
     }
   else
     {
       if ((it->second & WatchDogActive))
         {
-          log_oom_extra ("%s:%s: Seen unselect for %p but watchdog exists.",
+          log_oom ("%s:%s: Seen unselect for %p but watchdog exists.",
                          SRCNAME, __func__, explorer);
         }
       else
@@ -202,14 +202,14 @@ EVENT_SINK_INVOKE(ExplorerEvents)
     {
       case SelectionChange:
         {
-          log_oom_extra ("%s:%s: Selection change in explorer: %p",
+          log_oom ("%s:%s: Selection change in explorer: %p",
                          SRCNAME, __func__, this);
           changeSeen (m_object);
           break;
         }
       case Close:
         {
-          log_oom_extra ("%s:%s: Deleting event handler: %p",
+          log_oom ("%s:%s: Deleting event handler: %p",
                          SRCNAME, __func__, this);
 
           GpgolAddin::get_instance ()->unregisterExplorerSink (this);
@@ -222,7 +222,7 @@ EVENT_SINK_INVOKE(ExplorerEvents)
       default:
         break;
 #if 0
-        log_oom_extra ("%s:%s: Unhandled Event: %lx \n",
+        log_oom ("%s:%s: Unhandled Event: %lx \n",
                        SRCNAME, __func__, dispid);
 #endif
     }
index 8c765c1..5becba2 100644 (file)
@@ -49,7 +49,7 @@ EVENT_SINK_INVOKE(FolderEvents)
     {
       case BeforeItemMove:
         {
-          log_oom_extra ("%s:%s: Item Move in folder: %p",
+          log_oom ("%s:%s: Item Move in folder: %p",
                          SRCNAME, __func__, this);
 
           /* Parameters should be
@@ -70,7 +70,7 @@ EVENT_SINK_INVOKE(FolderEvents)
 
           if (!parms->rgvarg[1].pdispVal)
             {
-              log_oom_extra ("%s:%s: Passing delete",
+              log_oom ("%s:%s: Passing delete",
                              SRCNAME, __func__);
               break;
             }
@@ -126,7 +126,7 @@ EVENT_SINK_INVOKE(FolderEvents)
       default:
         break;
 #if 0
-        log_oom_extra ("%s:%s: Unhandled Event: %lx \n",
+        log_oom ("%s:%s: Unhandled Event: %lx \n",
                        SRCNAME, __func__, dispid);
 #endif
     }
index 75e9cc8..5f12ab8 100644 (file)
@@ -458,7 +458,7 @@ install_explorer_sinks (LPDISPATCH application)
         }
       else
         {
-          log_oom_extra ("%s:%s: created sink %p for explorer %i",
+          log_oom ("%s:%s: created sink %p for explorer %i",
                          SRCNAME, __func__, sink, i);
           GpgolAddin::get_instance ()->registerExplorerSink (sink);
         }
index 1f48fd2..f9dfe0c 100644 (file)
@@ -155,16 +155,16 @@ Mail::~Mail()
      while parsing. */
   gpgrt_lock_lock (&dtor_lock);
   memdbg_dtor ("Mail");
-  log_oom_extra ("%s:%s: dtor: Mail: %p item: %p",
+  log_oom ("%s:%s: dtor: Mail: %p item: %p",
                  SRCNAME, __func__, this, m_mailitem);
   std::map<LPDISPATCH, Mail *>::iterator it;
 
-  log_oom_extra ("%s:%s: Detaching event sink",
+  log_oom ("%s:%s: Detaching event sink",
                  SRCNAME, __func__);
   detach_MailItemEvents_sink (m_event_sink);
   gpgol_release(m_event_sink);
 
-  log_oom_extra ("%s:%s: Erasing mail",
+  log_oom ("%s:%s: Erasing mail",
                  SRCNAME, __func__);
   gpgrt_lock_lock (&mail_map_lock);
   it = s_mail_map.find(m_mailitem);
@@ -185,29 +185,29 @@ Mail::~Mail()
       gpgrt_lock_unlock (&uid_map_lock);
     }
 
-  log_oom_extra ("%s:%s: releasing mailitem",
+  log_oom ("%s:%s: releasing mailitem",
                  SRCNAME, __func__);
   gpgol_release(m_mailitem);
   xfree (m_cached_html_body);
   xfree (m_cached_plain_body);
   if (!m_uuid.empty())
     {
-      log_oom_extra ("%s:%s: destroyed: %p uuid: %s",
+      log_oom ("%s:%s: destroyed: %p uuid: %s",
                      SRCNAME, __func__, this, m_uuid.c_str());
     }
   else
     {
-      log_oom_extra ("%s:%s: non crypto (or sent) mail: %p destroyed",
+      log_oom ("%s:%s: non crypto (or sent) mail: %p destroyed",
                      SRCNAME, __func__, this);
     }
-  log_oom_extra ("%s:%s: nulling shared pointer",
+  log_oom ("%s:%s: nulling shared pointer",
                  SRCNAME, __func__);
   m_parser = nullptr;
   m_crypter = nullptr;
 
   releaseCurrentItem();
   gpgrt_lock_unlock (&dtor_lock);
-  log_oom_extra ("%s:%s: returning",
+  log_oom ("%s:%s: returning",
                  SRCNAME, __func__);
 }
 
@@ -277,7 +277,7 @@ Mail::preProcessMessage_m ()
                  SRCNAME, __func__);
       return 0;
     }
-  log_oom_extra ("%s:%s: GetBaseMessage OK for %p.",
+  log_oom ("%s:%s: GetBaseMessage OK for %p.",
                  SRCNAME, __func__, m_mailitem);
   /* Change the message class here. It is important that
      we change the message class in the before read event
@@ -1333,7 +1333,7 @@ void
 Mail::parsing_done()
 {
   TRACEPOINT;
-  log_oom_extra ("Mail %p Parsing done for parser num %i: %p",
+  log_oom ("Mail %p Parsing done for parser num %i: %p",
                  this, parsed_count++, m_parser.get());
   if (!m_parser)
     {
@@ -1919,13 +1919,13 @@ Mail::close (Mail *mail)
   dispparams.cArgs = 1;
   dispparams.cNamedArgs = 0;
 
-  log_oom_extra ("%s:%s: Invoking close for: %p",
+  log_oom ("%s:%s: Invoking close for: %p",
                  SRCNAME, __func__, mail->item());
   mail->setCloseTriggered (true);
   int rc = invoke_oom_method_with_parms (mail->item(), "Close",
                                        NULL, &dispparams);
 
-  log_oom_extra ("%s:%s: Returned from close",
+  log_oom ("%s:%s: Returned from close",
                  SRCNAME, __func__);
   return rc;
 }
@@ -3555,7 +3555,7 @@ Mail::releaseCurrentItem()
     {
       return;
     }
-  log_oom_extra ("%s:%s: releasing CurrentItem ref %p",
+  log_oom ("%s:%s: releasing CurrentItem ref %p",
                  SRCNAME, __func__, m_currentItemRef);
   LPDISPATCH tmp = m_currentItemRef;
   m_currentItemRef = nullptr;
index 03296f0..86af6a1 100644 (file)
@@ -142,7 +142,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
     {
       case Open:
         {
-          log_oom_extra ("%s:%s: Open : %p",
+          log_oom ("%s:%s: Open : %p",
                          SRCNAME, __func__, m_mail);
           int draft_flags = 0;
           if (!opt.encrypt_default && !opt.sign_default)
@@ -170,7 +170,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
         }
       case BeforeRead:
         {
-          log_oom_extra ("%s:%s: BeforeRead : %p",
+          log_oom ("%s:%s: BeforeRead : %p",
                          SRCNAME, __func__, m_mail);
           if (m_mail->preProcessMessage_m ())
             {
@@ -181,7 +181,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
         }
       case Read:
         {
-          log_oom_extra ("%s:%s: Read : %p",
+          log_oom ("%s:%s: Read : %p",
                          SRCNAME, __func__, m_mail);
           if (!m_mail->isCryptoMail ())
             {
@@ -323,7 +323,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
         }
       case CustomPropertyChange:
         {
-          log_oom_extra ("%s:%s: CustomPropertyChange : %p",
+          log_oom ("%s:%s: CustomPropertyChange : %p",
                          SRCNAME, __func__, m_mail);
           /* TODO */
           break;
@@ -345,7 +345,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
              If this encryption is successful and we pass the send
              as then the encrypted data is sent.
            */
-          log_oom_extra ("%s:%s: Send : %p",
+          log_oom ("%s:%s: Send : %p",
                          SRCNAME, __func__, m_mail);
           if (!m_mail->needs_crypto_m () && m_mail->cryptState () == Mail::NoCryptMail)
             {
@@ -553,7 +553,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
         }
       case Write:
         {
-          log_oom_extra ("%s:%s: Write : %p",
+          log_oom ("%s:%s: Write : %p",
                          SRCNAME, __func__, m_mail);
           /* This is a bit strange. We sometimes get multiple write events
              without a read in between. When we access the message in
@@ -671,7 +671,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
         }
       case AfterWrite:
         {
-          log_oom_extra ("%s:%s: AfterWrite : %p",
+          log_oom ("%s:%s: AfterWrite : %p",
                          SRCNAME, __func__, m_mail);
           if (m_mail->cryptState () == Mail::NeedsFirstAfterWrite)
             {
@@ -695,7 +695,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
         }
       case Close:
         {
-          log_oom_extra ("%s:%s: Close : %p",
+          log_oom ("%s:%s: Close : %p",
                          SRCNAME, __func__, m_mail);
           if (m_mail->isCryptoMail ())
             {
@@ -721,7 +721,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
                   return S_OK;
                 }
               *(parms->rgvarg[0].pboolVal) = VARIANT_TRUE;
-              log_oom_extra ("%s:%s: Canceling close event.",
+              log_oom ("%s:%s: Canceling close event.",
                              SRCNAME, __func__);
               if (Mail::close(m_mail))
                 {
@@ -733,12 +733,12 @@ EVENT_SINK_INVOKE(MailItemEvents)
         }
       case Unload:
         {
-          log_oom_extra ("%s:%s: Unload : %p",
+          log_oom ("%s:%s: Unload : %p",
                          SRCNAME, __func__, m_mail);
           log_debug ("%s:%s: Removing Mail for message: %p.",
                      SRCNAME, __func__, m_object);
           delete m_mail;
-          log_oom_extra ("%s:%s: deletion done",
+          log_oom ("%s:%s: deletion done",
                          SRCNAME, __func__);
           memdbg_dump ();
           return S_OK;
@@ -751,7 +751,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
         }
       case Forward:
         {
-          log_oom_extra ("%s:%s: %s : %p",
+          log_oom ("%s:%s: %s : %p",
                          SRCNAME, __func__, is_reply ? "reply" : "forward", m_mail);
           int draft_flags = 0;
           if (opt.encrypt_default)
@@ -867,7 +867,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
               Mail::clearLastMail ();
             }
 
-          log_oom_extra ("%s:%s: Reply Forward ReplyAll: %p",
+          log_oom ("%s:%s: Reply Forward ReplyAll: %p",
                          SRCNAME, __func__, m_mail);
           if (!opt.reply_crypt)
             {
@@ -900,7 +900,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
         }
       case AttachmentRemove:
         {
-          log_oom_extra ("%s:%s: AttachmentRemove: %p",
+          log_oom ("%s:%s: AttachmentRemove: %p",
                          SRCNAME, __func__, m_mail);
           if (!m_mail->isCryptoMail () || attachRemoveWarnShown ||
               m_mail->attachmentRemoveWarningDisabled ())
@@ -917,7 +917,7 @@ EVENT_SINK_INVOKE(MailItemEvents)
         }
 
       default:
-        log_oom_extra ("%s:%s: Message:%p Unhandled Event: %lx \n",
+        log_oom ("%s:%s: Message:%p Unhandled Event: %lx \n",
                        SRCNAME, __func__, m_object, dispid);
     }
   return S_OK;
index aa4939d..ffbdae9 100644 (file)
@@ -33,7 +33,7 @@ extern "C" {
 { \
   if (X) \
     { \
-      log_oom_extra ("%s:%s:%i AddRef on %p", \
+      log_oom ("%s:%s:%i AddRef on %p", \
                      SRCNAME, __func__, __LINE__, X); \
       _memdbg_addRef (X, __func__); \
     } \
index 2c162ab..143f13f 100644 (file)
@@ -1595,7 +1595,7 @@ get_oom_base_message_from_mapi (LPDISPATCH mapi_message)
   /* The call to GetBaseMessage is pretty much a jump
      in the dark. So it would not be surprising to get
      crashes here in the future. */
-  log_oom_extra("%s:%s: About to call GetBaseMessage.",
+  log_oom("%s:%s: About to call GetBaseMessage.",
                 SRCNAME, __func__);
   hr = secureMessage->GetBaseMessage (&message);
   memdbg_addRef (message);
index d239377..c6e585c 100644 (file)
@@ -536,7 +536,7 @@ void
 blockInv()
 {
   invalidation_blocked++;
-  log_oom_extra ("%s:%s: Invalidation block count %i",
+  log_oom ("%s:%s: Invalidation block count %i",
                  SRCNAME, __func__, invalidation_blocked);
 }
 
@@ -544,7 +544,7 @@ void
 unblockInv()
 {
   invalidation_blocked--;
-  log_oom_extra ("%s:%s: Invalidation block count %i",
+  log_oom ("%s:%s: Invalidation block count %i",
                  SRCNAME, __func__, invalidation_blocked);
 
   if (invalidation_blocked < 0)