pnm 0.6.1 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fdfdb7ac32419e9381856654116f77070fabb87713e2353c10230fb6ac9c79c9
-  data.tar.gz: 06042a5b1c3d26d5b4e982b3e2e4983b652d1e1ffab8987d5f8b183a017c4abb
+  metadata.gz: ef6f55635bbaa23976ceb49f1547b92cf78e2ad04d51363054ac12dbfe6943e0
+  data.tar.gz: 4b7c8730c80b28b6a98a4b27fc3a000a4e28eb11be34bf5c2717da188f833d9a
 SHA512:
-  metadata.gz: 17970b6a53282e449e76f9225e65c914f92084e4060b5fbe3b9bbbd56cae9ae830e122fbf8ed0bbb321bf443ef1c9538d4b8a2b859759528aa8862d479375796
-  data.tar.gz: b66f7f50567bb137a5b091293c7b77206966e570e300157a0ebde66647e46f2c09d07cb093499f3527a5b6149d01a0f88baa235c5450e61d9b0ba02db2a2cb5e
+  metadata.gz: f3b07074daa3fd1601d8d0c4f05dc2764c1c2434b24368d0e69c4752ecc9af1153751f14f1af679d2319d46b20b74cff4c9e10e50341eb12fbb82abc97ca69d5
+  data.tar.gz: 5356021622e4656db38f8f7c7b918c5e3bbffc63670e4b39b851ca542f55f5a51b7748b2384540426d7e154ebb3b151c4d106cd1adee00c526d6e249753e54c6

data/README.md

@@ -98,11 +98,11 @@ Requirements
 
 - No additional Ruby gems or native libraries are needed.
 
-- PNM has been tested with
+- PNM's current version has been tested with
 
-  - Ruby 3.1, 3.0,
-  - Ruby 2.7, 2.6, 2.5, 2.4, 2.3, 2.2, 2.1, 2.0.0,
-  - JRuby 9.3.4.0.
+  - Ruby 3.3, 3.2, 3.1, 3.0,
+  - Ruby 2.7, 2.6,
+  - JRuby 9.4.5.0.
 
 Documentation
 -------------
@@ -118,7 +118,7 @@ Report bugs on the PNM home page: <https://github.com/stomar/pnm/>
 License
 -------
 
-Copyright &copy; 2013-2022 Marcus Stollsteimer
+Copyright &copy; 2013-2024 Marcus Stollsteimer
 
 `PNM` is free software: you can redistribute it and/or modify
 it under the terms of the GNU General Public License version 3 or later (GPLv3+),

data/Rakefile

@@ -1,9 +1,5 @@
 # frozen_string_literal: true
 
-# rakefile for the PNM library.
-#
-# Copyright (C) 2013-2022 Marcus Stollsteimer
-
 require "rake/testtask"
 
 require_relative "lib/pnm"
@@ -25,7 +21,7 @@ end
 
 desc "Run benchmarks"
 task :benchmark do
-  Dir["benchmark/**/bm_*.rb"].each {|f| require_relative f }
+  Dir["benchmark/**/bm_*.rb"].sort.each {|f| require_relative f }
 end
 
 

data/benchmark/bm_converter.rb

@@ -1,10 +1,6 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
-# bm_converter.rb: Benchmarks for the PNM library.
-#
-# Copyright (C) 2013-2022 Marcus Stollsteimer
-
 require "benchmark"
 require_relative "../lib/pnm"
 
@@ -12,8 +8,8 @@ require_relative "../lib/pnm"
 class ConverterBenchmark
 
   def initialize
-    @repetitions = ARGV[0].to_i  if ARGV[0] =~ /[0-9]+/
-    @repetitions ||= 10
+    @repetitions = ENV.fetch("BM_RUNS", "")
+    @repetitions = @repetitions.match?(/\A[0-9]+\z/) ? @repetitions.to_i : 10
 
     @srcpath = File.dirname(__FILE__)
 

data/lib/pnm/converter.rb

@@ -53,14 +53,14 @@ module PNM
       bytes_per_row = byte_width(type, width)
       expected_size = bytes_per_row * height
 
