prawn-svg 0.34.2 → 0.35.0

data/lib/prawn/svg/elements/use.rb

@@ -9,7 +9,7 @@ class Prawn::SVG::Elements::Use < Prawn::SVG::Elements::Base
       raise SkipElementError, 'use tag has an href that is not a reference to an id; this is not supported'
     end
 
-    id = href[1..-1]
+    id = href[1..]
     referenced_element = @document.elements_by_id[id]
 
     if referenced_element
@@ -28,9 +28,7 @@ class Prawn::SVG::Elements::Use < Prawn::SVG::Elements::Base
 
     raise SkipElementError, "no tag with ID '#{id}' was found, referenced by use tag" if referenced_element_class.nil?
 
-    if referenced_element_source.name == 'symbol'
-      @referenced_element_class = Prawn::SVG::Elements::Viewport
-    end
+    @referenced_element_class = Prawn::SVG::Elements::Viewport if referenced_element_source.name == 'symbol'
 
     state.inside_use = true
 
@@ -45,9 +43,7 @@ class Prawn::SVG::Elements::Use < Prawn::SVG::Elements::Base
   end
 
   def apply
-    if @x || @y
-      add_call_and_enter 'translate', x_pixels(@x || 0), -y_pixels(@y || 0)
-    end
+    add_call_and_enter 'translate', x_pixels(@x || 0), -y_pixels(@y || 0) if @x || @y
   end
 
   def process_child_elements

data/lib/prawn/svg/elements/viewport.rb

@@ -12,9 +12,7 @@ class Prawn::SVG::Elements::Viewport < Prawn::SVG::Elements::Base
   end
 
   def apply
-    if @x != 0 || @y != 0
-      add_call 'transformation_matrix', 1, 0, 0, 1, @x, -@y
-    end
+    add_call 'transformation_matrix', 1, 0, 0, 1, @x, -@y if @x != 0 || @y != 0
 
     if overflow_hidden?
       add_call 'rectangle', [0, y(0)], @sizing.output_width, @sizing.output_height

data/lib/prawn/svg/elements.rb

@@ -1,42 +1,43 @@
 module Prawn::SVG::Elements
-  COMMA_WSP_REGEXP = /(?:\s+,?\s*|,\s*)/
+  COMMA_WSP_REGEXP = /(?:\s+,?\s*|,\s*)/.freeze
 end
 
 require 'prawn/svg/elements/call_duplicator'
 
