From b9f248eb586b768198598120e4ae3ee4c5770e8b Mon Sep 17 00:00:00 2001 From: marceloamadeu Date: Fri, 4 Jan 2019 12:56:05 -0200 Subject: [PATCH] Continue to remove nulls as requested in PR #802 --- src/main/java/org/takes/facets/fork/FkHitRefresh.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/takes/facets/fork/FkHitRefresh.java b/src/main/java/org/takes/facets/fork/FkHitRefresh.java index 98dd7302b..dbabb63a7 100644 --- a/src/main/java/org/takes/facets/fork/FkHitRefresh.java +++ b/src/main/java/org/takes/facets/fork/FkHitRefresh.java @@ -234,9 +234,10 @@ private boolean expired() throws IOException { /** * Directory contents updated? * @return TRUE if contents were updated - * @todo #744:30min `null` constant usage in the function - * `directoryUpdated` is violating the key principles, defined - * in README.md file and must be eliminated. + * @todo #802:30min Continue removing nulls from the codebase, + * which still exist in the code. The way it is violates + * the key principles, defined in the README.md file and should be + * deleted. */ private boolean directoryUpdated() { final long recent; @@ -248,7 +249,7 @@ private boolean directoryUpdated() { } final File[] files = this.dir.listFiles(); boolean expired = false; - if (files != null) { + if (new Opt.Single<>(files).has()) { for (final File file : files) { if (file.lastModified() > recent) { expired = true;