[ checkstyle-Feature Requests-2114350 ] ignore .svn / .cvs dir while scanning files

[SourceForge.net]

Feature Requests item #2114350, was opened at 2008-09-17 00:58
Message generated for change (Comment added) made by oburn
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397081&aid=2114350&group_id=29721

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: ignore .svn / .cvs dir while scanning files

Initial Comment:
The application should not check metadata files hidden in "." sub directories while traversing files to check.

In particular, you should avoid all ".svn" and ".CVS" dirs (where some data files are duplicates from subversion/CVS repositories) ...


corrective patch:
In class Main.java, add a test for "if (! aNode.getName().startsWith(".")) {" in method traverse()


    private static void traverse(File aNode, List<File> aFiles)
    {
        if (aNode.canRead()) {
            if (aNode.isDirectory()) {
                if (! aNode.getName().startsWith(".")) {
                    final File[] nodes = aNode.listFiles();
                    for (File element : nodes) {
                        traverse(element, aFiles);
                    }
                } else {
                    // ignore .cvs, .svn ...
                }
            }
            else if (aNode.isFile()) {
                aFiles.add(aNode);
            }
        }
    }


instead of

    private static void traverse(File aNode, List<File> aFiles)
    {
        if (aNode.canRead()) {
            if (aNode.isDirectory()) {
                final File[] nodes = aNode.listFiles();
                for (File element : nodes) {
                    traverse(element, aFiles);
                }
            }
            else if (aNode.isFile()) {
                aFiles.add(aNode);
            }
        }
    }





----------------------------------------------------------------------

>Comment By: Oliver Burn (oburn)
Date: 2008-09-18 20:13

Message:
this is a feature request

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397081&aid=2114350&group_id=29721

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Checkstyle-devel mailing list
Checkstyle-devel@...
https://lists.sourceforge.net/lists/listinfo/checkstyle-devel