Use constants instead of literals for some fstype calls.

hainesr · Jun 4, 2021 · a56c6d3 · a56c6d3
1 parent babfc62
commit a56c6d3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/zip/filesystem/file.rb b/lib/zip/filesystem/file.rb
@@ -27,7 +27,7 @@ def find_entry(filename)
 
       def unix_mode_cmp(filename, mode)
         e = find_entry(filename)
-        e.fstype == 3 && ((e.external_file_attributes >> 16) & mode) != 0
+        e.fstype == FSTYPE_UNIX && ((e.external_file_attributes >> 16) & mode) != 0
       rescue Errno::ENOENT
         false
       end
@@ -121,7 +121,7 @@ def chown(owner, group, *filenames)
       def chmod(mode, *filenames)
         filenames.each do |filename|
           e = find_entry(filename)
-          e.fstype = 3 # force convertion filesystem type to unix
+          e.fstype = FSTYPE_UNIX # Force conversion filesystem type to unix.
           e.unix_perms = mode
           e.external_file_attributes = mode << 16
           e.dirty = true

diff --git a/lib/zip/filesystem/file_stat.rb b/lib/zip/filesystem/file_stat.rb
@@ -92,7 +92,7 @@ def blksize
 
         def mode
           e = find_entry
-          if e.fstype == 3
+          if e.fstype == FSTYPE_UNIX
             e.external_file_attributes >> 16
           else
             0o100_666 # Equivalent to -rw-rw-rw-.