Continue to remove nulls as requested in PR #802

yegor256 · Jan 4, 2019 · b9f248e · b9f248e · 0pdd · Jan 4, 2019
1 parent 6e4df63
commit b9f248e
Showing 1 changed file with 5 additions and 4 deletions.
diff --git 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;