The following commit has been merged in the master branch:
commit 32dc071dd56aafdf86d949a67963d04ab8b10d40
Author: Niels Thykier <[email protected]>
Date:   Wed May 22 22:26:45 2013 +0200

    c/cruft: Use "return" instead of "next" to exit sub
    
    Signed-off-by: Niels Thykier <[email protected]>

diff --git a/checks/cruft b/checks/cruft
index b401e35..d67f8c5 100644
--- a/checks/cruft
+++ b/checks/cruft
@@ -444,7 +444,7 @@ sub find_cruft {
         close($fd);
     }
 
-    next if $info->is_non_free; # (license issue does not apply to non-free)
+    return if $info->is_non_free; # (license issue does not apply to non-free)
 
     # test license problem is source file (only text file)
     if (-T $basename) {

-- 
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