[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[SCM] Debian package checker branch, master, updated. 2.5.11-115-g0d42259



The following commit has been merged in the master branch:
commit 0d42259ce3a416d6f04efc63418ea02afbc74903
Author: Niels Thykier <niels@thykier.net>
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 <niels@thykier.net>

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


Reply to: