prawn 2.1.0 → 2.3.0

data/lib/prawn/{font → fonts}/dfont.rb

@@ -1,15 +1,9 @@
-# encoding: utf-8
-#
-# font.rb : The Prawn font class
-#
-# Copyright November 2008, Jamis Buck. All Rights Reserved.
-#
-# This is free software. Please see the LICENSE and COPYING files for details.
-#
+# frozen_string_literal: true
+
 require_relative 'ttf'
 
 module Prawn
-  class Font
+  module Fonts
     # @private
     class DFont < TTF
       # Returns a list of the names of all named fonts in the given dfont file.
@@ -20,7 +14,7 @@ module Prawn
       #
       def self.named_fonts(file)
         TTFunk::ResourceFile.open(file) do |f|
-          return f.resources_for("sfnt")
+          return f.resources_for('sfnt')
         end
       end
 
@@ -28,7 +22,7 @@ module Prawn
       #
       def self.font_count(file)
         TTFunk::ResourceFile.open(file) do |f|
-          return f.map["sfnt"][:list].length
+          return f.map['sfnt'][:list].length
         end
       end
 

data/lib/prawn/fonts/otf.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+require_relative 'ttf'
+
+module Prawn
+  module Fonts
+    # @private
+    class OTF < TTF
+    end
+  end
+end

data/lib/prawn/fonts/ttc.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+require_relative 'ttf'
+
+module Prawn
+  module Fonts
+    # @private
+    class TTC < TTF
+      # Returns a list of the names of all named fonts in the given ttc file.
+      # They are returned in order of their appearance in the file.
+      #
+      def self.font_names(file)
+        TTFunk::Collection.open(file) do |ttc|
+          ttc.map { |font| font.name.font_name.first }
+        end
+      end
+
+      private
+
+      def read_ttf_file
+        TTFunk::File.from_ttc(
+          @name,
+          font_option_to_index(@name, @options[:font])
+        )
+      end
+
+      def font_option_to_index(file, option)
+        if option.is_a?(Numeric)
+          option
+        else
+          self.class.font_names(file).index { |n| n == option } || 0
+        end
+      end
+    end
+  end
+end

data/lib/prawn/{font → fonts}/ttf.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 # prawn/font/ttf.rb : Implements AFM font support for Prawn
 #
@@ -11,7 +11,7 @@ require 'ttfunk'
 require 'ttfunk/subset_collection'
 
 module Prawn
-  class Font
+  module Fonts
     # @private
     class TTF < Font
       attr_reader :ttf, :subsets
@@ -23,17 +23,18 @@ module Prawn
       def initialize(document, name, options = {})
         super
 
-        @ttf              = read_ttf_file
-        @subsets          = TTFunk::SubsetCollection.new(@ttf)
+        @ttf = read_ttf_file
+        @subsets = TTFunk::SubsetCollection.new(@ttf)
+        @italic_angle = nil
 
-        @attributes       = {}
-        @bounding_boxes   = {}
-        @char_widths      = {}
+        @attributes = {}
+        @bounding_boxes = {}
+        @char_widths = {}
         @has_kerning_data = @ttf.kerning.exists? && @ttf.kerning.tables.any?
 
-        @ascender         = Integer(@ttf.ascent * scale_factor)
-        @descender        = Integer(@ttf.descent * scale_factor)
-        @line_gap         = Integer(@ttf.line_gap * scale_factor)
+        @ascender = Integer(@ttf.ascent * scale_factor)
+        @descender = Integer(@ttf.descent * scale_factor)
+        @line_gap = Integer(@ttf.line_gap * scale_factor)
       end
 
       # NOTE: +string+ must be UTF8-encoded.
@@ -44,7 +45,7 @@ module Prawn
             if r.is_a?(Numeric)
               s - r
             else
-              r.inject(s) { |s2, u| s2 + character_width_by_code(u) }
+              r.inject(s) { |a, e| a + character_width_by_code(e) }
             end
           end * scale
         else
