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 /org.aspectj.ajdt.core | |
| 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 'org.aspectj.ajdt.core')
| -rw-r--r-- | org.aspectj.ajdt.core/src/main/java/org/aspectj/ajdt/internal/core/builder/AjState.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/org.aspectj.ajdt.core/src/main/java/org/aspectj/ajdt/internal/core/builder/AjState.java b/org.aspectj.ajdt.core/src/main/java/org/aspectj/ajdt/internal/core/builder/AjState.java index 54fee3bfa..1e3310cd7 100644 --- a/org.aspectj.ajdt.core/src/main/java/org/aspectj/ajdt/internal/core/builder/AjState.java +++ b/org.aspectj.ajdt.core/src/main/java/org/aspectj/ajdt/internal/core/builder/AjState.java @@ -993,7 +993,7 @@ public class AjState implements CompilerConfigurationChangeFlags, TypeDelegateRe if (f.exists() && !f.isDirectory() && (f.lastModified() >= lastSuccessfulBuildTime)) { return true; } - if (checkClassFiles && f.exists() && f.isDirectory()) { + if (checkClassFiles && f.isDirectory()) { // We should use here a list/set of directories we know have or have not changed - some kind of // List<File> buildConfig.getClasspathEntriesWithChangedContents() @@ -1045,7 +1045,7 @@ public class AjState implements CompilerConfigurationChangeFlags, TypeDelegateRe if (f.exists() && !f.isDirectory() && (f.lastModified() >= lastSuccessfulBuildTime)) { return true; } - if (checkClassFiles && f.exists() && f.isDirectory()) { + if (checkClassFiles && f.isDirectory()) { // We should use here a list/set of directories we know have or have not changed - some kind of // List<File> buildConfig.getClasspathEntriesWithChangedContents() |