rmagick 4.0.0 → 5.3.0

data/lib/rmagick/version.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Magick
-  VERSION = '4.0.0'
+  VERSION = '5.3.0'
   MIN_RUBY_VERSION = '2.3.0'
   MIN_IM_VERSION = '6.7.7'
 end

data/lib/rmagick.rb

@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require 'rmagick_internal.rb'

data/lib/rmagick_internal.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # $Id: RMagick.rb,v 1.84 2009/09/15 22:08:41 rmagick Exp $
 #==============================================================================
 #                  Copyright (C) 2009 by Timothy P. Hunter
@@ -12,42 +14,51 @@ if RUBY_PLATFORM =~ /mingw/i
   begin
     require 'ruby_installer'
     ENV['PATH'].split(File::PATH_SEPARATOR).grep(/ImageMagick/i).each do |path|
-      RubyInstaller::Runtime.add_dll_directory(path)
+      RubyInstaller::Runtime.add_dll_directory(path) if File.exist?(File.join(path, 'CORE_RL_magick_.dll')) || File.exist?(File.join(path, 'CORE_RL_MagickCore_.dll'))
     end
   rescue LoadError
   end
 end
 
 require 'English'
+require 'observer'
 require 'RMagick2.so'
 
 module Magick
-  @formats = nil
-  @trace_proc = nil
-  @exit_block_set_up = nil
+  IMAGEMAGICK_VERSION = Magick::Magick_version.split[1].split('-').first
 
   class << self
+    # Describes the image formats supported by ImageMagick.
+    # If the optional block is present, calls the block once for each image format.
+    # The first argument, +k+, is the format name. The second argument, +v+, is the
+    # properties string described below.
+    #
+    # - +B+ is "*" if the format has native blob support, or " " otherwise.
+    # - +R+ is "r" if ImageMagick can read that format, or "-" otherwise.
+    # - +W+ is "w" if ImageMagick can write that format, or "-" otherwise.
+    # - +A+ is "+" if the format supports multi-image files, or "-" otherwise.
+    #
+    # @overload formats
+    #   @return [Hash] the formats hash
+    #
+    # @overload formats
+    #   @yield [k, v]
+    #   @yieldparam k [String] the format name
+    #   @yieldparam v [String] the properties string
+    #   @return [Magick]
+    #
+    # @example
+    #   p Magick.formats
+    #   => {"3FR"=>" r-+", "3G2"=>" r-+", "3GP"=>" r-+", "A"=>"*rw+",
+    #   ...
     def formats
-      @formats ||= init_formats
+      formats = init_formats
 
       if block_given?
-        @formats.each { |k, v| yield k, v }
+        formats.each { |k, v| yield k, v }
         self
       else
-        @formats
-      end
-    end
-
-    # remove reference to the proc at exit
-    def trace_proc=(p)
-      m = Mutex.new
-      m.synchronize do
-        if @trace_proc.nil? && !p.nil? && !@exit_block_set_up
-          at_exit { @trace_proc = nil }
-          @exit_block_set_up = true
-        end
-
-        @trace_proc = p
+        formats
       end
     end
   end
@@ -65,13 +76,15 @@ module Magick
   MinimumGeometry  = GeometryValue.new(:MinimumGeometry, 6).freeze
 
   class Geometry
-    FLAGS = ['', '%', '!', '<', '>', '@', '^']
-    RFLAGS = { '%' => PercentGeometry,
-               '!' => AspectGeometry,
-               '<' => LessGeometry,
-               '>' => GreaterGeometry,
-               '@' => AreaGeometry,
-               '^' => MinimumGeometry }
+    FLAGS = ['', '%', '!', '<', '>', '@', '^'].freeze
+    RFLAGS = {
+      '%' => PercentGeometry,
+      '!' => AspectGeometry,
+      '<' => LessGeometry,
+      '>' => GreaterGeometry,
+      '@' => AreaGeometry,
+      '^' => MinimumGeometry
+    }.freeze
 
     attr_accessor :width, :height, :x, :y, :flag
 
@@ -127,10 +140,10 @@ module Magick
 
     # Convert object to a geometry string
     def to_s
