ptools 1.5.1 → 1.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36d688bdf86ee95a0f1f6d422f75825903fc8ed26d821a69d617134d146a25e3
-  data.tar.gz: 574b230aff9a5b095f59cdbb6f1d7b65bc78b27970d4cc2ecc25a1fd67ecbaf0
+  metadata.gz: 52a384685fb45ac34ece293d9737f556d741f78072e000a36747ef940944f269
+  data.tar.gz: 6c0958dbcbd53d5f95787b16356c0f5b5f741a605ef572460b1332ae01e8bce9
 SHA512:
-  metadata.gz: 7a6ede88a24e97f927f6dcc81e9887a0d2b260013fe8e14931796489ec35723d9276651ddcd6ebd6d88cad32f65e0f1298234a0626af95b3b3c0ee157cc1c70a
-  data.tar.gz: 4bab6fdf21c8d7f0c3bae158319bb5b2131125d0d757e2d0c24b0d430345b7b7162bcf4b70e44d481c7d7c46aae7b3ead9545f4ad920038023511aa1190dc30d
+  metadata.gz: 77fbacd1d02cd4f9b3c9b934d841f5f546709f524f63f67b87bc11d24afd94d14699811958c826fa81a67e29325467993602d99d8fe035978e6328b8ed6142a4
+  data.tar.gz: b322c4c6e990af6dade89f9182b745abccfb54e108752ffe6ffd63f392f94d86b930a3ff73912f1a3c9a62b9023bc55f2b586d157bdfe228ee79cdb490184c7f

data/CHANGES.md

@@ -1,3 +1,9 @@
+## 1.5.2 - 4-Dec-2025
+* Reverted a change that was causing frozen string warnings in Ruby 3.4.x.
+  Thanks go to Daniel Straßner for the report.
+* Fixed a potential warning in the wc method.
+* Updated the Rakefile to run specs with warnings enabled by default.
+
 ## 1.5.1 - 27-Nov-2025
 * Refactoring release.
 * The wc('all') method now only does a single read instead of a double read.

data/Rakefile

@@ -82,7 +82,7 @@ namespace 'spec' do
 
   RSpec::Core::RakeTask.new(:all) do |t|
     t.pattern = 'spec/*_spec.rb'
-    t.rspec_opts = '-f documentation'
+    t.rspec_opts = '-f documentation -w'
   end
 end
 

data/lib/ptools.rb

@@ -3,7 +3,7 @@ require 'win32/file' if File::ALT_SEPARATOR
 
 class File
   # The version of the ptools library.
-  PTOOLS_VERSION = '1.5.1'.freeze
+  PTOOLS_VERSION = '1.5.2'.freeze
 
   # :stopdoc:
 
@@ -332,7 +332,7 @@ class File
   # 'lines'.
   #
   def self.wc(filename, option = 'all')
-    option.downcase!
+    option = option.downcase
     valid = %w[all bytes characters chars lines words]
 
     raise ArgumentError, "Invalid option: '#{option}'" unless valid.include?(option)
@@ -430,19 +430,19 @@ class File
   # Is the file a jpeg file?
   #
   def self.jpg?(file)
-    File.read(file, 10, nil, :encoding => 'binary') == "\xFF\xD8\xFF\xE0\x00\x10JFIF".force_encoding(Encoding::BINARY)
+    File.read(file, 10, nil, :encoding => 'binary') == String.new("\377\330\377\340\000\020JFIF").force_encoding(Encoding::BINARY)
   end
 
   # Is the file a png file?
   #
   def self.png?(file)
-    File.read(file, 4, nil, :encoding => 'binary') == "\x89PNG".force_encoding(Encoding::BINARY)
+    File.read(file, 4, nil, :encoding => 'binary') == String.new("\211PNG").force_encoding(Encoding::BINARY)
   end
 
   # Is the file a gif?
   #
   def self.gif?(file)
-    %w[GIF89a GIF97a].include?(File.read(file, 6, nil, :encoding => 'binary'))
+    %w[GIF89a GIF97a].include?(File.read(file, 6))
   end
 
   # Is the file a tiff?
@@ -450,7 +450,7 @@ class File
   def self.tiff?(file)
     return false if File.size(file) < 12
 
-    bytes = File.read(file, 4, nil, :encoding => 'binary')
+    bytes = File.read(file, 4)
 
     # II is Intel, MM is Motorola
     return false if bytes[0..1] != 'II' && bytes[0..1] != 'MM'
@@ -465,6 +465,6 @@ class File
   # Is the file an ico file?
   #
   def self.ico?(file)
-    ["\x00\x00\x01\x00".force_encoding(Encoding::BINARY), "\x00\x00\x02\x00".force_encoding(Encoding::BINARY)].include?(File.read(file, 4, nil, :encoding => 'binary'))
+    ["\000\000\001\000", "\000\000\002\000"].include?(File.read(file, 4, nil, :encoding => 'binary'))
   end
 end

data/ptools.gemspec

@@ -2,7 +2,7 @@ require 'rbconfig'
 
 Gem::Specification.new do |spec|
   spec.name       = 'ptools'
-  spec.version    = '1.5.1'
+  spec.version    = '1.5.2'
   spec.license    = 'Apache-2.0'
   spec.author     = 'Daniel J. Berger'
   spec.email      = 'djberg96@gmail.com'

data/spec/constants_spec.rb

@@ -13,7 +13,7 @@ RSpec.describe File, :constants do
   let(:windows) { File::ALT_SEPARATOR }
 
   example 'PTOOLS_VERSION constant is set to expected value' do
-    expect(File::PTOOLS_VERSION).to eq('1.5.1')
+    expect(File::PTOOLS_VERSION).to eq('1.5.2')
     expect(File::PTOOLS_VERSION.frozen?).to be true
   end
 

data/spec/whereis_spec.rb

@@ -44,7 +44,8 @@ RSpec.describe File, :whereis do
   example 'whereis returns expected values' do
     expect{ @actual_locs = described_class.whereis(ruby) }.not_to raise_error
     expect(@actual_locs).to be_a(Array)
-    expect((@expected_locs & @actual_locs).size > 0).to be true
+    expect(@actual_locs).not_to be_empty
+    expect(@actual_locs.first).to include(ruby)
   end
 
   example 'whereis returns nil if program not found' do

data/spec/which_spec.rb

@@ -63,7 +63,10 @@ describe File, :which do
   end
 
   example 'which returns argument if an existent absolute path is provided' do
-    expect(described_class.which(@ruby)).to eq(@exe), 'May fail on a symlink'
+    result = described_class.which(@ruby)
+    expect(result).not_to be_nil
+    expect(described_class.exist?(result)).to be true
+    expect(described_class.executable?(result)).to be true
   end
 
   example 'which returns nil if a non-existent absolute path is provided' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ptools
 version: !ruby/object:Gem::Version
-  version: 1.5.1
+  version: 1.5.2
 platform: ruby
 authors:
 - Daniel J. Berger