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

[lintian] 01/01: c/testsuite.pm: Call "is_open_ok" and remove obsolete comment



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

nthykier pushed a commit to branch master
in repository lintian.

commit b75b986e305e4aaf2752fa83eac983a61427008a
Author: Niels Thykier <niels@thykier.net>
Date:   Mon Sep 29 23:09:12 2014 +0200

    c/testsuite.pm: Call "is_open_ok" and remove obsolete comment
    
    The "is_open_ok" call is redundant, but we call it for the sake of
    making an example.
    
    Signed-off-by: Niels Thykier <niels@thykier.net>
---
 checks/testsuite.pm | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/checks/testsuite.pm b/checks/testsuite.pm
index 008e764..64ea156 100644
--- a/checks/testsuite.pm
+++ b/checks/testsuite.pm
@@ -69,13 +69,7 @@ sub run {
     if (defined($control)) {
         if (not $control->is_regular_file) {
             tag 'debian-tests-control-is-not-a-regular-file';
-        } else {
-            # Since $control is defined, we know that none of the
-            # parent segments (i.e. debian and debian/tests) were
-            # symlinks.  With the "is_regular_file" test above we know
-            # that the file itself is not a symlink.  These two facts
-            # _combined_ means we can skip the is_ancestor_of check
-            # here.
+        } elsif ($control->is_open_ok) {
             my $path = $control->fs_path;
             my $not_utf8_line = file_is_encoded_in_non_utf8($path);
 

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


Reply to: