The following commit has been merged in the master branch:
commit 0d42259ce3a416d6f04efc63418ea02afbc74903
Author: Niels Thykier <[email protected]>
Date:   Sun Jan 27 16:47:33 2013 +0100

    lintian: Avoid name-shadowing
    
    Rename local scope %overrides (table of overrides from a package's
    override file) to avoid name-shadowing/confusing it with the
    global scope %overrides (summary of number of tags overriden by
    "code").
    
    Signed-off-by: Niels Thykier <[email protected]>

diff --git a/frontend/lintian b/frontend/lintian
index 899e2c8..f57884c 100755
--- a/frontend/lintian
+++ b/frontend/lintian
@@ -1206,16 +1206,14 @@ sub post_pkg_process_overrides{
     my ($lpkg) = @_;
     # report unused overrides
     if (not $opt{'no-override'}) {
-        my $overrides = $TAGS->overrides ($lpkg);
+        my $pkg_overrides = $TAGS->overrides ($lpkg);
 
-        for my $tag (sort keys %$overrides) {
+        for my $tag (sort keys %$pkg_overrides) {
             next if $TAGS->suppressed($tag);
 
-            my $taginfo = $PROFILE->get_tag ($tag);
-
-            for my $extra (sort keys %{$overrides->{$tag}}) {
-                next if $overrides->{$tag}{$extra};
-                tag( 'unused-override', $tag, $extra );
+            for my $extra (sort keys %{$pkg_overrides->{$tag}}) {
+                next if $pkg_overrides->{$tag}{$extra};
+                tag ('unused-override', $tag, $extra);
             }
         }
     }

-- 
Debian package checker


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: http://lists.debian.org/[email protected]

Reply via email to