@@ -61,9 +62,12 @@ module Prawn
       end
 
       # Returns true if the font has kerning data, false otherwise
+      #
+      # rubocop: disable Naming/PredicateName
       def has_kerning_data?
         @has_kerning_data
       end
+      # rubocop: enable Naming/PredicateName
 
       # Perform any changes to the string that need to happen
       # before it is rendered to the canvas. Returns an array of
@@ -80,7 +84,9 @@ module Prawn
           last_subset = nil
           kern(text).inject([]) do |result, element|
             if element.is_a?(Numeric)
-              result.last[1] = [result.last[1]] unless result.last[1].is_a?(Array)
+              unless result.last[1].is_a?(Array)
+                result.last[1] = [result.last[1]]
+              end
               result.last[1] << element
               result
             else
@@ -100,7 +106,7 @@ module Prawn
             end
           end
         else
-          @subsets.encode(text.unpack("U*"))
+          @subsets.encode(text.unpack('U*'))
         end
       end
 
@@ -109,7 +115,7 @@ module Prawn
       end
 
       # not sure how to compute this for true-type fonts...
-      def stemV
+      def stem_v
         0
       end
 
@@ -118,7 +124,8 @@ module Prawn
 
         if @ttf.postscript.exists?
           raw = @ttf.postscript.italic_angle
-          hi, low = raw >> 16, raw & 0xFF
+          hi = raw >> 16
+          low = raw & 0xFF
           hi = -((hi ^ 0xFFFF) + 1) if hi & 0x8000 != 0
           @italic_angle = "#{hi}.#{low}".to_f
         else
@@ -131,7 +138,7 @@ module Prawn
       def cap_height
         @cap_height ||= begin
           height = @ttf.os2.exists? && @ttf.os2.cap_height || 0
-          height == 0 ? @ascender : height
+          height.zero? ? @ascender : height
         end
       end
 
@@ -154,35 +161,34 @@ module Prawn
       end
 
       def pdf_flags
-        @flags ||= begin
+        @pdf_flags ||= begin
           flags = 0
           flags |= 0x0001 if @ttf.postscript.fixed_pitch?
           flags |= 0x0002 if serif?
           flags |= 0x0008 if script?
           flags |= 0x0040 if italic_angle != 0
-          flags |= 0x0004 # assume the font contains at least some non-latin characters
+          # Assume the font contains at least some non-latin characters
+          flags | 0x0004
         end
       end
 
       def normalize_encoding(text)
-        begin
-          text.encode(::Encoding::UTF_8)
-        rescue => e
-          puts e
-          raise Prawn::Errors::IncompatibleStringEncoding, "Encoding " \
-            "#{text.encoding} can not be transparently converted to UTF-8. " \
-            "Please ensure the encoding of the string you are attempting " \
-            "to use is set correctly"
-        end
+        text.encode(::Encoding::UTF_8)
+      rescue StandardError => e
+        puts e
+        raise Prawn::Errors::IncompatibleStringEncoding, 'Encoding ' \
+          "#{text.encoding} can not be transparently converted to UTF-8. " \
+          'Please ensure the encoding of the string you are attempting ' \
+          'to use is set correctly'
       end
 
       def to_utf8(text)
-        text.encode("UTF-8")
+        text.encode('UTF-8')
       end
 
       def glyph_present?(char)
         code = char.codepoints.first
-        cmap[code] > 0
+        cmap[code].positive?
       end
 
       # Returns the number of characters in +str+ (a UTF-8-encoded string).
@@ -194,7 +200,7 @@ module Prawn
       private
 
       def cmap
-        @cmap ||= @ttf.cmap.unicode.first or fail("no unicode cmap for font")
+        (@cmap ||= @ttf.cmap.unicode.first) || raise('no unicode cmap for font')
       end
 
       # +string+ must be UTF8-encoded.
