win32-file 0.7.1 → 0.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 047468a1a793c4485f3e336084cbdab03ec973ee
-  data.tar.gz: f2cab49afe63c66b94995e83c0dcf7374ba52efa
+  metadata.gz: f3ec34d5df87b28440da5d0476baf65bd6b8a49b
+  data.tar.gz: 1572144c7862601cd65bc652e903ee08d6818b05
 SHA512:
-  metadata.gz: eae986e7739d2fc6307f2ded1c1349ae1e0590248f01c98456f368215f93ba4499c358bc80986f3f80e2e83f0e62693f81565e795af1b9f71880ae965c751218
-  data.tar.gz: 04634af88ef4466e08e8506e3c661d3afabb7fe31cf42c34f22b7ef121b9d3b07c565ba1964e079448300f5c6fecc0231de5649a5500b362c75709b8bfd76e42
+  metadata.gz: 9b401fa9b87331da00cf249adae527d020387241acb7687df4a6c446d554f0a35f515779b5d01e5387422bdff407b9bedf38576e11bd356050a202405e890ad1
+  data.tar.gz: baef5724c05e13c1e0c49e02aa6a9487916a8cbaa07fdd1241d775b941052ebc613b21419efda22bf383e04daebc764b1487b5989522a46c247b23958d473ce6

data/CHANGES

@@ -1,3 +1,9 @@
+== 0.7.2 - 7-Oct-2014
+* Replaced File.exists? with File.exist? to avoid deprecation warnings
+  for Ruby 2.x.
+* Some minor memory improvements by explicitly freeing some pointers.
+* Minor README updates.
+
 == 0.7.1 - 28-Apr-2014
 * Modified all custom singleton methods to accept arguments that define either
   to_str or to_path to be in line with MRI's behavior.

data/README

@@ -3,7 +3,6 @@
   methods have been redefined to make them work properly on MS Windows.
 
 == Prerequisites
-  * windows-pr
   * win32-file-stat
 
 == Installation
@@ -49,9 +48,9 @@
   * File.world_writable?  # Not implemented in MRI
 
 == Known issues or bugs
-  None that I'm aware of.
+  The File.exist? method will return true on stale symlinks.
 
-  Please report any issues you find on the github page at:
+  Please report any other issues you find on the github page at:
 
   https://github.com/djberg96/win32-file/issues
 

data/Rakefile

@@ -19,7 +19,7 @@ namespace 'gem' do
   desc 'Install the win32-file gem'
   task :install => [:create] do
     file = Dir['win32-file*.gem'].first
-    sh "gem install #{file}"
+    sh "gem install -l #{file}"
   end
 end
 

data/lib/win32/file.rb

@@ -9,7 +9,7 @@ class File
   extend Windows::File::Functions
 
   # The version of the win32-file library
-  WIN32_FILE_VERSION = '0.7.1'
+  WIN32_FILE_VERSION = '0.7.2'
 
   class << self
     alias_method :join_orig, :join
@@ -82,6 +82,8 @@ class File
     file = wfile.encode(encoding)[/^[^\0]*/]
     file.sub!(/\\+\z/, '') # Trim trailing slashes
 
+    ptr.free
+
     file
   end
 
@@ -138,6 +140,8 @@ class File
     # Return to original encoding
     file = wfile.tr(0.chr, '').encode(encoding)
 
+    ptr.free
+
     file
   end
 
@@ -235,7 +239,7 @@ class File
   # Returns whether or not +file+ is a symlink.
   #
   def self.symlink?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
 
     bool  = false
     wfile = string_check(file).wincode
@@ -302,7 +306,7 @@ class File
     if symlink?(file)
       if relative_to
         result = File.join(relative_to, File.basename(readlink(file)))
-        if File.exists?(result)
+        if File.exist?(result)
           result
         else
           raise SystemCallError.new(result, 2) # Errno::ENOENT
@@ -317,15 +321,17 @@ class File
 
   # Returns the path of the of the symbolic link referred to by +file+.
   #
+  # Unlike unixy platforms, this will raise an error if the link is stale.
+  #
   def self.readlink(file)
     file = string_check(file)
 
-    if exists?(file) && !symlink?(file)
+    if exist?(file) && !symlink?(file)
       raise SystemCallError.new(22) # EINVAL, match the spec
     end
 
     wfile = file.wincode
-    path  = 0.chr * 512
+    path  = 0.chr * 1024
 
     if File.directory?(file)
       flags = FILE_FLAG_BACKUP_SEMANTICS