-%w(base depth_first_base root container clip_path viewport text text_component line polyline polygon circle ellipse rect path use image gradient marker ignored).each do |filename|
+%w[base depth_first_base root container clip_path viewport text text_component line polyline polygon circle ellipse
+   rect path use image gradient marker ignored].each do |filename|
   require "prawn/svg/elements/#{filename}"
 end
 
 module Prawn::SVG::Elements
   TAG_CLASS_MAPPING = {
-    g: Prawn::SVG::Elements::Container,
-    symbol: Prawn::SVG::Elements::Container,
-    defs: Prawn::SVG::Elements::Container,
-    a: Prawn::SVG::Elements::Container,
-    clipPath: Prawn::SVG::Elements::ClipPath,
-    switch: Prawn::SVG::Elements::Container,
-    svg: Prawn::SVG::Elements::Viewport,
-    text: Prawn::SVG::Elements::Text,
-    line: Prawn::SVG::Elements::Line,
-    polyline: Prawn::SVG::Elements::Polyline,
-    polygon: Prawn::SVG::Elements::Polygon,
-    circle: Prawn::SVG::Elements::Circle,
-    ellipse: Prawn::SVG::Elements::Ellipse,
-    rect: Prawn::SVG::Elements::Rect,
-    path: Prawn::SVG::Elements::Path,
-    use: Prawn::SVG::Elements::Use,
-    image: Prawn::SVG::Elements::Image,
+    g:              Prawn::SVG::Elements::Container,
+    symbol:         Prawn::SVG::Elements::Container,
+    defs:           Prawn::SVG::Elements::Container,
+    a:              Prawn::SVG::Elements::Container,
+    clipPath:       Prawn::SVG::Elements::ClipPath,
+    switch:         Prawn::SVG::Elements::Container,
+    svg:            Prawn::SVG::Elements::Viewport,
+    text:           Prawn::SVG::Elements::Text,
+    line:           Prawn::SVG::Elements::Line,
+    polyline:       Prawn::SVG::Elements::Polyline,
+    polygon:        Prawn::SVG::Elements::Polygon,
+    circle:         Prawn::SVG::Elements::Circle,
+    ellipse:        Prawn::SVG::Elements::Ellipse,
+    rect:           Prawn::SVG::Elements::Rect,
+    path:           Prawn::SVG::Elements::Path,
+    use:            Prawn::SVG::Elements::Use,
+    image:          Prawn::SVG::Elements::Image,
     linearGradient: Prawn::SVG::Elements::Gradient,
     radialGradient: Prawn::SVG::Elements::Gradient,
-    marker: Prawn::SVG::Elements::Marker,
-    style: Prawn::SVG::Elements::Ignored, # because it is pre-parsed by Document
-    title: Prawn::SVG::Elements::Ignored,
-    desc: Prawn::SVG::Elements::Ignored,
-    metadata: Prawn::SVG::Elements::Ignored,
-    foreignObject: Prawn::SVG::Elements::Ignored,
-    :"font-face" => Prawn::SVG::Elements::Ignored,
-    filter: Prawn::SVG::Elements::Ignored, # unsupported
-    mask: Prawn::SVG::Elements::Ignored, # unsupported
-  }
+    marker:         Prawn::SVG::Elements::Marker,
+    style:          Prawn::SVG::Elements::Ignored, # because it is pre-parsed by Document
+    title:          Prawn::SVG::Elements::Ignored,
+    desc:           Prawn::SVG::Elements::Ignored,
+    metadata:       Prawn::SVG::Elements::Ignored,
+    foreignObject:  Prawn::SVG::Elements::Ignored,
+    'font-face':    Prawn::SVG::Elements::Ignored,
+    filter:         Prawn::SVG::Elements::Ignored, # unsupported
+    mask:           Prawn::SVG::Elements::Ignored # unsupported
+  }.freeze
 end

data/lib/prawn/svg/extension.rb

@@ -17,7 +17,8 @@ module Prawn
       def svg(data, options = {}, &block)
         svg = Prawn::SVG::Interface.new(data, self, options, &block)
         svg.draw
-        {:warnings => svg.document.warnings, :width => svg.document.sizing.output_width, :height => svg.document.sizing.output_height}
+        { warnings: svg.document.warnings, width: svg.document.sizing.output_width,
+          height: svg.document.sizing.output_height }
       end
     end
   end

data/lib/prawn/svg/font.rb

@@ -1,11 +1,11 @@
 class Prawn::SVG::Font
   GENERIC_CSS_FONT_MAPPING = {
-    "serif"      => "Times-Roman",
-    "sans-serif" => "Helvetica",
-    "cursive"    => "Times-Roman",
-    "fantasy"    => "Times-Roman",
-    "monospace"  => "Courier"
-  }
+    'serif'      => 'Times-Roman',
+    'sans-serif' => 'Helvetica',
+    'cursive'    => 'Times-Roman',
+    'fantasy'    => 'Times-Roman',
+    'monospace'  => 'Courier'
+  }.freeze
 
   attr_reader :name, :weight, :style
 
@@ -38,7 +38,7 @@ class Prawn::SVG::Font
 
   # Construct a subfamily name, ensuring that the subfamily is a valid one for the font.
   def subfamily
-    if subfamilies = @font_registry.installed_fonts[name]
+    if (subfamilies = @font_registry.installed_fonts[name])
       if subfamilies.key?(subfamily_name)
         subfamily_name
       elsif subfamilies.key?(:normal)

data/lib/prawn/svg/font_registry.rb

@@ -1,11 +1,11 @@
 class Prawn::SVG::FontRegistry
   DEFAULT_FONT_PATHS = [
-    "/Library/Fonts",
-    "/System/Library/Fonts",
-    "#{ENV["HOME"]}/Library/Fonts",
-    "/usr/share/fonts/truetype",
-    "/mnt/c/Windows/Fonts", # Bash on Ubuntu on Windows
-  ]
+    '/Library/Fonts',
+    '/System/Library/Fonts',
+    "#{Dir.home}/Library/Fonts",
+    '/usr/share/fonts/truetype',
+    '/mnt/c/Windows/Fonts' # Bash on Ubuntu on Windows
+  ].freeze
 
   @font_path = DEFAULT_FONT_PATHS.select { |path| Dir.exist?(path) }
 