@@ -220,12 +226,12 @@ module Prawn
       end
 
       def kern_pairs_table
-        @kerning_data ||= has_kerning_data? ? @ttf.kerning.tables.first.pairs : {}
-      end
-
-      def cid_to_gid_map
-        max = cmap.code_map.keys.max
-        (0..max).map { |cid| cmap[cid] }.pack("n*")
+        @kern_pairs_table ||=
+          if has_kerning_data?
+            @ttf.kerning.tables.first.pairs
+          else
+            {}
+          end
       end
 
       def hmtx
@@ -243,16 +249,16 @@ module Prawn
       end
 
       def scale_factor
-        @scale ||= 1000.0 / @ttf.header.units_per_em
+        @scale_factor ||= 1000.0 / @ttf.header.units_per_em
       end
 
       def register(subset)
-        temp_name = @ttf.name.postscript_name.gsub("\0", "").to_sym
-        ref = @document.ref!(:Type => :Font, :BaseFont => temp_name)
+        temp_name = @ttf.name.postscript_name.delete("\0").to_sym
+        ref = @document.ref!(Type: :Font, BaseFont: temp_name)
 
         # Embed the font metrics in the document after everything has been
         # drawn, just before the document is emitted.
-        @document.renderer.before_render { |doc| embed(ref, subset) }
+        @document.renderer.before_render { |_doc| embed(ref, subset) }
 
         ref
       end
@@ -267,29 +273,32 @@ module Prawn
 
         # empirically, it looks like Adobe Reader will not display fonts
         # if their font name is more than 33 bytes long. Strange. But true.
-        basename = font.name.postscript_name[0, 33].gsub("\0", "")
+        basename = font.name.postscript_name[0, 33].delete("\0")
 
-        fail "Can't detect a postscript name for #{file}" if basename.nil?
+        raise "Can't detect a postscript name for #{file}" if basename.nil?
 
-        fontfile = @document.ref!(:Length1 => font_content.size)
+        fontfile = @document.ref!(Length1: font_content.size)
         fontfile.stream << font_content
         fontfile.stream.compress!
 
-        descriptor = @document.ref!(:Type        => :FontDescriptor,
-                                    :FontName    => basename.to_sym,
-                                    :FontFile2   => fontfile,
-                                    :FontBBox    => bbox,
-                                    :Flags       => pdf_flags,
-                                    :StemV       => stemV,
-                                    :ItalicAngle => italic_angle,
-                                    :Ascent      => @ascender,
-                                    :Descent     => @descender,
-                                    :CapHeight   => cap_height,
-                                    :XHeight     => x_height)
+        descriptor = @document.ref!(
+          Type: :FontDescriptor,
+          FontName: basename.to_sym,
+          FontFile2: fontfile,
+          FontBBox: bbox,
+          Flags: pdf_flags,
+          StemV: stem_v,
+          ItalicAngle: italic_angle,
+          Ascent: @ascender,
+          Descent: @descender,
+          CapHeight: cap_height,
+          XHeight: x_height
+        )
 
         hmtx = font.horizontal_metrics
-        widths = font.cmap.tables.first.code_map.map { |gid|
-          Integer(hmtx.widths[gid] * scale_factor) }[32..-1]
+        widths = font.cmap.tables.first.code_map.map do |gid|
+          Integer(hmtx.widths[gid] * scale_factor)
+        end[32..-1]
 
         # It would be nice to have Encoding set for the macroman subsets,
         # and only do a ToUnicode cmap for non-encoded unicode subsets.
@@ -304,14 +313,22 @@ module Prawn
         map = @subsets[subset].to_unicode_map
 
         ranges = [[]]
-        map.keys.sort.inject("") do |s, code|
+        map.keys.sort.inject('') do |_s, code|
           ranges << [] if ranges.last.length >= 100
           unicode = map[code]
