You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@creadur.apache.org by rd...@apache.org on 2013/12/08 22:32:36 UTC
svn commit: r1549303 -
/creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/walker/Walker.java
Author: rdonkin
Date: Sun Dec 8 21:32:35 2013
New Revision: 1549303
URL: http://svn.apache.org/r1549303
Log:
Apply PMD change.
Modified:
creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/walker/Walker.java
Modified: creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/walker/Walker.java
URL: http://svn.apache.org/viewvc/creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/walker/Walker.java?rev=1549303&r1=1549302&r2=1549303&view=diff
==============================================================================
--- creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/walker/Walker.java (original)
+++ creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/walker/Walker.java Sun Dec 8 21:32:35 2013
@@ -48,12 +48,12 @@ public abstract class Walker implements
*/
protected static FilenameFilter regexFilter(final Pattern pattern) {
return new FilenameFilter() {
- public boolean accept(File dir, String name) {
- final boolean result;
+ public boolean accept(final File dir, final String name) {
+ boolean result = false;
if (pattern == null) {
result = true;
} else {
- result = !pattern.matcher(name).matches();
+ result ^= pattern.matcher(name).matches();
}
return result;
}
@@ -67,10 +67,9 @@ public abstract class Walker implements
* the file
* @return true, if is restricted
*/
- protected boolean isRestricted(File file) {
+ protected boolean isRestricted(final File file) {
String name = file.getName();
- boolean result = name.startsWith(".");
- return result;
+ return name.charAt(0) == '.';
}
/**
@@ -85,7 +84,7 @@ public abstract class Walker implements
if (filter != null) {
final String name = file.getName();
final File dir = file.getParentFile();
- result = !filter.accept(dir, name);
+ result ^= filter.accept(dir, name);
}
return result;
}
@@ -98,7 +97,7 @@ public abstract class Walker implements
* @param filter
* the filter
*/
- public Walker(File file, final FilenameFilter filter) {
+ public Walker(final File file, final FilenameFilter filter) {
this(file.getPath(), file, filter);
}