@@ -37,8 +37,8 @@ class Prawn::SVG::FontRegistry
   def merge_external_fonts
     if @font_case_mapping.nil?
       self.class.load_external_fonts unless self.class.external_font_families
-      @font_families.merge!(self.class.external_font_families) do |key, v1, v2|
-       v1
+      @font_families.merge!(self.class.external_font_families) do |_key, v1, _v2|
+        v1
       end
       @font_case_mapping = @font_families.keys.each.with_object({}) do |key, result|
         result[key.downcase] = key
@@ -54,11 +54,11 @@ class Prawn::SVG::FontRegistry
 
       external_font_paths.each do |filename|
         ttf = Prawn::SVG::TTF.new(filename)
-        if ttf.family
-          subfamily = (ttf.subfamily || "normal").gsub(/\s+/, "_").downcase.to_sym
-          subfamily = :normal if subfamily == :regular
-          (external_font_families[ttf.family] ||= {})[subfamily] ||= filename
-        end
+        next unless ttf.family
+
+        subfamily = (ttf.subfamily || 'normal').gsub(/\s+/, '_').downcase.to_sym
+        subfamily = :normal if subfamily == :regular
+        (external_font_families[ttf.family] ||= {})[subfamily] ||= filename
       end
     end
 

data/lib/prawn/svg/gradients.rb

@@ -9,9 +9,9 @@ module Prawn::SVG
       id &&= id.strip
       return unless id && id != ''
 
-      if element = @gradients_by_id[id]
+      if (element = @gradients_by_id[id])
         element
-      elsif raw_element = find_raw_gradient_element_by_id(id)
+      elsif (raw_element = find_raw_gradient_element_by_id(id))
         create_gradient_element(raw_element)
       end
     end
@@ -37,6 +37,7 @@ module Prawn::SVG
 
     def gradient_element?(raw_element)
       return false if raw_element.nil? || raw_element.name.nil?
+
       Elements::TAG_CLASS_MAPPING[raw_element.name.to_sym] == Elements::Gradient
     end
 

data/lib/prawn/svg/interface.rb

@@ -8,12 +8,12 @@ module Prawn
       VALID_OPTIONS = %i[
         at position vposition width height cache_images enable_web_requests
         enable_file_requests_with_root fallback_font_name color_mode
-      ]
+      ].freeze
 
       INHERITABLE_OPTIONS = %i[
         enable_web_requests enable_file_requests_with_root
         cache_images fallback_font_name color_mode
-      ]
+      ].freeze
 
       attr_reader :data, :prawn, :document, :options
 
@@ -60,7 +60,8 @@ module Prawn
         @renderer.position
       end
 
-      def self.font_path # backwards support for when the font_path used to be stored on this class
+      # backwards support for when the font_path used to be stored on this class
+      def self.font_path
         Prawn::SVG::FontRegistry.font_path
       end
     end

data/lib/prawn/svg/loaders/data.rb

@@ -1,6 +1,6 @@
 module Prawn::SVG::Loaders
   class Data
-    REGEXP = %r{\Adata:image/(png|jpeg|svg\+xml);base64(;[a-z0-9]+)*,}i
+    REGEXP = %r{\Adata:image/(png|jpeg|svg\+xml);base64(;[a-z0-9]+)*,}i.freeze
 
     def from_url(url)
       return if url[0..4].downcase != 'data:'
@@ -8,7 +8,7 @@ module Prawn::SVG::Loaders
       matches = url.match(REGEXP)
       if matches.nil?
         raise Prawn::SVG::UrlLoader::Error,
-              'prawn-svg only supports base64-encoded image/png, image/jpeg, and image/svg+xml data URLs'
+          'prawn-svg only supports base64-encoded image/png, image/jpeg, and image/svg+xml data URLs'
       end
 
       matches.post_match.unpack1('m')

data/lib/prawn/svg/loaders/file.rb

@@ -24,7 +24,7 @@ require 'addressable/uri'
 # FILES READ AS BINARY
 # ====================
 # At the moment, prawn-svg uses this class only to load graphical files, which are binary.