@@ -370,35 +376,35 @@ class File
   # block device is a removable drive, cdrom or ramdisk.
   #
   def self.blockdev?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).blockdev?
   end
 
   # Returns whether or not the file is a character device.
   #
   def self.chardev?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).chardev?
   end
 
   # Returns whether or not the file is a directory.
   #
   def self.directory?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).directory?
   end
 
   # Returns whether or not the file is executable.
   #
   def self.executable?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).executable?
   end
 
   # Returns whether or not the file is a regular file.
   #
   def self.file?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).file?
   end
 
@@ -412,35 +418,35 @@ class File
   # Returns true if the process owner's ID is the same as one of the file's groups.
   #
   def self.grpowned?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).grpowned?
   end
 
   # Returns whether or not the current process owner is the owner of the file.
   #
   def self.owned?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).owned?
   end
 
   # Returns whether or not the file is a pipe.
   #
   def self.pipe?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).pipe?
   end
 
   # Returns whether or not the file is readable by the process owner.
   #
   def self.readable?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).readable?
   end
 
   # Synonym for File.readable?
   #
   def self.readable_real?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).readable_real?
   end
 
@@ -454,7 +460,7 @@ class File
   # merely returns true or false, not permission bits (or nil).
   #
   def self.world_readable?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).world_readable?
   end
 
@@ -462,21 +468,21 @@ class File
   # merely returns true or false, not permission bits (or nil).
   #
   def self.world_writable?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).world_writable?
   end
 
   # Returns whether or not the file is writable by the current process owner.
   #
   def self.writable?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).writable?
   end
 
   # Synonym for File.writable?
   #
   def self.writable_real?(file)
-    return false unless File.exists?(file)
+    return false unless File.exist?(file)
     File::Stat.new(file).writable_real?
   end
 

data/test/test_win32_file_link.rb

@@ -28,7 +28,7 @@ class TC_Win32_File_Link < Test::Unit::TestCase
   test "symlink to a file works as expected" do
     omit_unless(@admin)
     assert_nothing_raised{ File.symlink(@@file, @link) }
-    assert_true(File.exists?(@link))
+    assert_true(File.exist?(@link))
     assert_true(File.symlink?(@link))
   end
 
@@ -135,7 +135,7 @@ class TC_Win32_File_Link < Test::Unit::TestCase
   end
 
   def self.shutdown
-    File.delete(@@file) if File.exists?(@@file)
+    File.delete(@@file) if File.exist?(@@file)
     @@file = nil
   end
 end

data/test/test_win32_file_misc.rb

@@ -6,7 +6,7 @@ require 'test-unit'
 
 class TC_Win32_File_Misc < Test::Unit::TestCase
   test "version constant is set to expected value" do
-    assert_equal('0.7.1', File::WIN32_FILE_VERSION)
+    assert_equal('0.7.2', File::WIN32_FILE_VERSION)
   end
 
   test "ffi functions are private" do

data/test/test_win32_file_path.rb

@@ -276,7 +276,7 @@ class TC_Win32_File_Path < Test::Unit::TestCase
   end
 
   def self.shutdown
-    File.delete(@@file) if File.exists?(@@file)
+    File.delete(@@file) if File.exist?(@@file)
     @@file = nil
   end
 end

data/test/test_win32_file_stat.rb

@@ -79,7 +79,7 @@ class TC_Win32_File_Stat < Test::Unit::TestCase
 
   test "blockdev? returns true for block devices" do
     omit_unless(@@block_dev)
-    omit_unless(File.exists?(@@block_dev), "No media in device - skipping")
+    omit_unless(File.exist?(@@block_dev), "No media in device - skipping")
     assert_true(File.blockdev?(@@block_dev))
   end
 
@@ -273,8 +273,8 @@ class TC_Win32_File_Stat < Test::Unit::TestCase
   end
 
   def self.shutdown
-    File.delete(@@txt_file) if File.exists?(@@txt_file)
-    File.delete(@@exe_file) if File.exists?(@@exe_file)
+    File.delete(@@txt_file) if File.exist?(@@txt_file)
+    File.delete(@@exe_file) if File.exist?(@@exe_file)
     @@txt_file = nil
     @@exe_file = nil
     @@elevated = nil

data/win32-file.gemspec

@@ -2,7 +2,7 @@ require 'rubygems'
 
 Gem::Specification.new do |spec|
   spec.name       = 'win32-file'
-  spec.version    = '0.7.1'
+  spec.version    = '0.7.2'
   spec.authors    = ['Daniel J. Berger', 'Park Heesob']
   spec.license    = 'Artistic 2.0'
   spec.email      = 'djberg96@gmail.com'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: win32-file
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.2
 platform: ruby
 authors:
 - Daniel J. Berger
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-28 00:00:00.000000000 Z
+date: 2014-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi
@@ -128,7 +128,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: win32utils
-rubygems_version: 2.2.2
+rubygems_version: 2.4.1
 signing_key: 
 specification_version: 4
 summary: Extra or redefined methods for the File class on Windows.