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

lamby pushed a commit to branch master
in repository lintian.

commit 39adcde0f416bec92f99800f2c623b9e62886ecb
Author: Chris Lamb <la...@debian.org>
Date:   Tue Feb 13 14:35:33 2018 +0000

    checks/files.pm: Tidy logic for detecting allowed rel="" values in <link/> 
HTML tags.
---
 checks/files.pm  | 18 +++++++-----------
 debian/changelog |  3 +++
 2 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/checks/files.pm b/checks/files.pm
index a6e8edd..e4480c1 100644
--- a/checks/files.pm
+++ b/checks/files.pm
@@ -2051,17 +2051,13 @@ sub _check_tag_url_privacy_breach {
                       >,xismog;
         my $relcontent = $1;
         if (defined($relcontent)) {
-            if ($relcontent eq 'schema.dct') {
-                # see #736992
-                return;
-            } elsif  ($relcontent eq 'bookmark') {
-                # see #746656
-                return;
-            } elsif ($relcontent eq 'generator-home') {
-                # generator-home is used by texinfo
-                return;
-                # reparse for alternate (css alternate is loaded)
-            } elsif ($relcontent eq 'alternate') {
+            my %allowed = (
+                'bookmark'          => 1, # #746656
+                'generator-home'    => 1, # texinfo
+                'schema.dct'        => 1, # #736992
+            );
+            return if ($allowed{$relcontent});
+            if ($relcontent eq 'alternate') {
                 my $type = $fulltag;
                 $type =~ m,<link
                       (?:\s[^>]+)? \s+
diff --git a/debian/changelog b/debian/changelog
index f8b47e5..711b25f 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -5,6 +5,9 @@ lintian (2.5.75) UNRELEASED; urgency=medium
   * checks/changelog-file.desc:
     + [CL] When checking latest-debian-changelog-entry-without-new-version
       ignore any change of epoch.  (Closes: #889991)
+  * checks/files.pm:
+    + [CL] Tidy logic for detecting allowed rel="" values in <link/>
+      HTML tags.
   * checks/init.desc:
     + [CL] Improve various parts of the long description for
       init.d-script-should-always-start-service.

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

Reply via email to