-      str = ''
+      str = String.new
       if @width > 0
         fmt = @width.truncate == @width ? '%d' : '%.2f'
-        str << format(fmt, @width)
+        str << sprintf(fmt, @width)
         str << '%' if @flag == PercentGeometry
       end
 
@@ -138,10 +151,10 @@ module Magick
 
       if @height > 0
         fmt = @height.truncate == @height ? '%d' : '%.2f'
-        str << format(fmt, @height)
+        str << sprintf(fmt, @height)
         str << '%' if @flag == PercentGeometry
       end
-      str << format('%+d%+d', @x, @y) if @x != 0 || @y != 0
+      str << sprintf('%+d%+d', @x, @y) if @x != 0 || @y != 0
       str << FLAGS[@flag.to_i] if @flag != PercentGeometry
       str
     end
@@ -232,20 +245,23 @@ module Magick
     # Apply coordinate transformations to support scaling (s), rotation (r),
     # and translation (t). Angles are specified in radians.
     def affine(sx, rx, ry, sy, tx, ty)
-      primitive 'affine ' + format('%g,%g,%g,%g,%g,%g', sx, rx, ry, sy, tx, ty)
+      primitive 'affine ' + sprintf('%g,%g,%g,%g,%g,%g', sx, rx, ry, sy, tx, ty)
     end
 
     # Set alpha (make transparent) in image according to the specified
     # colorization rule
     def alpha(x, y, method)
       Kernel.raise ArgumentError, 'Unknown paint method' unless PAINT_METHOD_NAMES.key?(method.to_i)
-      primitive 'matte ' + format('%g,%g, %s', x, y, PAINT_METHOD_NAMES[method.to_i])
+      name = Gem::Version.new(Magick::IMAGEMAGICK_VERSION) > Gem::Version.new('7.0.0') ? 'alpha ' : 'matte '
+      primitive name + sprintf('%g,%g, %s', x, y, PAINT_METHOD_NAMES[method.to_i])
     end
 
     # Draw an arc.
     def arc(start_x, start_y, end_x, end_y, start_degrees, end_degrees)
-      primitive 'arc ' + format('%g,%g %g,%g %g,%g',
-                                start_x, start_y, end_x, end_y, start_degrees, end_degrees)
+      primitive 'arc ' + sprintf(
+        '%g,%g %g,%g %g,%g',
+        start_x, start_y, end_x, end_y, start_degrees, end_degrees
+      )
     end
 
     # Draw a bezier curve.
@@ -255,12 +271,12 @@ module Magick
       elsif points.length.odd?
         Kernel.raise ArgumentError, 'odd number of arguments specified'
       end
-      primitive 'bezier ' + points.map! { |x| format('%g', x) }.join(',')
+      primitive 'bezier ' + points.map! { |x| sprintf('%g', x) }.join(',')
     end
 
     # Draw a circle
     def circle(origin_x, origin_y, perim_x, perim_y)
-      primitive 'circle ' + format('%g,%g %g,%g', origin_x, origin_y, perim_x, perim_y)
+      primitive 'circle ' + sprintf('%g,%g %g,%g', origin_x, origin_y, perim_x, perim_y)
     end
 
     # Invoke a clip-path defined by def_clip_path.
@@ -284,7 +300,7 @@ module Magick
     # point, replace, floodfill, filltoborder,reset
     def color(x, y, method)
       Kernel.raise ArgumentError, "Unknown PaintMethod: #{method}" unless PAINT_METHOD_NAMES.key?(method.to_i)
-      primitive 'color ' + format('%g,%g,%s', x, y, PAINT_METHOD_NAMES[method.to_i])
+      primitive 'color ' + sprintf('%g,%g,%s', x, y, PAINT_METHOD_NAMES[method.to_i])
     end
 
     # Specify EITHER the text decoration (none, underline, overline,
@@ -315,8 +331,10 @@ module Magick
 
     # Draw an ellipse
     def ellipse(origin_x, origin_y, width, height, arc_start, arc_end)