-          ranges.last << "<%02x><%04x>" % [code, unicode]
+          ranges.last << format(
+            '<%<code>02x><%<unicode>04x>',
+            code: code,
+            unicode: unicode
+          )
         end
 
-        range_blocks = ranges.inject("") do |s, list|
-          s << "%d beginbfchar\n%s\nendbfchar\n" % [list.length, list.join("\n")]
+        range_blocks = ranges.inject(+'') do |s, list|
+          s << format(
+            "%<lenght>d beginbfchar\n%<list>s\nendbfchar\n",
+            lenght: list.length,
+            list: list.join("\n")
+          )
         end
 
         to_unicode_cmap = UNICODE_CMAP_TEMPLATE % range_blocks.strip
@@ -320,16 +337,18 @@ module Prawn
         cmap << to_unicode_cmap
         cmap.stream.compress!
 
-        reference.data.update(:Subtype => :TrueType,
-                              :BaseFont => basename.to_sym,
-                              :FontDescriptor => descriptor,
-                              :FirstChar => 32,
-                              :LastChar => 255,
-                              :Widths => @document.ref!(widths),
-                              :ToUnicode => cmap)
+        reference.data.update(
+          Subtype: :TrueType,
+          BaseFont: basename.to_sym,
+          FontDescriptor: descriptor,
+          FirstChar: 32,
+          LastChar: 255,
+          Widths: @document.ref!(widths),
+          ToUnicode: cmap
+        )
       end
 
-      UNICODE_CMAP_TEMPLATE = <<-STR.strip.gsub(/^\s*/, "")
+      UNICODE_CMAP_TEMPLATE = <<-STR.strip.gsub(/^\s*/, '')
         /CIDInit /ProcSet findresource begin
         12 dict begin
         begincmap

data/lib/prawn/graphics.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 # graphics.rb : Implements PDF drawing primitives
 #
@@ -6,14 +6,14 @@
 #
 # This is free software. Please see the LICENSE and COPYING files for details.
 
-require_relative "graphics/blend_mode"
-require_relative "graphics/color"
-require_relative "graphics/dash"
-require_relative "graphics/cap_style"
-require_relative "graphics/join_style"
-require_relative "graphics/transparency"
-require_relative "graphics/transformation"
-require_relative "graphics/patterns"
+require_relative 'graphics/blend_mode'
+require_relative 'graphics/color'
+require_relative 'graphics/dash'
+require_relative 'graphics/cap_style'
+require_relative 'graphics/join_style'
+require_relative 'graphics/transparency'
+require_relative 'graphics/transformation'
+require_relative 'graphics/patterns'
 
 module Prawn
   # Implements the drawing facilities for Prawn::Document.
@@ -66,9 +66,11 @@ module Prawn
     #   pdf.curve_to [100,100], :bounds => [[90,90],[75,75]]
     #
     def curve_to(dest, options = {})