-# This class therefore uses IO.binread to read file data.  If it is ever used in the future
+# This class therefore uses File.binread to read file data.  If it is ever used in the future
 # to load text files, it will have to be taught about what kind of file it's expecting to
 # read, and adjust the file read function accordingly.
 #
@@ -34,7 +34,8 @@ module Prawn::SVG::Loaders
 
     def initialize(root_path)
       if root_path.empty?
-        raise ArgumentError, "An empty string is not a valid root path.  Use '.' if you want the current working directory."
+        raise ArgumentError,
+          "An empty string is not a valid root path.  Use '.' if you want the current working directory."
       end
 
       @root_path = ::File.expand_path(root_path)
@@ -62,14 +63,12 @@ module Prawn::SVG::Loaders
       path = build_absolute_and_expand_path(path)
       assert_valid_path!(path)
       assert_file_exists!(path)
-      IO.binread(path)
+      ::File.binread(path)
     end
 
     def build_uri(url)
-      begin
-        URI(url)
-      rescue URI::InvalidURIError
-      end
+      URI(url)
+    rescue URI::InvalidURIError
     end
 
     def assert_valid_path!(path)
@@ -77,7 +76,7 @@ module Prawn::SVG::Loaders
       # making it dependent on whether the file system is case sensitive or not.
       # Leaving it like this until it's a problem for someone.
 
-      if !path.start_with?("#{root_path}#{::File::SEPARATOR}")
+      unless path.start_with?("#{root_path}#{::File::SEPARATOR}")
         raise Prawn::SVG::UrlLoader::Error, "file path is not inside the root path of #{root_path}"
       end
     end
@@ -88,19 +87,18 @@ module Prawn::SVG::Loaders
 
     def assert_valid_file_uri!(uri)
       unless uri.host.nil? || uri.host.empty?
-        raise Prawn::SVG::UrlLoader::Error, "prawn-svg does not suport file: URLs with a host. Your URL probably doesn't start with three slashes, and it should."
+        raise Prawn::SVG::UrlLoader::Error,
+          "prawn-svg does not suport file: URLs with a host. Your URL probably doesn't start with three slashes, and it should."
       end
     end
 
     def assert_file_exists!(path)
-      if !::File.exist?(path)
-        raise Prawn::SVG::UrlLoader::Error, "File #{path} does not exist"
-      end
+      raise Prawn::SVG::UrlLoader::Error, "File #{path} does not exist" unless ::File.exist?(path)
     end
 
     def fix_windows_path(path)
-      if matches = path.match(%r(\A/[a-z]:/)i)
-        path[1..-1]
+      if path.match(%r{\A/[a-z]:/}i)
+        path[1..]
       else
         path
       end

data/lib/prawn/svg/loaders/web.rb

@@ -5,23 +5,19 @@ module Prawn::SVG::Loaders
     def from_url(url)
       uri = build_uri(url)
 
-      if uri && %w(http https).include?(uri.scheme)
-        perform_request(uri)
-      end
+      perform_request(uri) if uri && %w[http https].include?(uri.scheme)
     end
 
     private
 
     def build_uri(url)
-      begin
-        URI(url)
-      rescue URI::InvalidURIError
-      end
+      URI(url)
+    rescue URI::InvalidURIError
     end
 
     def perform_request(uri)
       Net::HTTP.get(uri)
-    rescue => e
+    rescue StandardError => e
       raise Prawn::SVG::UrlLoader::Error, e.message
     end
   end

data/lib/prawn/svg/pathable.rb

@@ -24,25 +24,26 @@ module Prawn::SVG::Pathable
       when Line
         add_call 'line_to', translate(command.destination)
       when Curve
-        add_call 'curve_to', translate(command.destination), bounds: [translate(command.point1), translate(command.point2)]
+        add_call 'curve_to', translate(command.destination),
+          bounds: [translate(command.point1), translate(command.point2)]
       else
-        raise NotImplementedError, "Unknown path command type"
+        raise NotImplementedError, 'Unknown path command type'
       end
     end
   end
 
   def apply_markers
