Fixed bug 1560944 - CPD now skips symlinks.

git-svn-id: https://pmd.svn.sourceforge.net/svnroot/pmd/trunk@4836 51baf565-9d33-0410-a72c-fc3788e3496d

pmd/etc/changelog.txt

@@ -17,6 +17,7 @@ Fixed bug 1580859 - AvoidDecimalLiteralsInBigDecimalConstructor now catches more
 Fixed bug 1581123 - False +: UnnecessaryWrapperObjectCreation.
 Fixed bug 1592710 - VariableNamingConventions no longer reports false positives on certain enum declarations.
 Fixed bug 1593292 - The CPD GUI now works with the 'by extension' option selected.
+Fixed bug 1560944 - CPD now skips symlinks.
 Applied patch 1551189 - SingularField false + for initialization blocks
 Applied patch 1573981 - false + in CloneMethodMustImplementCloneable
 Applied patch 1574988 - false + in OverrideBothEqualsAndHashcode

pmd/src/net/sourceforge/pmd/cpd/CPD.java

@@ -87,6 +87,11 @@ public class CPD {
             current.add(signature);
         }
 
+        if (!file.getCanonicalPath().equals(file.getAbsolutePath())) {
+            System.out.println("Skipping " + file + " since it appears to be a symlink");
+            return;
+        }
+
         listener.addedFile(fileCount, file);
         SourceCode sourceCode = new SourceCode(new SourceCode.FileCodeLoader(file));
         language.getTokenizer().tokenize(sourceCode, tokens);

pmd/xdocs/credits.xml

@@ -54,6 +54,7 @@
         </subsection>
         <subsection name="Contributors">
         <ul>
+            <li>Larry Brigman - Reported symlink bug in CPD</li>
             <li>Harald Rohan - Reported bug in CPD GUI</li>
             <li>classens - Noted missing varargs setting in ASTFormalParameter</li>
             <li>piair - Implemented StringBufferInstantiationWithChar, AvoidUsingOctalValues</li>