[lintian] 01/01: c/r-sync-state: Always check if we should clear error counters

[Niels Thykier <nthykier@moszumanska.debian.org>]

This is an automated email from the git hooks/post-receive script.

nthykier pushed a commit to branch master
in repository lintian.

commit 3f0faecc184c243aa0374c8c7a5eb4bd9400e5dc
Author: Niels Thykier <niels@thykier.net>
Date:   Tue Feb 6 21:21:52 2018 +0000

    c/r-sync-state: Always check if we should clear error counters
    
    Signed-off-by: Niels Thykier <niels@thykier.net>
---
 commands/reporting-sync-state.pm | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/commands/reporting-sync-state.pm b/commands/reporting-sync-state.pm
index 0163233..ed2f1c4 100644
--- a/commands/reporting-sync-state.pm
+++ b/commands/reporting-sync-state.pm
@@ -300,16 +300,17 @@ sub cleanup_group_state {
             log_debug("Marking ${group_id} as out of date: In backlog")
               if $backlog->{$group_id};
         }
-        if (
-            exists($group_data->{'processing-errors'})
+    }
+    # Check for and possible clear the error counters
+    if (
+        exists($group_data->{'processing-errors'})
             and (not exists($group_data->{'last-error-by'})
-                or $group_data->{'last-error-by'} ne $OPT{'desired-version'})
-          ) {
-            log_debug(
-                "Clearing error flag for ${group_id}: New version of lintian");
-            delete($group_data->{'processing-errors'});
-            delete($group_data->{'last-error-by'});
-        }
+            or $group_data->{'last-error-by'} ne $OPT{'desired-version'})
+    ) {
+        log_debug(
+            "Clearing error flag for ${group_id}: New version of lintian");
+        delete($group_data->{'processing-errors'});
+        delete($group_data->{'last-error-by'});
     }
 
     if (not %{$members}) {

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git