-      options[:bounds] or fail Prawn::Errors::InvalidGraphicsPath,
-                               "Bounding points for bezier curve must be specified " \
-                               "as :bounds => [[x1,y1],[x2,y2]]"
+      options[:bounds] || raise(
+        Prawn::Errors::InvalidGraphicsPath,
+        'Bounding points for bezier curve must be specified ' \
+        'as :bounds => [[x1,y1],[x2,y2]]'
+      )
 
       curve_points = PDF::Core.real_params(
         (options[:bounds] << dest).flat_map { |e| map_to_absolute(e) }
@@ -97,7 +99,9 @@ module Prawn
     #
     def rounded_rectangle(point, width, height, radius)
       x, y = point
-      rounded_polygon(radius, point, [x + width, y], [x + width, y - height], [x, y - height])
+      rounded_polygon(
+        radius, point, [x + width, y], [x + width, y - height], [x, y - height]
+      )
     end
 
     ###########################################################
@@ -146,11 +150,7 @@ module Prawn
     #  horizontal_line 25, 100, :at => 75
     #
     def horizontal_line(x1, x2, options = {})
-      if options[:at]
-        y1 = options[:at]
-      else
-        y1 = y - bounds.absolute_bottom
-      end
+      y1 = options[:at] || y - bounds.absolute_bottom
 
       line(x1, y1, x2, y1)
     end
@@ -186,9 +186,9 @@ module Prawn
     #
     KAPPA = 4.0 * ((Math.sqrt(2) - 1.0) / 3.0)
 
-    # Draws a circle of radius <tt>radius</tt> with the centre-point at <tt>point</tt>
-    # as a complete subpath. The drawing point will be moved to the
-    # centre-point upon completion of the drawing the circle.
+    # Draws a circle of radius <tt>radius</tt> with the centre-point at
+    # <tt>point</tt> as a complete subpath. The drawing point will be moved to
+    # the centre-point upon completion of the drawing the circle.
     #
     #    pdf.circle [100,100], 25
     #
@@ -196,36 +196,36 @@ module Prawn
       ellipse(center, radius, radius)
     end
 
-    # Draws an ellipse of +x+ radius <tt>r1</tt> and +y+ radius <tt>r2</tt>
-    # with the centre-point at <tt>point</tt> as a complete subpath. The
-    # drawing point will be moved to the centre-point upon completion of the
-    # drawing the ellipse.
+    # Draws an ellipse of +x+ radius <tt>radius1</tt> and +y+ radius
+    # <tt>radius2</tt> with the centre-point at <tt>point</tt> as a complete
+    # subpath. The drawing point will be moved to the centre-point upon
+    # completion of the drawing the ellipse.
     #
     #    # draws an ellipse with x-radius 25 and y-radius 50
     #    pdf.ellipse [100,100], 25, 50
     #
-    def ellipse(point, r1, r2 = r1)
+    def ellipse(point, radius1, radius2 = radius1)
       x, y = point
-      l1 = r1 * KAPPA
-      l2 = r2 * KAPPA
+      l1 = radius1 * KAPPA
+      l2 = radius2 * KAPPA
 
-      move_to(x + r1, y)
+      move_to(x + radius1, y)
 
       # Upper right hand corner
-      curve_to [x,  y + r2],
-               :bounds => [[x + r1, y + l2], [x + l1, y + r2]]
+      curve_to [x,  y + radius2],
+        bounds: [[x + radius1, y + l2], [x + l1, y + radius2]]
 
       # Upper left hand corner
-      curve_to [x - r1, y],
-               :bounds => [[x - l1, y + r2], [x - r1, y + l2]]
+      curve_to [x - radius1, y],
+        bounds: [[x - l1, y + radius2], [x - radius1, y + l2]]
 
       # Lower left hand corner
-      curve_to [x, y - r2],
-               :bounds => [[x - r1, y - l2], [x - l1, y - r2]]
+      curve_to [x, y - radius2],
+        bounds: [[x - radius1, y - l2], [x - l1, y - radius2]]
 
       # Lower right hand corner
-      curve_to [x + r1, y],
-               :bounds => [[x + l1, y - r2], [x + r1, y - l2]]
+      curve_to [x + radius1, y],
+        bounds: [[x + l1, y - radius2], [x + radius1, y - l2]]
 
       move_to(x, y)
     end
@@ -241,35 +241,47 @@ module Prawn
         line_to(*point)
       end
       # close the path
-      renderer.add_content "h"
+      renderer.add_content 'h'
     end
 
-    # Draws a rounded polygon from specified points using the radius to define bezier curves
+    # Draws a rounded polygon from specified points using the radius to define
+    # bezier curves
     #
-    #  # draws a rounded filled in polygon
-    #   pdf.fill_and_stroke_rounded_polygon(10, [100, 250], [200, 300], [300, 250],
-    #                 [300, 150], [200, 100], [100, 150])
+    #   # draws a rounded filled in polygon
+    #   pdf.fill_and_stroke_rounded_polygon(
+    #     10, [100, 250], [200, 300], [300, 250], [300, 150], [200, 100],
+    #     [100, 150]
+    #   )
     def rounded_polygon(radius, *points)
       move_to point_on_line(radius, points[1], points[0])
       sides = points.size
       points << points[0] << points[1]
-      (sides).times do |i|
+      sides.times do |i|
         rounded_vertex(radius, points[i], points[i + 1], points[i + 2])
       end
       # close the path
-      renderer.add_content "h"
+      renderer.add_content 'h'
     end
 
-    # Creates a rounded vertex for a line segment used for building a rounded polygon
-    # requires a radius to define bezier curve and three points. The first two points define
-    # the line segment and the third point helps define the curve for the vertex.
+    # Creates a rounded vertex for a line segment used for building a rounded
+    # polygon requires a radius to define bezier curve and three points. The
+    # first two points define the line segment and the third point helps define
+    # the curve for the vertex.
     def rounded_vertex(radius, *points)
-      radial_point_1 = point_on_line(radius, points[0], points[1])
-      bezier_point_1 = point_on_line((radius - radius * KAPPA), points[0], points[1])
-      radial_point_2 = point_on_line(radius, points[2], points[1])
-      bezier_point_2 = point_on_line((radius - radius * KAPPA), points[2], points[1])
-      line_to(radial_point_1)
-      curve_to(radial_point_2, :bounds => [bezier_point_1, bezier_point_2])
+      radial_point1 = point_on_line(radius, points[0], points[1])
+      bezier_point1 = point_on_line(
+        (radius - radius * KAPPA),
+        points[0],
+        points[1]
+      )
+      radial_point2 = point_on_line(radius, points[2], points[1])
+      bezier_point2 = point_on_line(
+        (radius - radius * KAPPA),
+        points[2],
+        points[1]
+      )
+      line_to(radial_point1)
+      curve_to(radial_point2, bounds: [bezier_point1, bezier_point2])
     end
 
     # Strokes the current path. If a block is provided, yields to the block
@@ -277,7 +289,7 @@ module Prawn
     #
     def stroke
       yield if block_given?
-      renderer.add_content "S"
+      renderer.add_content 'S'
     end
 
     # Closes and strokes the current path. If a block is provided, yields to
@@ -285,7 +297,7 @@ module Prawn
     #
     def close_and_stroke
       yield if block_given?
-      renderer.add_content "s"
+      renderer.add_content 's'
     end
 
     # Draws and strokes a rectangle represented by the current bounding box
@@ -320,38 +332,56 @@ module Prawn
     #
     def stroke_axis(options = {})
       options = {
-        :at => [0, 0],
-        :height => bounds.height.to_i - (options[:at] || [0, 0])[1],
-        :width => bounds.width.to_i - (options[:at] || [0, 0])[0],
-        :step_length => 100,
-        :negative_axes_length => 20,
-        :color => "000000"
+        at: [0, 0],
+        height: bounds.height.to_i - (options[:at] || [0, 0])[1],
+        width: bounds.width.to_i - (options[:at] || [0, 0])[0],
+        step_length: 100,
+        negative_axes_length: 20,
+        color: '000000'
       }.merge(options)
 
-      Prawn.verify_options([:at, :width, :height, :step_length,
-                            :negative_axes_length, :color], options)
+      Prawn.verify_options(
+        %i[
+          at width height step_length
+          negative_axes_length color
+        ], options
+      )
 
       save_graphics_state do
         fill_color(options[:color])
         stroke_color(options[:color])
 
-        dash(1, :space => 4)
-        stroke_horizontal_line(options[:at][0] - options[:negative_axes_length],
-                               options[:at][0] + options[:width], :at => options[:at][1])
-        stroke_vertical_line(options[:at][1] - options[:negative_axes_length],
-                             options[:at][1] + options[:height], :at => options[:at][0])
+        dash(1, space: 4)
+        stroke_horizontal_line(
+          options[:at][0] - options[:negative_axes_length],
+          options[:at][0] + options[:width], at: options[:at][1]
+        )
+        stroke_vertical_line(
+          options[:at][1] - options[:negative_axes_length],
+          options[:at][1] + options[:height], at: options[:at][0]
+        )
         undash
 
         fill_circle(options[:at], 1)
 
-        (options[:step_length]..options[:width]).step(options[:step_length]) do |point|
+        (options[:step_length]..options[:width])
+          .step(options[:step_length]) do |point|
           fill_circle([options[:at][0] + point, options[:at][1]], 1)
-          draw_text(point, :at => [options[:at][0] + point - 5, options[:at][1] - 10], :size => 7)
+          draw_text(
+            point,
+            at: [options[:at][0] + point - 5, options[:at][1] - 10],
+            size: 7
+          )
         end
 
-        (options[:step_length]..options[:height]).step(options[:step_length]) do |point|
+        (options[:step_length]..options[:height])
+          .step(options[:step_length]) do |point|
           fill_circle([options[:at][0], options[:at][1] + point], 1)
-          draw_text(point, :at => [options[:at][0] - 17, options[:at][1] + point - 2], :size => 7)
+          draw_text(
+            point,
+            at: [options[:at][0] - 17, options[:at][1] + point - 2],
+            size: 7
+          )
         end
       end
     end
@@ -365,7 +395,7 @@ module Prawn
     #
     def fill(options = {})
       yield if block_given?
-      renderer.add_content(options[:fill_rule] == :even_odd ? "f*" : "f")
+      renderer.add_content(options[:fill_rule] == :even_odd ? 'f*' : 'f')
     end
 
     # Closes, fills, and strokes the current path. If a block is provided,
@@ -379,13 +409,13 @@ module Prawn
     #
     def fill_and_stroke(options = {})
       yield if block_given?
-      renderer.add_content(options[:fill_rule] == :even_odd ? "b*" : "b")
+      renderer.add_content(options[:fill_rule] == :even_odd ? 'b*' : 'b')
     end
 
     # Closes the current path.
     #
     def close_path
-      renderer.add_content "h"
+      renderer.add_content 'h'
     end
 
     ##
@@ -594,12 +624,20 @@ module Prawn
     # :call-seq:
     #   fill_and_stroke_rounded_polygon(radius, *points)
 
-    ops    = %w{fill stroke fill_and_stroke}
-    shapes = %w{line_to curve_to rectangle rounded_rectangle line horizontal_line horizontal_rule vertical_line
-                curve circle_at circle ellipse_at ellipse polygon rounded_polygon rounded_vertex}
+    ops = %w[fill stroke fill_and_stroke]
+    shapes = %w[
+      line_to curve_to rectangle rounded_rectangle line horizontal_line
+      horizontal_rule vertical_line curve circle_at circle ellipse_at ellipse
+      polygon rounded_polygon rounded_vertex
+    ]
 
     ops.product(shapes).each do |operation, shape|
-      class_eval "def #{operation}_#{shape}(*args); #{shape}(*args); #{operation}; end"
+      class_eval <<-METHOD, __FILE__, __LINE__ + 1
+        def #{operation}_#{shape}(*args)
+          #{shape}(*args)
+          #{operation}
+        end
+      METHOD
     end
 
     private
@@ -629,8 +667,8 @@ module Prawn
       angle * Math::PI / 180
     end
 
-    # Returns the coordinates for a point on a line that is a given distance away from the second
-    # point defining the line segement
+    # Returns the coordinates for a point on a line that is a given distance
+    # away from the second point defining the line segement
     def point_on_line(distance_from_end, *points)
       x0, y0, x1, y1 = points.flatten
       length = Math.sqrt((x1 - x0)**2 + (y1 - y0)**2)