-      primitive 'ellipse ' + format('%g,%g %g,%g %g,%g',
-                                    origin_x, origin_y, width, height, arc_start, arc_end)
+      primitive 'ellipse ' + sprintf(
+        '%g,%g %g,%g %g,%g',
+        origin_x, origin_y, width, height, arc_start, arc_end
+      )
     end
 
     # Let anything through, but the only defined argument
@@ -382,7 +400,7 @@ module Magick
     def image(composite, x, y, width, height, image_file_path)
       Kernel.raise ArgumentError, 'Unknown composite' unless composite.is_a?(CompositeOperator)
       composite_name = composite.to_s.sub!('CompositeOp', '')
-      primitive 'image ' + format('%s %g,%g %g,%g %s', composite_name, x, y, width, height, enquote(image_file_path))
+      primitive 'image ' + sprintf('%s %g,%g %g,%g %s', composite_name, x, y, width, height, enquote(image_file_path))
     end
 
     # IM 6.5.5-8 and later
@@ -423,7 +441,7 @@ module Magick
 
     # Draw a line
     def line(start_x, start_y, end_x, end_y)
-      primitive 'line ' + format('%g,%g %g,%g', start_x, start_y, end_x, end_y)
+      primitive 'line ' + sprintf('%g,%g %g,%g', start_x, start_y, end_x, end_y)
     end
 
     # Specify drawing fill and stroke opacities. If the value is a string
@@ -445,7 +463,7 @@ module Magick
     # as the argument to the 'fill' or 'stroke' methods
     def pattern(name, x, y, width, height)
       push('defs')
-      push("pattern #{name} " + format('%g %g %g %g', x, y, width, height))
+      push("pattern #{name} " + sprintf('%g %g %g %g', x, y, width, height))
       push('graphic-context')
       yield
     ensure
@@ -456,13 +474,13 @@ module Magick
 
     # Set point to fill color.
     def point(x, y)
-      primitive 'point ' + format('%g,%g', x, y)
+      primitive 'point ' + sprintf('%g,%g', x, y)
     end
 
     # Specify the font size in points. Yes, the primitive is "font-size" but
     # in other places this value is called the "pointsize". Give it both names.
     def pointsize(points)
-      primitive 'font-size ' + format('%g', points)
+      primitive 'font-size ' + sprintf('%g', points)
     end
     alias font_size pointsize
 
@@ -473,7 +491,7 @@ module Magick
       elsif points.length.odd?
         Kernel.raise ArgumentError, 'odd number of points specified'
       end
-      primitive 'polygon ' + points.map! { |x| format('%g', x) }.join(',')
+      primitive 'polygon ' + points.map! { |x| sprintf('%g', x) }.join(',')
     end
 
     # Draw a polyline
@@ -483,7 +501,7 @@ module Magick
       elsif points.length.odd?
         Kernel.raise ArgumentError, 'odd number of points specified'
       end
-      primitive 'polyline ' + points.map! { |x| format('%g', x) }.join(',')
+      primitive 'polyline ' + points.map! { |x| sprintf('%g', x) }.join(',')
     end
 
     # Return to the previously-saved set of whatever
@@ -517,32 +535,36 @@ module Magick
 
     # Draw a rectangle
     def rectangle(upper_left_x, upper_left_y, lower_right_x, lower_right_y)
-      primitive 'rectangle ' + format('%g,%g %g,%g',
-                                      upper_left_x, upper_left_y, lower_right_x, lower_right_y)
+      primitive 'rectangle ' + sprintf(
+        '%g,%g %g,%g',
+        upper_left_x, upper_left_y, lower_right_x, lower_right_y
+      )
     end
 
     # Specify coordinate space rotation. "angle" is measured in degrees
     def rotate(angle)
-      primitive 'rotate ' + format('%g', angle)
+      primitive 'rotate ' + sprintf('%g', angle)
     end
 
     # Draw a rectangle with rounded corners
     def roundrectangle(center_x, center_y, width, height, corner_width, corner_height)
-      primitive 'roundrectangle ' + format('%g,%g,%g,%g,%g,%g',
-                                           center_x, center_y, width, height, corner_width, corner_height)
+      primitive 'roundrectangle ' + sprintf(
+        '%g,%g,%g,%g,%g,%g',
+        center_x, center_y, width, height, corner_width, corner_height
+      )
     end
 
     # Specify scaling to be applied to coordinate space on subsequent drawing commands.
     def scale(x, y)
