[Commits] [SCM] claws branch, gtk2, updated. 3.19.0-62-ge4b3d9ea6

mones at claws-mail.org mones at claws-mail.org
Tue Aug 16 18:34:26 UTC 2022


The branch, gtk2 has been updated
       via  e4b3d9ea699d673338479dabc057293c0c25387f (commit)
      from  c51af48ddcfab6e8754b68397aceb5d8b91dcde7 (commit)

Summary of changes:
 src/filtering.c | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)


- Log -----------------------------------------------------------------
commit e4b3d9ea699d673338479dabc057293c0c25387f
Author: Ricardo Mones <ricardo at mones.org>
Date:   Tue Aug 16 20:33:27 2022 +0200

    Fix CID 1508488: Control flow issues (DEADCODE)
    
    And the logic surrounding the logging of filtering.

diff --git a/src/filtering.c b/src/filtering.c
index cf99d6be4..429861de8 100644
--- a/src/filtering.c
+++ b/src/filtering.c
@@ -600,12 +600,12 @@ static gboolean filtering_match_condition(FilteringProp *filtering, MsgInfo *inf
 
 		/* debug output */
 		if (debug_filtering_session) {
-			if (matches && prefs_common.filtering_debug_level >= FILTERING_DEBUG_LEVEL_HIGH) {
+			if (matches && prefs_common.filtering_debug_level >= FILTERING_DEBUG_LEVEL_MED) {
 				if (filtering->account_id == 0) {
 					log_status_ok(LOG_DEBUG_FILTERING,
 							_("rule is not account-based\n"));
 				} else {
-					if (prefs_common.filtering_debug_level >= FILTERING_DEBUG_LEVEL_MED) {
+					if (prefs_common.filtering_debug_level >= FILTERING_DEBUG_LEVEL_HIGH) {
 						log_status_ok(LOG_DEBUG_FILTERING,
 								_("rule is account-based [id=%d, name='%s'], "
 								"matching the account currently used to retrieve messages\n"),
@@ -615,19 +615,19 @@ static gboolean filtering_match_condition(FilteringProp *filtering, MsgInfo *inf
 			}
 			else
 			if (!matches) {
-				if (prefs_common.filtering_debug_level >= FILTERING_DEBUG_LEVEL_MED) {
+				if (prefs_common.filtering_debug_level >= FILTERING_DEBUG_LEVEL_HIGH) {
+					PrefsAccount *account = account_find_from_id(filtering->account_id);
+
 					log_status_skip(LOG_DEBUG_FILTERING,
-							_("rule is account-based, "
-							"not matching the account currently used to retrieve messages\n"));
+							_("rule is account-based [id=%d, name='%s'], "
+							"not matching the account currently used to retrieve messages [id=%d, name='%s']\n"),
+							filtering->account_id, account?account->account_name:_("NON_EXISTENT"),
+							ac_prefs->account_id, ac_prefs?ac_prefs->account_name:_("NON_EXISTENT"));
 				} else {
-					if (prefs_common.filtering_debug_level >= FILTERING_DEBUG_LEVEL_HIGH) {
-						PrefsAccount *account = account_find_from_id(filtering->account_id);
-
+					if (prefs_common.filtering_debug_level >= FILTERING_DEBUG_LEVEL_MED) {
 						log_status_skip(LOG_DEBUG_FILTERING,
-								_("rule is account-based [id=%d, name='%s'], "
-								"not matching the account currently used to retrieve messages [id=%d, name='%s']\n"),
-								filtering->account_id, account?account->account_name:_("NON_EXISTENT"),
-								ac_prefs->account_id, ac_prefs?ac_prefs->account_name:_("NON_EXISTENT"));
+								_("rule is account-based, "
+								"not matching the account currently used to retrieve messages\n"));
 					}
 				}
 			}

-----------------------------------------------------------------------


hooks/post-receive
-- 
Claws Mail


More information about the Commits mailing list