-      data.slice!(-1)  if data.size == expected_size + 1 && data[-1] =~ /[ \t\r\n]/
+      data.slice!(-1)  if data.size == expected_size + 1 && data[-1].match?(/[ \t\r\n]/)
 
       assert_data_size(data.size, expected_size)
 
       pixel_matrix = case type
                      when :pbm
                        rows = data.scan(/.{#{bytes_per_row}}/m)
-                       rows.map {|row| row.unpack("B*").first[0, width].each_char.map(&:to_i) }
+                       rows.map {|row| row.unpack1("B*")[0, width].each_char.map(&:to_i) }
                      when :pgm
                        data.each_byte.each_slice(bytes_per_row).to_a
                      when :ppm
@@ -109,7 +109,7 @@ module PNM
       values_as_string = if type == :pbm
                            data.gsub(/[ \t\r\n]+/, "").chars
                          else
-                           data.gsub(/\A[ \t\r\n]+/, "").split(/[ \t\r\n]+/)
+                           data.sub(/\A[ \t\r\n]+/, "").split(/[ \t\r\n]+/)
                          end
 
       values_as_string.map {|value| Integer(value) }

data/lib/pnm/image.rb

@@ -181,7 +181,7 @@ module PNM
 
     def self.assert_valid_maxgray(maxgray)  # :nodoc:
       return  unless maxgray
-      return  if maxgray.is_a?(Integer) && maxgray > 0 && maxgray <= 255
+      return  if maxgray.is_a?(Integer) && maxgray.positive? && maxgray <= 255
 
       msg = "invalid maxgray value - #{maxgray.inspect}"
       raise PNM::ArgumentError, msg
@@ -258,7 +258,7 @@ module PNM
       return [""]  if comment.empty?
 
       keep_trailing_null_fields = -1  # magic value for split limit
-      comment.split(/\n/, keep_trailing_null_fields)
+      comment.split("\n", keep_trailing_null_fields)
     end
 
     def to_ascii  # :nodoc:

data/lib/pnm/parser.rb

@@ -29,7 +29,7 @@ module PNM
         token = next_token!(content)
 
         if token.start_with?("#")
-          comments << token.gsub(/# */, "")
+          comments << strip_comment_prefix(token)
         else
           magic_number = token
         end
@@ -38,11 +38,11 @@ module PNM
       assert_valid_magic_number(magic_number)
 
       while tokens.size < token_number[magic_number]
-        content.gsub!(/\A[ \t\r\n]+/, "")
+        content.sub!(/\A[ \t\r\n]+/, "")
         token = next_token!(content)
 
         if token.start_with?("#")
-          comments << token.gsub(/# */, "")
+          comments << strip_comment_prefix(token)
         else
           tokens << token
         end
@@ -100,6 +100,10 @@ module PNM
       token
     end
 
+    def self.strip_comment_prefix(comment)
+      comment.sub(/\A# */, "")
+    end
+
     def self.assert_valid_magic_number(magic_number)
       return  if %w[P1 P2 P3 P4 P5 P6].include?(magic_number)
 
@@ -108,21 +112,21 @@ module PNM
     end
 
     def self.assert_integer(value_string, value_name)
-      return  if value_string =~ /\A[0-9]+\Z/
+      return  if value_string.match?(/\A[0-9]+\z/)
 
       msg = "#{value_name} must be an integer - `#{value_string}'"
       raise PNM::ParserError, msg
     end
 
     def self.assert_positive(value, name)
-      return  if value > 0
+      return  if value.positive?
 
       msg = "#{name} must be greater than 0 - `#{value}'"
       raise PNM::ParserError, msg
     end
 
     def self.assert_in_maxgray_range(value)
-      return  if value > 0 && value <= 255
+      return  if value.positive? && value <= 255
 
       msg = "invalid maxgray value - `#{value}'"
       raise PNM::ParserError, msg

data/lib/pnm/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module PNM
-  VERSION = "0.6.1"
-  DATE    = "2022-05-09"
+  VERSION = "0.7.0"
+  DATE    = "2024-01-05"
 end

data/lib/pnm.rb

@@ -87,7 +87,7 @@ require_relative "pnm/exceptions"
 #
 # == Author
 #
-# Copyright (C) 2013-2022 Marcus Stollsteimer
+# Copyright (C) 2013-2024 Marcus Stollsteimer
 #
 # License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
 #
@@ -97,8 +97,8 @@ module PNM
   HOMEPAGE = "https://github.com/stomar/pnm"
   TAGLINE  = "create/read/write PNM image files (PBM, PGM, PPM)"
 
-  COPYRIGHT = <<-TEXT.gsub(/^ +/, "")
-    Copyright (C) 2013-2022 Marcus Stollsteimer.
+  COPYRIGHT = <<~TEXT
+    Copyright (C) 2013-2024 Marcus Stollsteimer.
     License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>.
     This is free software: you are free to change and redistribute it.
     There is NO WARRANTY, to the extent permitted by law.

data/pnm.gemspec

@@ -25,7 +25,7 @@ Gem::Specification.new do |s|
 
   s.license = "GPL-3.0"
 
-  s.required_ruby_version = ">= 2.0.0"
+  s.required_ruby_version = ">= 2.6.0"
 
   s.require_paths = ["lib"]
 

data/test/test_converter.rb

@@ -1,14 +1,8 @@
 # frozen_string_literal: true
 
-# test_converter.rb: Unit tests for the PNM library.
-#
-# Copyright (C) 2013-2022 Marcus Stollsteimer
-
 require "minitest/autorun"
 require "pnm/converter"
 
-require_relative "backports"
-
 
 describe PNM::Converter do
 

data/test/test_exceptions.rb

@@ -1,15 +1,9 @@
 # frozen_string_literal: true
 
-# test_exceptions.rb: Unit tests for the PNM library.
-#
-# Copyright (C) 2013-2022 Marcus Stollsteimer
-
 require "minitest/autorun"
 require "stringio"
 require "pnm"
 
-require_relative "backports"
-
 
 describe "PNM.create" do
 

data/test/test_image.rb

@@ -1,14 +1,8 @@
 # frozen_string_literal: true
 
-# test_image.rb: Unit tests for the PNM library.
-#
-# Copyright (C) 2013-2022 Marcus Stollsteimer
-
 require "minitest/autorun"
 require "pnm"
 
-require_relative "backports"
-
 
 describe PNM::Image do
 

data/test/test_parser.rb

@@ -1,14 +1,8 @@
 # frozen_string_literal: true
 
-# test_parser.rb: Unit tests for the PNM library.
-#
-# Copyright (C) 2013-2022 Marcus Stollsteimer
-
 require "minitest/autorun"
 require "pnm/parser"
 
-require_relative "backports"
-
 
 describe PNM::Parser do
 
@@ -17,7 +11,7 @@ describe PNM::Parser do
   end
 
   it "can parse ASCII encoded PBM data" do
-    content = <<-PBM.chomp.gsub(/^ */, "")
+    content = <<~PBM.chomp
       P1 6 2
       0 1 0 0 1 1
       0 0 0 1 1 1
@@ -33,7 +27,7 @@ describe PNM::Parser do
   end
 
   it "can parse ASCII encoded PGM data" do
-    content = <<-PGM.chomp.gsub(/^ */, "")
+    content = <<~PGM.chomp
       P2 4 2 100
       10 20 30 40
       50 60 70 80
@@ -98,7 +92,7 @@ describe PNM::Parser do
   end
 
   it "can parse comments" do
-    content = <<-PBM.chomp.gsub(/^ */, "")
+    content = <<~PBM.chomp
       # Comment 1
       P1  # Comment 2
       6# Comment 3
@@ -119,4 +113,25 @@ describe PNM::Parser do
 
     _(@parser.parse(content)).must_equal expected
   end
+
+  it "can parse a comment containing a # character" do
+    content = <<~PBM.chomp
+      # Comment 1 with # character
+      P1
+      # Comment 2: '#   foo' is a comment
+      3
+      2
+      0 1 0
+      1 0 1
+    PBM
+    expected = {
+      magic_number: "P1",
+      width:        3,
+      height:       2,
+      comments:     ["Comment 1 with # character", "Comment 2: '#   foo' is a comment"],
+      data:         "0 1 0\n1 0 1"
+    }
+
+    _(@parser.parse(content)).must_equal expected
+  end
 end

data/test/test_pnm.rb

@@ -1,14 +1,8 @@
 # frozen_string_literal: true
 
-# test_pnm.rb: Unit tests for the PNM library.
-#
-# Copyright (C) 2013-2022 Marcus Stollsteimer
-
 require "minitest/autorun"
 require "pnm"
 
-require_relative "backports"
-
 
 describe PNM do
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pnm
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.7.0
 platform: ruby
 authors:
 - Marcus Stollsteimer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-05-09 00:00:00.000000000 Z
+date: 2024-01-05 00:00:00.000000000 Z
 dependencies: []
 description: 'PNM is a pure Ruby library for creating, reading, and writing of PNM
   image files (Portable Anymap): PBM (Portable Bitmap), PGM (Portable Graymap), and
@@ -33,7 +33,6 @@ files:
 - lib/pnm/parser.rb
 - lib/pnm/version.rb
 - pnm.gemspec
-- test/backports.rb
 - test/bilevel_2_binary.pbm
 - test/bilevel_ascii.pbm
 - test/bilevel_binary.pbm
@@ -61,14 +60,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.0.0
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.5.3
 signing_key: 
 specification_version: 4
 summary: PNM - create/read/write PNM image files (PBM, PGM, PPM)

data/test/backports.rb

@@ -1,19 +0,0 @@
-# frozen_string_literal: true
-
-module PNM
-  module Backports  # :nodoc:
-    module Minitest  # :nodoc:
-
-      # Provides workaround for missing value wrappers in minitest < 5.6.0.
-      def _(value = nil, &block)
-        block || value
-      end
-    end
-  end
-end
-
-
-if Gem.loaded_specs["minitest"].version < Gem::Version.new("5.6.0")
-  warn "Using workaround for missing value wrappers in minitest < 5.6.0."
-  MiniTest::Spec.send(:include, PNM::Backports::Minitest)
-end