-    if marker = extract_element_from_url_id_reference(properties.marker_start, "marker")
+    if (marker = extract_element_from_url_id_reference(properties.marker_start, 'marker'))
       marker.apply_marker(self, point: commands.first.destination, angle: angles.first)
     end
 
-    if marker = extract_element_from_url_id_reference(properties.marker_mid, "marker")
-      (1..commands.length-2).each do |index|
+    if (marker = extract_element_from_url_id_reference(properties.marker_mid, 'marker'))
+      (1..commands.length - 2).each do |index|
         marker.apply_marker(self, point: commands[index].destination, angle: angles[index])
       end
     end
 
-    if marker = extract_element_from_url_id_reference(properties.marker_end, "marker")
+    if (marker = extract_element_from_url_id_reference(properties.marker_end, 'marker'))
       marker.apply_marker(self, point: commands.last.destination, angle: angles.last)
     end
   end
@@ -53,23 +54,26 @@ module Prawn::SVG::Pathable
     last_point = nil
 
     destination_angles = commands.map do |command|
-      angles = case command
-      when Move
-        [nil, nil]
-      when Close, Line
-        angle = Math.atan2(command.destination[1] - last_point[1], command.destination[0] - last_point[0]) * 180.0 / Math::PI
-        [angle, angle]
-      when Curve
-        point = select_non_equal_point(last_point, command.point1, command.point2, command.destination)
-        start = Math.atan2(point[1] - last_point[1], point[0] - last_point[0]) * 180.0 / Math::PI
-
-        point = select_non_equal_point(command.destination, command.point2, command.point1, last_point)
-        stop = Math.atan2(command.destination[1] - point[1], command.destination[0] - point[0]) * 180.0 / Math::PI
-
-        [start, stop]
-      else
-        raise NotImplementedError, "Unknown path command type"
-      end
+      angles =
+        case command
+        when Move
+          [nil, nil]
+        when Close, Line
+          angle = Math.atan2(command.destination[1] - last_point[1],
+            command.destination[0] - last_point[0]) * 180.0 / Math::PI
+          [angle, angle]
+        when Curve
+          point = select_non_equal_point(last_point, command.point1, command.point2, command.destination)
+          start = Math.atan2(point[1] - last_point[1], point[0] - last_point[0]) * 180.0 / Math::PI
+
+          point = select_non_equal_point(command.destination, command.point2, command.point1, last_point)
+          stop = Math.atan2(command.destination[1] - point[1],
+            command.destination[0] - point[0]) * 180.0 / Math::PI
+
+          [start, stop]
+        else
+          raise NotImplementedError, 'Unknown path command type'
+        end
 
       last_point = command.destination
       angles
@@ -99,10 +103,10 @@ module Prawn::SVG::Pathable
       bisect = (first + second) / 2.0
 
       angles << if (first - second).abs > 180
-        bisect >= 0 ? bisect - 180 : bisect + 180
-      else
-        bisect
-      end
+                  bisect >= 0 ? bisect - 180 : bisect + 180
+                else
+                  bisect
+                end
     else
       angles << destination_angles.last.last
     end
@@ -111,19 +115,19 @@ module Prawn::SVG::Pathable
   end
 
   def parse_points(points_string)
-    values = points_string.
-      to_s.
-      strip.
-      gsub(/(\d)-(\d)/, '\1 -\2').
-      split(Prawn::SVG::Elements::COMMA_WSP_REGEXP).
-      map(&:to_f)
-
-    if values.length % 2 == 1
-      document.warnings << "points attribute has an odd number of points; ignoring the last one"
+    values = points_string
+             .to_s
+             .strip
+             .gsub(/(\d)-(\d)/, '\1 -\2')
+             .split(Prawn::SVG::Elements::COMMA_WSP_REGEXP)
+             .map(&:to_f)
+
+    if values.length.odd?
+      document.warnings << 'points attribute has an odd number of points; ignoring the last one'
       values.pop
     end
 
-    raise Prawn::SVG::Elements::Base::SkipElementQuietly if values.length == 0
+    raise Prawn::SVG::Elements::Base::SkipElementQuietly if values.empty?
 
     values.each_slice(2).to_a
   end

data/lib/prawn/svg/properties.rb

@@ -10,40 +10,41 @@ class Prawn::SVG::Properties
     'large'    => EM / 4 * 5,
     'x-large'  => EM / 4 * 6,
     'xx-large' => EM / 4 * 7