-      primitive 'scale ' + format('%g,%g', x, y)
+      primitive 'scale ' + sprintf('%g,%g', x, y)
     end
 
     def skewx(angle)
-      primitive 'skewX ' + format('%g', angle)
+      primitive 'skewX ' + sprintf('%g', angle)
     end
 
     def skewy(angle)
-      primitive 'skewY ' + format('%g', angle)
+      primitive 'skewY ' + sprintf('%g', angle)
     end
 
     # Specify the object stroke, a color name or pattern name.
@@ -572,7 +594,7 @@ module Magick
 
     # Specify the initial offset in the dash pattern
     def stroke_dashoffset(value = 0)
-      primitive 'stroke-dashoffset ' + format('%g', value)
+      primitive 'stroke-dashoffset ' + sprintf('%g', value)
     end
 
     def stroke_linecap(value)
@@ -599,7 +621,7 @@ module Magick
 
     # Specify stroke (outline) width in pixels.
     def stroke_width(pixels)
-      primitive 'stroke-width ' + format('%g', pixels)
+      primitive 'stroke-width ' + sprintf('%g', pixels)
     end
 
     # Draw text at position x,y. Add quotes to text that is not already quoted.
@@ -617,7 +639,7 @@ module Magick
         # escape existing braces, surround with braces
         text = '{' + text.gsub(/[}]/) { |b| '\\' + b } + '}'
       end
-      primitive 'text ' + format('%g,%g %s', x, y, text)
+      primitive 'text ' + sprintf('%g,%g %s', x, y, text)
     end
 
     # Specify text alignment relative to a given point
@@ -646,7 +668,7 @@ module Magick
     # Specify center of coordinate space to use for subsequent drawing
     # commands.
     def translate(x, y)
-      primitive 'translate ' + format('%g,%g', x, y)
+      primitive 'translate ' + sprintf('%g,%g', x, y)
     end
   end # class Magick::Draw
 
@@ -749,12 +771,6 @@ module Magick
     module Post_ObjectData_Descriptor
       Confirmed_ObjectData_Size              = '9:10'
     end
-
-    # Make all constants above immutable
-    constants.each do |record|
-      rec = const_get(record)
-      rec.constants.each { |ds| rec.const_get(ds).freeze }
-    end
   end # module Magick::IPTC
 
   # Ruby-level Magick::Image methods
@@ -854,7 +870,7 @@ module Magick
       else
         get_exif_by_number # ensure properties is populated with exif data
         tag.each do |num|
-          rval = self[format('#%04X', num.to_i)]
+          rval = self[sprintf('#%04X', num.to_i)]
           hash[num] = rval == 'unknown' ? nil : rval
         end
       end
@@ -1062,7 +1078,7 @@ module Magick
 
         # Define a getter and a setter for each channel.
         %i[red green blue opacity].each do |c|
