diff options
| author | Andrey Turbanov <turbanoff@gmail.com> | 2021-11-20 16:53:30 +0300 |
|---|---|---|
| committer | Andrey Turbanov <turbanoff@gmail.com> | 2021-11-20 16:53:30 +0300 |
| commit | 793a015abbdf77e20052362db626d9aea855c16b (patch) | |
| tree | fb01864e965e38f4f6f5c8fb0f78e4f5be4abef9 /build | |
| parent | 0f85ca109b9e6ab849e201e76f62d5023cbfcb98 (diff) | |
| download | aspectj-793a015abbdf77e20052362db626d9aea855c16b.tar.gz aspectj-793a015abbdf77e20052362db626d9aea855c16b.zip | |
According to javadoc File.isDirectory 'true' if and only if the file denoted by this abstract pathname exists and is a directory.
It means that separate File.exists() check before File.isDirectory() check is redundant.
Diffstat (limited to 'build')
| -rw-r--r-- | build/src/test/java/org/aspectj/build/BuildModuleTests.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/build/src/test/java/org/aspectj/build/BuildModuleTests.java b/build/src/test/java/org/aspectj/build/BuildModuleTests.java index 64f8b1016..181d84fcc 100644 --- a/build/src/test/java/org/aspectj/build/BuildModuleTests.java +++ b/build/src/test/java/org/aspectj/build/BuildModuleTests.java @@ -175,7 +175,7 @@ public class BuildModuleTests extends TestCase { void checkSourceDirectory(File srcDir, String module) { final String label = "source dir " + srcDir + " (module " + module + ")"; - assertTrue(label, (srcDir.exists() && srcDir.isDirectory())); + assertTrue(label, srcDir.isDirectory()); String license = getLicense(module); // if (replacing) { // if (replacing && true) { |