-  }
+  }.freeze
 
   PROPERTIES = {
-    "clip-path"        => Config.new("none", false, %w(inherit none)),
-    "color"            => Config.new('', true),
-    "display"          => Config.new("inline", false, %w(inherit inline none), true),
-    "fill"             => Config.new("black", true, %w(inherit none currentColor)),
-    "fill-opacity"     => Config.new("1", true),
-    "fill-rule"        => Config.new("nonzero", true, %w(inherit nonzero evenodd)),
-    "font-family"      => Config.new("sans-serif", true),
-    "font-size"        => Config.new("medium", true, %w(inherit xx-small x-small small medium large x-large xx-large larger smaller)),
-    "font-style"       => Config.new("normal", true, %w(inherit normal italic oblique), true),
-    "font-variant"     => Config.new("normal", true, %w(inherit normal small-caps), true),
-    "font-weight"      => Config.new("normal", true, %w(inherit normal bold 100 200 300 400 500 600 700 800 900), true), # bolder/lighter not supported
-    "letter-spacing"   => Config.new("normal", true, %w(inherit normal)),
-    "marker-end"       => Config.new("none", true, %w(inherit none)),
-    "marker-mid"       => Config.new("none", true, %w(inherit none)),
-    "marker-start"     => Config.new("none", true, %w(inherit none)),
-    "opacity"          => Config.new("1", false),
-    "overflow"         => Config.new('visible', false, %w(inherit visible hidden scroll auto), true),
-    "stop-color"       => Config.new("black", false, %w(inherit none currentColor)),
-    "stroke"           => Config.new("none", true, %w(inherit none currentColor)),
-    "stroke-dasharray" => Config.new("none", true, %w(inherit none)),
-    "stroke-linecap"   => Config.new("butt", true, %w(inherit butt round square), true),
-    "stroke-linejoin"  => Config.new("miter", true, %w(inherit miter round bevel), true),
-    "stroke-opacity"   => Config.new("1", true),
-    "stroke-width"     => Config.new("1", true),
-    "text-anchor"      => Config.new("start", true, %w(inherit start middle end), true),
-    'text-decoration'  => Config.new('none', true, %w(inherit none underline), true),
-    "dominant-baseline" => Config.new("auto", true, %w(inherit auto middle), true),
+    'clip-path'         => Config.new('none', false, %w[inherit none]),
+    'color'             => Config.new('', true),
+    'display'           => Config.new('inline', false, %w[inherit inline none], true),
+    'fill'              => Config.new('black', true, %w[inherit none currentColor]),
+    'fill-opacity'      => Config.new('1', true),
+    'fill-rule'         => Config.new('nonzero', true, %w[inherit nonzero evenodd]),
+    'font-family'       => Config.new('sans-serif', true),
+    'font-size'         => Config.new('medium', true,
+      %w[inherit xx-small x-small small medium large x-large xx-large larger smaller]),
+    'font-style'        => Config.new('normal', true, %w[inherit normal italic oblique], true),
+    'font-variant'      => Config.new('normal', true, %w[inherit normal small-caps], true),
+    'font-weight'       => Config.new('normal', true, %w[inherit normal bold 100 200 300 400 500 600 700 800 900], true), # bolder/lighter not supported
+    'letter-spacing'    => Config.new('normal', true, %w[inherit normal]),
+    'marker-end'        => Config.new('none', true, %w[inherit none]),
+    'marker-mid'        => Config.new('none', true, %w[inherit none]),
+    'marker-start'      => Config.new('none', true, %w[inherit none]),
+    'opacity'           => Config.new('1', false),
+    'overflow'          => Config.new('visible', false, %w[inherit visible hidden scroll auto], true),
+    'stop-color'        => Config.new('black', false, %w[inherit none currentColor]),
+    'stroke'            => Config.new('none', true, %w[inherit none currentColor]),
+    'stroke-dasharray'  => Config.new('none', true, %w[inherit none]),
+    'stroke-linecap'    => Config.new('butt', true, %w[inherit butt round square], true),
+    'stroke-linejoin'   => Config.new('miter', true, %w[inherit miter round bevel], true),
+    'stroke-opacity'    => Config.new('1', true),
+    'stroke-width'      => Config.new('1', true),
+    'text-anchor'       => Config.new('start', true, %w[inherit start middle end], true),
+    'text-decoration'   => Config.new('none', true, %w[inherit none underline], true),
+    'dominant-baseline' => Config.new('auto', true, %w[inherit auto middle], true)
   }.freeze
 
   PROPERTIES.each do |name, value|