-          module_eval <<-END_EVAL
+          module_eval <<-END_EVAL, __FILE__, __LINE__ + 1
             def #{c}
                 return collect { |p| p.#{c} }
             end
@@ -1118,7 +1134,6 @@ module Magick
           each { |x| @view[x] = rv.dup }
           changed
           notify_observers(self)
-          nil
         end
 
         # A pixel has been modified. Tell the view.
@@ -1241,16 +1256,14 @@ module Magick
 
     protected
 
-    def is_an_image(obj)
+    def assert_image(obj)
       Kernel.raise ArgumentError, "Magick::Image required (#{obj.class} given)" unless obj.is_a? Magick::Image
-      true
     end
 
     # Ensure array is always an array of Magick::Image objects
-    def is_an_image_array(ary)
+    def assert_image_array(ary)
       Kernel.raise ArgumentError, "Magick::ImageList or array of Magick::Images required (#{ary.class} given)" unless ary.respond_to? :each
-      ary.each { |obj| is_an_image obj }
-      true
+      ary.each { |obj| assert_image obj }
     end
 
     # Find old current image, update scene number
@@ -1282,7 +1295,7 @@ module Magick
       if n.nil?
         Kernel.raise IndexError, 'scene number out of bounds' unless @images.length.zero?
         @scene = nil
-        return @scene
+        return
       elsif @images.length.zero?
         Kernel.raise IndexError, 'scene number out of bounds'
       end
@@ -1290,13 +1303,12 @@ module Magick
       n = Integer(n)
       Kernel.raise IndexError, 'scene number out of bounds' if n < 0 || n > length - 1
       @scene = n
-      @scene
     end
 
     # All the binary operators work the same way.
     # 'other' should be either an ImageList or an Array
     %w[& + - |].each do |op|
-      module_eval <<-END_BINOPS
+      module_eval <<-END_BINOPS, __FILE__, __LINE__ + 1
         def #{op}(other)
           ilist = self.class.new
           begin
@@ -1306,7 +1318,7 @@ module Magick
           end
           current = get_current()
           a.each do |image|
-            is_an_image image
+            assert_image image
             ilist << image
           end
           ilist.set_current current
@@ -1325,7 +1337,7 @@ module Magick
     end
 
     def <<(obj)
-      is_an_image obj
+      assert_image obj
       @images << obj
       @scene = @images.length - 1
       self
@@ -1366,20 +1378,21 @@ module Magick
     def []=(*args)
       obj = @images.[]=(*args)
       if obj && obj.respond_to?(:each)
-        is_an_image_array(obj)
+        assert_image_array(obj)
         set_current obj.last.__id__
       elsif obj
-        is_an_image(obj)
+        assert_image(obj)
         set_current obj.__id__
       else
         set_current nil
       end
-      obj
     end
 
-    %i[at each each_index empty? fetch
-       first hash include? index length rindex sort!].each do |mth|
-      module_eval <<-END_SIMPLE_DELEGATES
+    %i[
+      at each each_index empty? fetch
+      first hash include? index length rindex sort!
+    ].each do |mth|
+      module_eval <<-END_SIMPLE_DELEGATES, __FILE__, __LINE__ + 1
         def #{mth}(*args, &block)
           @images.#{mth}(*args, &block)
         end
@@ -1401,7 +1414,7 @@ module Magick
     # override Enumerable#collect
     def collect(&block)
       current = get_current
-      a = @images.collect(&block)
+      a = @images.map(&block)
       ilist = self.class.new
       a.each { |image| ilist << image }
       ilist.set_current current
@@ -1409,8 +1422,8 @@ module Magick
     end
 
     def collect!(&block)
-      @images.collect!(&block)
-      is_an_image_array @images
+      @images.map!(&block)
+      assert_image_array @images
       self
     end
 
@@ -1419,7 +1432,6 @@ module Magick
       ditto = self.class.new
       @images.each { |f| ditto << f.copy }
       ditto.scene = @scene
-      ditto.taint if tainted?
       ditto
     end
 
@@ -1430,6 +1442,7 @@ module Magick
     end
 
     # ImageList#map took over the "map" name. Use alternatives.
+    alias map collect
     alias __map__ collect
     alias map! collect!
     alias __map__! collect!
@@ -1454,7 +1467,7 @@ module Magick
     end
 
     def concat(other)
-      is_an_image_array other
+      assert_image_array other
       other.each { |image| @images << image }
       @scene = length - 1
       self
@@ -1468,7 +1481,7 @@ module Magick
     end
 
     def delete(obj, &block)
-      is_an_image obj
+      assert_image obj
       current = get_current
       a = @images.delete(obj, &block)
       set_current current
@@ -1493,12 +1506,11 @@ module Magick
       ditto = self.class.new
       @images.each { |img| ditto << img }
       ditto.scene = @scene
-      ditto.taint if tainted?
       ditto
     end
 
     def eql?(other)
-      is_an_image_array other
+      assert_image_array other
       eql = other.eql?(@images)
       begin # "other" is another ImageList
         eql &&= @scene == other.scene
@@ -1509,10 +1521,10 @@ module Magick
     end
 
     def fill(*args, &block)
-      is_an_image args[0] unless block_given?
+      assert_image args[0] unless block_given?
       current = get_current
       @images.fill(*args, &block)
-      is_an_image_array self
+      assert_image_array self
       set_current current
       self
     end
@@ -1520,7 +1532,7 @@ module Magick
     # Override Enumerable's find_all
     def find_all(&block)
       current = get_current
-      a = @images.find_all(&block)
+      a = @images.select(&block)
       ilist = self.class.new
       a.each { |image| ilist << image }
       ilist.set_current current
@@ -1544,14 +1556,12 @@ module Magick
       filenames.each do |f|
         Magick::Image.read(f, &block).each { |n| @images << n }
       end
-      if length > 0
-        @scene = length - 1 # last image in array
-      end
-      self
+
+      @scene = length - 1 if length > 0 # last image in array
     end
 
     def insert(index, *args)
-      args.each { |image| is_an_image image }
+      args.each { |image| assert_image image }
       current = get_current
       @images.insert(index, *args)
       set_current current
@@ -1570,7 +1580,6 @@ module Magick
       n = Integer(n)
       Kernel.raise ArgumentError, 'iterations must be between 0 and 65535' if n < 0 || n > 65_535
       @images.each { |f| f.iterations = n }
-      self
     end
 
     def last(*args)
@@ -1651,7 +1660,7 @@ module Magick
 
     def push(*objs)
       objs.each do |image|
-        is_an_image image
+        assert_image image
         @images << image
       end
       @scene = length - 1
@@ -1687,7 +1696,7 @@ module Magick
     end
 
     def replace(other)
-      is_an_image_array other
+      assert_image_array other
       current = get_current
       @images.clear
       other.each { |image| @images << image }
@@ -1785,7 +1794,7 @@ module Magick
 
     # @scene -> new object
     def unshift(obj)
-      is_an_image obj
+      assert_image obj
       @images.unshift(obj)
       @scene = 0
       self
@@ -1802,6 +1811,11 @@ module Magick
     alias indices values_at
   end # Magick::ImageList
 
+  class Pixel
+    # include Observable for Image::View class
+    include Observable
+  end
+
   #  Collects non-specific optional method arguments
   class OptionalMethodArguments
     def initialize(img)

data/lib/rvg/clippath.rb

@@ -30,7 +30,8 @@ module Magick
         yield(self) if block_given?
       end
 
-      def add_primitives(gc, _style) #:nodoc:
+      # @private
+      def add_primitives(gc, _style)
         name = __id__.to_s
         gc.define_clip_path(name) do
           gc.clip_units(@clip_path_units)

data/lib/rvg/container.rb

@@ -6,7 +6,8 @@ module Magick
   class RVG
     # Content is simply an Array with a deep_copy method.
     # When unit-testing, it also has a deep_equal method.
-    class Content < Array #:nodoc:
+    # @private
+    class Content < Array
       def deep_copy(h = {})
         me = __id__
         copy = h[me]
@@ -64,7 +65,8 @@ module Magick
         yield(self) if block_given?
       end
 
-      def add_primitives(gc) #:nodoc:
+      # @private
+      def add_primitives(gc)
         gc.push
         add_transform_primitives(gc)
         add_style_primitives(gc)
@@ -73,13 +75,15 @@ module Magick
       end
 
       # Translate container according to #use arguments
-      def ref(x, y, _width, _height) #:nodoc:
+      # @private
+      def ref(x, y, _width, _height)
         translate(x, y) if x != 0 || y != 0
       end
 
       # Append an arbitrary object to the group's content. Called
       # by #use to insert a non-container object into a group.
-      def <<(obj) #:nodoc:
+      # @private
+      def <<(obj)
         @content << obj
       end
     end # class Group
@@ -113,7 +117,8 @@ module Magick
         @element.ref(x, y, width, height)
       end
 
-      def add_primitives(gc) #:nodoc:
+      # @private
+      def add_primitives(gc)
         gc.push
         add_transform_primitives(gc)
         add_style_primitives(gc)

data/lib/rvg/describable.rb

@@ -11,7 +11,7 @@ module Magick
     module Describable
       private
 
-      def initialize(*args, &block) #:nodoc:
+      def initialize(*args, &block)
         super
         @title, @desc, @metadata = nil
       end