From e46ce2bf25ff7417f310bd340e3c640808d8659b Mon Sep 17 00:00:00 2001
From: Colin Kelley <colindkelley@gmail.com>
Date: Sun, 9 Jan 2022 15:20:47 -0800
Subject: [PATCH] issue #548: use empty_dirname? method

---
 lib/listen/record.rb | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/lib/listen/record.rb b/lib/listen/record.rb
index efa6a37a..28305edc 100644
--- a/lib/listen/record.rb
+++ b/lib/listen/record.rb
@@ -18,7 +18,7 @@ def initialize(directory, silencer)
     end
 
     def add_dir(rel_path)
-      if ![nil, '', '.'].include?(rel_path)
+      if !empty_dirname?(rel_path.to_s)
         @tree[rel_path] ||= {}
       end
     end
@@ -35,7 +35,7 @@ def unset_path(rel_path)
 
     def file_data(rel_path)
       dirname, basename = Pathname(rel_path).split.map(&:to_s)
-      if [nil, '', '.'].include?(dirname)
+      if empty_dirname?(dirname)
         @tree[basename] ||= {}
         @tree[basename].dup
       else
@@ -47,7 +47,7 @@ def file_data(rel_path)
 
     def dir_entries(rel_path)
       rel_path_s = rel_path.to_s
-      subtree = if ['', '.'].include?(rel_path_s)
+      subtree = if empty_dirname?(rel_path_s)
         @tree
       else
         @tree[rel_path_s]
@@ -74,12 +74,16 @@ def build
 
     private
 
+    def empty_dirname?(dirname)
+      dirname == '.' || dirname == ''
+    end
+
     def reset_tree
       @tree = Hash.new { |h, k| h[k] = {} }
     end
 
     def _fast_update_file(dirname, basename, data)
-      if [nil, '', '.'].include?(dirname)
+      if empty_dirname?(dirname.to_s)
         @tree[basename] = (@tree[basename] || {}).merge(data)
       else
         @tree[dirname] ||= {}
@@ -90,7 +94,7 @@ def _fast_update_file(dirname, basename, data)
     def _fast_unset_path(dirname, basename)
       # this may need to be reworked to properly remove
       # entries from a tree, without adding non-existing dirs to the record
-      if [nil, '', '.'].include?(dirname)
+      if empty_dirname?(dirname.to_s)
         if @tree.key?(basename)
           @tree.delete(basename)
         end