log_error() Win32: Only call LogShowActivity() for debug level LOG_LEVEL_REQUEST
authorFabian Keil <fk@fabiankeil.de>
Wed, 31 May 2023 10:29:18 +0000 (12:29 +0200)
committerFabian Keil <fk@fabiankeil.de>
Thu, 15 Jun 2023 10:37:19 +0000 (12:37 +0200)
As of b94bbe62a950, which was part of Privoxy 3.0.29,
LOG_LEVEL_REQUEST is used for all requests including
crunched ones.

Previously LogShowActivity() was called twice for crunched
requests, (presumably) resulting in an aborted animation.

errlog.c

index e1d9925..27bb4e0 100644 (file)
--- a/errlog.c
+++ b/errlog.c
@@ -659,11 +659,11 @@ void log_error(int loglevel, const char *fmt, ...)
 
 #if defined(_WIN32) && !defined(_WIN_CONSOLE)
    /*
-    * Irrespective of debug setting, a GET/POST/CONNECT makes
-    * the taskbar icon animate.  (There is an option to disable
-    * this but checking that is handled inside LogShowActivity()).
+    * Irrespective of debug setting, a request makes the taskbar icon
+    * animate. (There is an option to disable this but checking that is
+    * handled inside LogShowActivity()).
     */
-   if ((loglevel == LOG_LEVEL_REQUEST) || (loglevel == LOG_LEVEL_CRUNCH))
+   if (loglevel == LOG_LEVEL_REQUEST)
    {
       LogShowActivity();
    }