-    value.attr = name.gsub("-", "_")
+    value.attr = name.gsub('-', '_')
     value.ivar = "@#{value.attr}"
   end
 
@@ -51,7 +52,7 @@ class Prawn::SVG::Properties
   NAMES = PROPERTIES.keys
   ATTR_NAMES = PROPERTIES.keys.map { |name| name.gsub('-', '_') }
 
-  attr_accessor *ATTR_NAMES
+  attr_accessor(*ATTR_NAMES)
 
   def load_default_stylesheet
     PROPERTY_CONFIGS.each do |config|
@@ -62,7 +63,7 @@ class Prawn::SVG::Properties
   end
 
   def set(name, value)
-    if config = PROPERTIES[name.to_s.downcase]
+    if (config = PROPERTIES[name.to_s.downcase])
       value = value.strip
       keyword = value.downcase
       keywords = config.keywords || ['inherit']
@@ -92,7 +93,7 @@ class Prawn::SVG::Properties
       value = other.send(config.attr)
 
       if value && value != 'inherit'
-        value = compute_font_size_property(value).to_s if config.attr == "font_size"
+        value = compute_font_size_property(value).to_s if config.attr == 'font_size'
         instance_variable_set(config.ivar, value)
 
       elsif value.nil? && !config.inheritable?
@@ -109,7 +110,7 @@ class Prawn::SVG::Properties
   private
 
   def compute_font_size_property(value)
-    if value[-1] == "%"
+    if value[-1] == '%'
       numerical_font_size * (value.to_f / 100.0)
     elsif value == 'larger'
       numerical_font_size + 4

data/lib/prawn/svg/renderer.rb

@@ -74,7 +74,7 @@ module Prawn
         when :top
           sizing.bounds[1]
         when :center, :centre
-          sizing.bounds[1] - (sizing.bounds[1] - sizing.output_height) / 2.0
+          sizing.bounds[1] - ((sizing.bounds[1] - sizing.output_height) / 2.0)
         when :bottom
           sizing.output_height
         when Numeric
@@ -135,20 +135,20 @@ module Prawn
             @cursor = [at[0], at[1]]
           end
 
-          if offset = options.delete(:offset)
+          if (offset = options.delete(:offset))
             at[0] += offset[0]
             at[1] -= offset[1]
           end
 
           width = prawn.width_of(text, options.merge(kerning: true))
 
-          if stretch_to_width = options.delete(:stretch_to_width)
+          if (stretch_to_width = options.delete(:stretch_to_width))
             factor = stretch_to_width.to_f * 100 / width.to_f
             prawn.add_content "#{factor} Tz"
             width = stretch_to_width.to_f
           end
 
-          if pad_to_width = options.delete(:pad_to_width)
+          if (pad_to_width = options.delete(:pad_to_width))
             padding_required = pad_to_width.to_f - width.to_f
             padding_per_character = padding_required / text.length.to_f
             prawn.add_content "#{padding_per_character} Tc"
@@ -158,7 +158,7 @@ module Prawn
           case options.delete(:text_anchor)
           when 'middle'
             at[0] -= width / 2
-            @cursor = [at[0] + width / 2, at[1]]
+            @cursor = [at[0] + (width / 2), at[1]]
           when 'end'
             at[0] -= width
             @cursor = at.dup
@@ -178,8 +178,8 @@ module Prawn
         when 'transformation_matrix'
           left = prawn.bounds.absolute_left
           top = prawn.bounds.absolute_top
-          arguments[4] += left - (left * arguments[0] + top * arguments[2])
-          arguments[5] += top - (left * arguments[1] + top * arguments[3])
+          arguments[4] += left - ((left * arguments[0]) + (top * arguments[2]))
+          arguments[5] += top - ((left * arguments[1]) + (top * arguments[3]))
 
         when 'clip'
           prawn.add_content 'W n' # clip to path