kramdown 1.17.0 → 2.0.0.beta1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d55901c3c77f65f0c910d6a500fb7aad1f4f1d0f7ac06c3bdb9b6dde621fb4af
-  data.tar.gz: 510414df93e71258d08b88e53070a1290800e98a75c821bf673a154a12e10fa9
+  metadata.gz: a46f6cad363a7cede373f0a518b899c5f46fa0091be9475c8e679e9f5e211b9b
+  data.tar.gz: f8a7d90234109921697e0deb76b370c063759676c92a568e7f40e0a1274aa943
 SHA512:
-  metadata.gz: 90b87fa1dc1dd8a9b1eb683b69f2b2dfe05837d16732b20dc73e71ad4b77baa28be7e0a3980d82d67b61e104ab33311e49bc9e305e8aa849461db022697d3d0f
-  data.tar.gz: 5eab1933ed1bd0c5615291e52e915aad27da34347b3f1a25891417fd6c3f3ec2676c26ccee68b47fc0302f05b683641595a4b77c3ca0be9a7d1d08886c794b00
+  metadata.gz: 2e4036db169432b82bc680f151a41e38b29354c07984cd165f056068d45b052e2a261d2c35786932d8c030a5a0277b1ddc8553e27ae75d585ba2d0074c61e15b
+  data.tar.gz: 2ceebbad2256128c1c0a92f350c6bd275b6d6cbae1363ea6b00ed6779d80f029088dd28b447f95faa06c51a4eff6dbae49b7abe42e07e5d07f26f5465b911ebd

data/CONTRIBUTERS

@@ -1,11 +1,12 @@
   Count Name
 ======= ====
-    887 Thomas Leitner <t_leitner@gmx.at>
+    895 Thomas Leitner <t_leitner@gmx.at>
       7 Christian Cornelssen <ccorn@1tein.de>
       6 Gioele Barabucci <gioele@svario.it>
       4 Ted Pak <powerpak006@gmail.com>
       4 Shuanglei Tao <tsl0922@gmail.com>
       4 Gleb Mazovetskiy <glex.spb@gmail.com>
+      4 Dan Allen <dan.j.allen@gmail.com>
       4 Arne Brasseur <arne@arnebrasseur.net>
       3 Henning Perl <perl@fast-sicher.de>
       3 gettalong <t_leitner@gmx.at>
@@ -17,7 +18,6 @@
       2 Nathanael Jones <nathanael.jones@gmail.com>
       2 Max Meyer <dev@fedux.org>
       2 Jo Hund <jhund@clearcove.ca>
-      2 Dan Allen <dan.j.allen@gmail.com>
       2 Bran <m.versum@gmail.com>
       1 winniehell <git@winniehell.de>
       1 William <suttonwilliamd@gmail.com>
@@ -29,6 +29,7 @@
       1 Tim Besard <tim.besard@gmail.com>
       1 Tim Bates <tim@rumpuslabs.com>
       1 Sun Yaozhu <yzyzsun@gmail.com>
+      1 Stephen Crosby <stevecrozz@gmail.com>
       1 Simon Lydell <simon.lydell@gmail.com>
       1 Shusaku NAKAZATO <cu393uc@gmail.com>
       1 Sebastian Boehm <sebastian@sometimesfood.org>
@@ -45,6 +46,7 @@
       1 Luca Barbato <luca.barbato@gmail.com>
       1 l3kn <hello@l3kn.de>
       1 Kir Kolyshkin <kolyshkin@gmail.com>
+      1 Jun Aruga <jaruga@redhat.com>
       1 Jonathan Hooper <jonathan.hooper@gsa.gov>
       1 John Croisant <jacius@gmail.com>
       1 Joe Fiorini <joe@faithfulgeek.org>

data/VERSION

@@ -1 +1 @@
-1.17.0
+2.0.0.beta1

data/bin/kramdown

@@ -2,7 +2,7 @@
 # -*- coding: utf-8 -*-
 #
 #--
-# Copyright (C) 2009-2015 Thomas Leitner <t_leitner@gmx.at>
+# Copyright (C) 2009-2019 Thomas Leitner <t_leitner@gmx.at>
 #
 # This file is part of kramdown which is licensed under the MIT.
 #++
@@ -24,7 +24,7 @@ begin
                  File.expand_path(ENV['XDG_CONFIG_HOME'] || '~/.config')
                end
   config_file = File.join(config_dir, "kramdownrc")
-rescue
+rescue StandardError
 end
 
 options = {}
@@ -32,23 +32,23 @@ format = ['html']
 
 OptionParser.new do |opts|
   opts.banner = "Usage: kramdown [options] [FILE FILE ...]"
-  opts.summary_indent = ' '*4
+  opts.summary_indent = ' ' * 4
 
   opts.separator ""
   opts.separator "Command line options:"
   opts.separator ""
 
   opts.on("-i", "--input ARG", "Specify the input format: kramdown (default), " \
-          "html, GFM or markdown") {|v| options[:input] = v}
+          "html, or markdown") {|v| options[:input] = v }
   opts.on("-o", "--output ARG", Array, "Specify one or more output formats separated by commas: " \
-          "html (default),", "kramdown, latex, pdf, man or remove_html_tags") {|v| format = v}
+          "html (default),", "kramdown, latex, man or remove_html_tags") {|v| format = v }
   opts.separator ""
   opts.on("--no-config-file", "Do not read any configuration file. Default behavior is to check " \
-          "for a", "configuration file and read it if it exists.") {config_file = nil}
+          "for a", "configuration file and read it if it exists.") { config_file = nil }
   opts.on("--config-file FILE", "Specify the name of a configuration file with kramdown options " \
           "in YAML", "format, e.g. \"auto_id_prefix: ARG\" instead of \"--auto-id-prefix ARG\"",
           "and \"auto_ids: false\" instead of \"--no-auto-ids\".",
-          "Default: #{config_file}") {|v| config_file = v}
+          "Default: #{config_file}") {|v| config_file = v }
   opts.separator ""
   opts.on("-v", "--version", "Show the version of kramdown") do
     puts Kramdown::VERSION
@@ -66,26 +66,25 @@ OptionParser.new do |opts|
   Kramdown::Options.definitions.sort.each do |n, definition|
     no = n.to_s.tr('_', '-')
     if definition.type == Kramdown::Options::Boolean
-      opts.on("--[no-]#{no}") {|v| options[n] = Kramdown::Options.parse(n, v)}
+      opts.on("--[no-]#{no}") {|v| options[n] = Kramdown::Options.parse(n, v) }
     else
       type = definition.type
       type = String if type == Symbol || type == Object
-      opts.on("--#{no} ARG", type) {|v| options[n] = Kramdown::Options.parse(n, v)}
+      opts.on("--#{no} ARG", type) {|v| options[n] = Kramdown::Options.parse(n, v) }
     end
 
     definition.desc.split(/\n/).each do |line|
-      opts.separator opts.summary_indent + ' '*6 + line
+      opts.separator opts.summary_indent + ' ' * 6 + line
     end
     opts.separator ''
   end
-
 end.parse!
 
 begin
   if config_file && File.exist?(config_file)
     config_file_options = YAML.safe_load(File.read(config_file), [Symbol])
     case config_file_options
-    when nil  # empty configuration file except perhaps YAML header and comments
+    when nil # empty configuration file except perhaps YAML header and comments
       # Nothing to do
     when Hash
       options = config_file_options.merge(options)
@@ -95,9 +94,9 @@ begin
   end
   doc = Kramdown::Document.new(ARGF.read, options)
   result = ''
-  format.each {|f| result = doc.send("to_#{f}")}
+  format.each {|f| result = doc.send("to_#{f}") }
   puts result
-  doc.warnings.each {|warn| $stderr.puts "Warning: #{warn}"}
+  doc.warnings.each {|warn| $stderr.puts "Warning: #{warn}" }
 rescue Kramdown::Error => e
   $stderr.puts "Error: #{e.message}"
   exit(1)

data/lib/kramdown.rb

@@ -1,7 +1,7 @@
-# -*- coding: utf-8 -*-
+# -*- coding: utf-8; frozen_string_literal: true -*-
 #
 #--
-# Copyright (C) 2009-2016 Thomas Leitner <t_leitner@gmx.at>
+# Copyright (C) 2009-2019 Thomas Leitner <t_leitner@gmx.at>
 #
 # This file is part of kramdown which is licensed under the MIT.
 #++

data/lib/kramdown/converter.rb

@@ -1,7 +1,7 @@
-# -*- coding: utf-8 -*-
+# -*- coding: utf-8; frozen_string_literal: true -*-
 #
 #--
-# Copyright (C) 2009-2016 Thomas Leitner <t_leitner@gmx.at>
+# Copyright (C) 2009-2019 Thomas Leitner <t_leitner@gmx.at>
 #
 # This file is part of kramdown which is licensed under the MIT.
 #++
@@ -25,7 +25,6 @@ module Kramdown
     autoload :Kramdown, 'kramdown/converter/kramdown'
     autoload :Toc, 'kramdown/converter/toc'
     autoload :RemoveHtmlTags, 'kramdown/converter/remove_html_tags'
-    autoload :Pdf, 'kramdown/converter/pdf'
     autoload :HashAST, 'kramdown/converter/hash_ast'
     autoload :HashAst, 'kramdown/converter/hash_ast'
     autoload :Man, 'kramdown/converter/man'
@@ -34,7 +33,7 @@ module Kramdown
 
     configurable(:syntax_highlighter)
 
-    ['Minted', "Coderay", "Rouge"].each do |klass_name|
+    ['Minted', "Rouge"].each do |klass_name|
       kn_down = klass_name.downcase.intern
       add_syntax_highlighter(kn_down) do |converter, text, lang, type, opts|
         require "kramdown/converter/syntax_highlighter/#{kn_down}"
@@ -42,7 +41,7 @@ module Kramdown
         if !klass.const_defined?(:AVAILABLE) || klass::AVAILABLE
           add_syntax_highlighter(kn_down, klass)
         else
-          add_syntax_highlighter(kn_down) {|*args| nil}
+          add_syntax_highlighter(kn_down) { nil }
         end
         syntax_highlighter(kn_down).call(converter, text, lang, type, opts)
       end
@@ -50,7 +49,7 @@ module Kramdown
 
     configurable(:math_engine)
 
-    ["Mathjax", "MathjaxNode", "Katex", "SsKaTeX", "Ritex", "Itex2MML"].each do |klass_name|
+    ["Mathjax"].each do |klass_name|
       kn_down = klass_name.downcase.intern
       add_math_engine(kn_down) do |converter, el, opts|
         require "kramdown/converter/math_engine/#{kn_down}"
@@ -58,7 +57,7 @@ module Kramdown
         if !klass.const_defined?(:AVAILABLE) || klass::AVAILABLE
           add_math_engine(kn_down, klass)
         else
-          add_math_engine(kn_down) {|*args| nil}
+          add_math_engine(kn_down) { nil }
         end
         math_engine(kn_down).call(converter, el, opts)
       end

data/lib/kramdown/converter/base.rb

@@ -1,7 +1,7 @@
-# -*- coding: utf-8 -*-
+# -*- coding: utf-8; frozen_string_literal: true -*-
 #
 #--
-# Copyright (C) 2009-2016 Thomas Leitner <t_leitner@gmx.at>
+# Copyright (C) 2009-2019 Thomas Leitner <t_leitner@gmx.at>
 #
 # This file is part of kramdown which is licensed under the MIT.
 #++
@@ -101,10 +101,16 @@ module Kramdown
       def self.convert(tree, options = {})
         converter = new(tree, ::Kramdown::Options.merge(options.merge(tree.options[:options] || {})))
 
-        apply_template(converter, '') if !converter.options[:template].empty? && converter.apply_template_before?
+        if !converter.options[:template].empty? && converter.apply_template_before?
+          apply_template(converter, '')
+        end
         result = converter.convert(tree)
-        result.encode!(tree.options[:encoding]) if result.respond_to?(:encode!) && result.encoding != Encoding::BINARY
-        result = apply_template(converter, result) if !converter.options[:template].empty? && converter.apply_template_after?
+        if result.respond_to?(:encode!) && result.encoding != Encoding::BINARY
+          result.encode!(tree.options[:encoding])
+        end
+        if !converter.options[:template].empty? && converter.apply_template_after?
+          result = apply_template(converter, result)
+        end
 
         [result, converter.warnings]
       end
@@ -112,7 +118,7 @@ module Kramdown
       # Convert the element +el+ and return the resulting object.
       #
       # This is the only method that has to be implemented by sub-classes!
-      def convert(el)
+      def convert(_el)
         raise NotImplementedError
       end
 
@@ -125,28 +131,11 @@ module Kramdown
         obj = Object.new
         obj.instance_variable_set(:@converter, converter)
         obj.instance_variable_set(:@body, body)
-        erb.result(obj.instance_eval{binding})
+        erb.result(obj.instance_eval { binding })
       end
 
       # Return the template specified by +template+.
-      def self.get_template(template)
-        #DEPRECATED: use content of #get_template_new in 2.0
-        format_ext = '.' + self.name.split(/::/).last.downcase
-        shipped = File.join(::Kramdown.data_dir, template + format_ext)
-        if File.exist?(template)
-          File.read(template)
-        elsif File.exist?(template + format_ext)
-          File.read(template + format_ext)
-        elsif File.exist?(shipped)
-          File.read(shipped)
-        elsif template.start_with?('string://')
-          template.sub(/\Astring:\/\//, '')
-        else
-          get_template_new(template)
-        end
-      end
-
-      def self.get_template_new(template) # :nodoc:
+      def self.get_template(template) # :nodoc:
         format_ext = '.' + ::Kramdown::Utils.snake_case(self.name.split(/::/).last)
         shipped = File.join(::Kramdown.data_dir, template + format_ext)
         if File.exist?(template)
@@ -232,10 +221,10 @@ module Kramdown
       def generate_id(str)
         str = ::Kramdown::Utils::Unidecoder.decode(str) if @options[:transliterated_header_ids]
         gen_id = basic_generate_id(str)
-        gen_id = 'section' if gen_id.length == 0
+        gen_id = 'section' if gen_id.empty?
         @used_ids ||= {}
-        if @used_ids.has_key?(gen_id)
-          gen_id += '-' << (@used_ids[gen_id] += 1).to_s
+        if @used_ids.key?(gen_id)
+          gen_id += "-#{@used_ids[gen_id] += 1}"
         else
           @used_ids[gen_id] = 0
         end
@@ -252,7 +241,7 @@ module Kramdown
         gen_id
       end
 
-      SMART_QUOTE_INDICES = {:lsquo => 0, :rsquo => 1, :ldquo => 2, :rdquo => 3} # :nodoc:
+      SMART_QUOTE_INDICES = {lsquo: 0, rsquo: 1, ldquo: 2, rdquo: 3} # :nodoc:
 
       # Return the entity that represents the given smart_quote element.
       def smart_quote_entity(el)

data/lib/kramdown/converter/hash_ast.rb

@@ -1,7 +1,7 @@
-# -*- coding: utf-8 -*-
+# -*- coding: utf-8; frozen_string_literal: true -*-
 #
 #--
-# Copyright (C) 2009-2016 Thomas Leitner <t_leitner@gmx.at>
+# Copyright (C) 2009-2019 Thomas Leitner <t_leitner@gmx.at>
 #
 # This file is part of kramdown which is licensed under the MIT.
 #++
@@ -19,13 +19,13 @@ module Kramdown
     class HashAST < Base
 
       def convert(el)
-        hash = {:type => el.type}
+        hash = {type: el.type}
         hash[:attr] = el.attr unless el.attr.empty?
         hash[:value] = el.value unless el.value.nil?
         hash[:options] = el.options unless el.options.empty?
         unless el.children.empty?
           hash[:children] = []
-          el.children.each {|child| hash[:children] << convert(child)}
+          el.children.each {|child| hash[:children] << convert(child) }
         end
         hash
       end

data/lib/kramdown/converter/html.rb

@@ -1,7 +1,7 @@
-# -*- coding: utf-8 -*-
+# -*- coding: utf-8; frozen_string_literal: true -*-
 #
 #--
-# Copyright (C) 2009-2016 Thomas Leitner <t_leitner@gmx.at>
+# Copyright (C) 2009-2019 Thomas Leitner <t_leitner@gmx.at>
 #
 # This file is part of kramdown which is licensed under the MIT.
 #++
@@ -49,7 +49,7 @@ module Kramdown
       end
 
       # The mapping of element type to conversion method.
-      DISPATCHER = Hash.new {|h,k| h[k] = "convert_#{k}"}
+      DISPATCHER = Hash.new {|h, k| h[k] = "convert_#{k}" }
 
       # Dispatch the conversion of the element +el+ to a +convert_TYPE+ method using the +type+ of
       # the element.
@@ -63,7 +63,7 @@ module Kramdown
       # Pushes +el+ onto the @stack before converting the child elements and pops it from the stack
       # afterwards.
       def inner(el, indent)
-        result = ''
+        result = +''
         indent += @indent
         @stack.push(el)
         el.children.each do |inner_el|
@@ -73,11 +73,11 @@ module Kramdown
         result
       end
 
-      def convert_blank(el, indent)
+      def convert_blank(_el, _indent)
         "\n"
       end
 
-      def convert_text(el, indent)
+      def convert_text(el, _indent)
         escape_html(el.value, :text)
       end
 
@@ -97,7 +97,7 @@ module Kramdown
 
         if highlighted_code
           add_syntax_highlighter_to_class_attr(attr, lang || hl_opts[:default_lang])
-          "#{' '*indent}<div#{html_attributes(attr)}>#{highlighted_code}#{' '*indent}</div>\n"
+          "#{' ' * indent}<div#{html_attributes(attr)}>#{highlighted_code}#{' ' * indent}</div>\n"
         else
           result = escape_html(el.value)
           result.chomp!
@@ -114,7 +114,8 @@ module Kramdown
           end
           code_attr = {}
           code_attr['class'] = "language-#{lang}" if lang
-          "#{' '*indent}<pre#{html_attributes(attr)}><code#{html_attributes(code_attr)}>#{result}\n</code></pre>\n"
+          "#{' ' * indent}<pre#{html_attributes(attr)}>" \
+            "<code#{html_attributes(code_attr)}>#{result}\n</code></pre>\n"
         end
       end
 
@@ -133,42 +134,42 @@ module Kramdown
       end
 
       def convert_hr(el, indent)
-        "#{' '*indent}<hr#{html_attributes(el.attr)} />\n"
+        "#{' ' * indent}<hr#{html_attributes(el.attr)} />\n"
       end
 
       def convert_ul(el, indent)
         if !@toc_code && (el.options[:ial][:refs].include?('toc') rescue nil)
-          @toc_code = [el.type, el.attr, (0..128).to_a.map{|a| rand(36).to_s(36)}.join]
+          @toc_code = [el.type, el.attr, (0..128).to_a.map { rand(36).to_s(36) }.join]
           @toc_code.last
         elsif !@footnote_location && el.options[:ial] && (el.options[:ial][:refs] || []).include?('footnotes')
-          @footnote_location = (0..128).to_a.map{|a| rand(36).to_s(36)}.join
+          @footnote_location = (0..128).to_a.map { rand(36).to_s(36) }.join
         else
           format_as_indented_block_html(el.type, el.attr, inner(el, indent), indent)
         end
       end
-      alias :convert_ol :convert_ul
+      alias convert_ol convert_ul
 
       def convert_dl(el, indent)
         format_as_indented_block_html(el.type, el.attr, inner(el, indent), indent)
       end
 
       def convert_li(el, indent)
-        output = ' '*indent << "<#{el.type}" << html_attributes(el.attr) << ">"
+        output = ' ' * indent << "<#{el.type}" << html_attributes(el.attr) << ">"
         res = inner(el, indent)
         if el.children.empty? || (el.children.first.type == :p && el.children.first.options[:transparent])
-          output << res << (res =~ /\n\Z/ ? ' '*indent : '')
+          output << res << (res =~ /\n\Z/ ? ' ' * indent : '')
         else
-          output << "\n" << res << ' '*indent
+          output << "\n" << res << ' ' * indent
         end
         output << "</#{el.type}>\n"
       end
-      alias :convert_dd :convert_li
+      alias convert_dd convert_li
 
       def convert_dt(el, indent)
         attr = el.attr.dup
         @stack.last.options[:ial][:refs].each do |ref|
           if ref =~ /\Aauto_ids(?:-([\w-]+))?/
-            attr['id'] = ($1 ? $1 : '') << basic_generate_id(el.options[:raw_text])
+            attr['id'] = "#{$1}#{basic_generate_id(el.options[:raw_text])}".lstrip
             break
           end
         end if !attr['id'] && @stack.last.options[:ial] && @stack.last.options[:ial][:refs]
@@ -178,17 +179,20 @@ module Kramdown
       def convert_html_element(el, indent)
         res = inner(el, indent)
         if el.options[:category] == :span
-          "<#{el.value}#{html_attributes(el.attr)}" << (res.empty? && HTML_ELEMENTS_WITHOUT_BODY.include?(el.value) ? " />" : ">#{res}</#{el.value}>")
+          "<#{el.value}#{html_attributes(el.attr)}" + \
+            (res.empty? && HTML_ELEMENTS_WITHOUT_BODY.include?(el.value) ? " />" : ">#{res}</#{el.value}>")
         else
-          output = ''
-          output << ' '*indent if @stack.last.type != :html_element || @stack.last.options[:content_model] != :raw
+          output = +''
+          if @stack.last.type != :html_element || @stack.last.options[:content_model] != :raw
+            output << ' ' * indent
+          end
           output << "<#{el.value}#{html_attributes(el.attr)}"
           if el.options[:is_closed] && el.options[:content_model] == :raw
             output << " />"
           elsif !res.empty? && el.options[:content_model] != :block
             output << ">#{res}</#{el.value}>"
           elsif !res.empty?
-            output << ">\n#{res.chomp}\n"  << ' '*indent << "</#{el.value}>"
+            output << ">\n#{res.chomp}\n" << ' ' * indent << "</#{el.value}>"
           elsif HTML_ELEMENTS_WITHOUT_BODY.include?(el.value)
             output << " />"
           else
@@ -200,21 +204,22 @@ module Kramdown
       end
 
       def convert_xml_comment(el, indent)
-        if el.options[:category] == :block && (@stack.last.type != :html_element || @stack.last.options[:content_model] != :raw)
-          ' '*indent << el.value << "\n"
+        if el.options[:category] == :block &&
+            (@stack.last.type != :html_element || @stack.last.options[:content_model] != :raw)
+          ' ' * indent << el.value << "\n"
         else
           el.value
         end
       end
-      alias :convert_xml_pi :convert_xml_comment
+      alias convert_xml_pi convert_xml_comment
 
       def convert_table(el, indent)
         format_as_indented_block_html(el.type, el.attr, inner(el, indent), indent)
       end
-      alias :convert_thead :convert_table
-      alias :convert_tbody :convert_table
-      alias :convert_tfoot :convert_table
-      alias :convert_tr  :convert_table
+      alias convert_thead convert_table
+      alias convert_tbody convert_table
+      alias convert_tfoot convert_table
+      alias convert_tr convert_table
 
       ENTITY_NBSP = ::Kramdown::Utils::Entities.entity('nbsp') # :nodoc:
 
@@ -225,20 +230,20 @@ module Kramdown
         alignment = @stack[-3].options[:alignment][@stack.last.children.index(el)]
         if alignment != :default
           attr = el.attr.dup
-          attr['style'] = (attr.has_key?('style') ? "#{attr['style']}; ": '') << "text-align: #{alignment}"
+          attr['style'] = (attr.key?('style') ? "#{attr['style']}; " : '') + "text-align: #{alignment}"
         end
         format_as_block_html(type, attr, res.empty? ? entity_to_str(ENTITY_NBSP) : res, indent)
       end
 
       def convert_comment(el, indent)
         if el.options[:category] == :block
-          "#{' '*indent}<!-- #{el.value} -->\n"
+          "#{' ' * indent}<!-- #{el.value} -->\n"
         else
           "<!-- #{el.value} -->"
         end
       end
 
-      def convert_br(el, indent)
+      def convert_br(_el, _indent)
         "<br />"
       end
 
@@ -246,11 +251,11 @@ module Kramdown
         format_as_span_html(el.type, el.attr, inner(el, indent))
       end
 
-      def convert_img(el, indent)
+      def convert_img(el, _indent)
         "<img#{html_attributes(el.attr)} />"
       end
 
-      def convert_codespan(el, indent)
+      def convert_codespan(el, _indent)
         attr = el.attr.dup
         lang = extract_code_language(attr)
         hl_opts = {}
@@ -264,7 +269,7 @@ module Kramdown
         format_as_span_html('code', attr, result)
       end
 
-      def convert_footnote(el, indent)
+      def convert_footnote(el, _indent)
         repeat = ''
         if (footnote = @footnotes_by_name[el.options[:name]])
           number = footnote[2]
@@ -275,10 +280,11 @@ module Kramdown
           @footnotes << [el.options[:name], el.value, number, 0]
           @footnotes_by_name[el.options[:name]] = @footnotes.last
         end
-        "<sup id=\"fnref:#{el.options[:name]}#{repeat}\"><a href=\"#fn:#{el.options[:name]}\" class=\"footnote\">#{number}</a></sup>"
+        "<sup id=\"fnref:#{el.options[:name]}#{repeat}\">" \
+          "<a href=\"#fn:#{el.options[:name]}\" class=\"footnote\">#{number}</a></sup>"
       end
 
-      def convert_raw(el, indent)
+      def convert_raw(el, _indent)
         if !el.options[:type] || el.options[:type].empty? || el.options[:type].include?('html')
           el.value + (el.options[:category] == :block ? "\n" : '')
         else
@@ -289,39 +295,41 @@ module Kramdown
       def convert_em(el, indent)
         format_as_span_html(el.type, el.attr, inner(el, indent))
       end
-      alias :convert_strong :convert_em
+      alias convert_strong convert_em
 
-      def convert_entity(el, indent)
+      def convert_entity(el, _indent)
         entity_to_str(el.value, el.options[:original])
       end
 
       TYPOGRAPHIC_SYMS = {
-        :mdash => [::Kramdown::Utils::Entities.entity('mdash')],
-        :ndash => [::Kramdown::Utils::Entities.entity('ndash')],
-        :hellip => [::Kramdown::Utils::Entities.entity('hellip')],
-        :laquo_space => [::Kramdown::Utils::Entities.entity('laquo'), ::Kramdown::Utils::Entities.entity('nbsp')],
-        :raquo_space => [::Kramdown::Utils::Entities.entity('nbsp'), ::Kramdown::Utils::Entities.entity('raquo')],
-        :laquo => [::Kramdown::Utils::Entities.entity('laquo')],
-        :raquo => [::Kramdown::Utils::Entities.entity('raquo')]
+        mdash: [::Kramdown::Utils::Entities.entity('mdash')],
+        ndash: [::Kramdown::Utils::Entities.entity('ndash')],
+        hellip: [::Kramdown::Utils::Entities.entity('hellip')],
+        laquo_space: [::Kramdown::Utils::Entities.entity('laquo'),
+                      ::Kramdown::Utils::Entities.entity('nbsp')],
+        raquo_space: [::Kramdown::Utils::Entities.entity('nbsp'),
+                      ::Kramdown::Utils::Entities.entity('raquo')],
+        laquo: [::Kramdown::Utils::Entities.entity('laquo')],
+        raquo: [::Kramdown::Utils::Entities.entity('raquo')],
       } # :nodoc:
-      def convert_typographic_sym(el, indent)
+      def convert_typographic_sym(el, _indent)
         if (result = @options[:typographic_symbols][el.value])
           escape_html(result, :text)
         else
-          TYPOGRAPHIC_SYMS[el.value].map {|e| entity_to_str(e)}.join('')
+          TYPOGRAPHIC_SYMS[el.value].map {|e| entity_to_str(e) }.join('')
         end
       end
 
-      def convert_smart_quote(el, indent)
+      def convert_smart_quote(el, _indent)
         entity_to_str(smart_quote_entity(el))
       end
 
       def convert_math(el, indent)
-        if (result = format_math(el, :indent => indent))
+        if (result = format_math(el, indent: indent))
           result
         else
           attr = el.attr.dup
-          (attr['class'] = (attr['class'] || '') << " kdmath").lstrip!
+          attr['class'] = "#{attr['class']} kdmath".lstrip
           if el.options[:category] == :block
             format_as_block_html('div', attr, "$$\n#{el.value}\n$$", indent)
           else
@@ -330,7 +338,7 @@ module Kramdown
         end
       end
 
-      def convert_abbreviation(el, indent)
+      def convert_abbreviation(el, _indent)
         title = @root.options[:abbrev_defs][el.value]
         attr = @root.options[:abbrev_attr][el.value].dup
         attr['title'] = title unless title.empty?
@@ -346,7 +354,7 @@ module Kramdown
         end
         if @toc_code
           toc_tree = generate_toc_tree(@toc, @toc_code[0], @toc_code[1] || {})
-          text = if toc_tree.children.size > 0
+          text = if !toc_tree.children.empty?
                    convert(toc_tree, 0)
                  else
                    ''
@@ -363,13 +371,13 @@ module Kramdown
 
       # Format the given element as block HTML.
       def format_as_block_html(name, attr, body, indent)
-        "#{' '*indent}<#{name}#{html_attributes(attr)}>#{body}</#{name}>\n"
+        "#{' ' * indent}<#{name}#{html_attributes(attr)}>#{body}</#{name}>\n"
       end
 
       # Format the given element as block HTML with a newline after the start tag and indentation
       # before the end tag.
       def format_as_indented_block_html(name, attr, body, indent)
-        "#{' '*indent}<#{name}#{html_attributes(attr)}>\n#{body}#{' '*indent}</#{name}>\n"
+        "#{' ' * indent}<#{name}#{html_attributes(attr)}>\n#{body}#{' ' * indent}</#{name}>\n"
       end
 
       # Add the syntax highlighter name to the 'class' attribute of the given attribute hash. And
@@ -381,12 +389,12 @@ module Kramdown
 
       # Generate and return an element tree for the table of contents.
       def generate_toc_tree(toc, type, attr)
-        sections = Element.new(type, nil, attr)
+        sections = Element.new(type, nil, attr.dup)
         sections.attr['id'] ||= 'markdown-toc'
         stack = []
         toc.each do |level, id, children|
-          li = Element.new(:li, nil, nil, {:level => level})
-          li.children << Element.new(:p, nil, nil, {:transparent => true})
+          li = Element.new(:li, nil, nil, level: level)
+          li.children << Element.new(:p, nil, nil, transparent: true)
           a = Element.new(:a, nil)
           a.attr['href'] = "##{id}"
           a.attr['id'] = "#{sections.attr['id']}-#{id}"
@@ -395,7 +403,7 @@ module Kramdown
           li.children << Element.new(type)
 
           success = false
-          while !success
+          until success
             if stack.empty?
               sections.children << li
               stack << li
@@ -406,13 +414,13 @@ module Kramdown
               success = true
             else
               item = stack.pop
-              item.children.pop unless item.children.last.children.size > 0
+              item.children.pop if item.children.last.children.empty?
             end
           end
         end
-        while !stack.empty?
+        until stack.empty?
           item = stack.pop
-          item.children.pop unless item.children.last.children.size > 0
+          item.children.pop if item.children.last.children.empty?
         end
         sections
       end
@@ -442,9 +450,9 @@ module Kramdown
 
       # Obfuscate the +text+ by using HTML entities.
       def obfuscate(text)
-        result = ""
+        result = +''
         text.each_byte do |b|
-          result << (b > 128 ? b.chr : "&#%03d;" % b)
+          result << (b > 128 ? b.chr : sprintf("&#%03d;", b))
         end
         result.force_encoding(text.encoding)
         result
@@ -460,7 +468,7 @@ module Kramdown
         backlink_text = escape_html(@options[:footnote_backlink], :text)
         while i < @footnotes.length
           name, data, _, repeat = *@footnotes[i]
-          li = Element.new(:li, nil, {'id' => "fn:#{name}"})
+          li = Element.new(:li, nil, 'id' => "fn:#{name}")
           li.children = Marshal.load(Marshal.dump(data.children))
 
           para = nil
@@ -480,16 +488,23 @@ module Kramdown
 
           unless @options[:footnote_backlink].empty?
             nbsp = entity_to_str(ENTITY_NBSP)
-            para.children << Element.new(:raw, FOOTNOTE_BACKLINK_FMT % [insert_space ? nbsp : '', name, backlink_text])
+            value = sprintf(FOOTNOTE_BACKLINK_FMT, (insert_space ? nbsp : ''), name, backlink_text)
+            para.children << Element.new(:raw, value)
             (1..repeat).each do |index|
-              para.children << Element.new(:raw, FOOTNOTE_BACKLINK_FMT % [nbsp, "#{name}:#{index}", "#{backlink_text}<sup>#{index+1}</sup>"])
+              value = sprintf(FOOTNOTE_BACKLINK_FMT, nbsp, "#{name}:#{index}",
+                              "#{backlink_text}<sup>#{index + 1}</sup>")
+              para.children << Element.new(:raw, value)
             end
           end
 
           ol.children << Element.new(:raw, convert(li, 4))
           i += 1
         end
-        (ol.children.empty? ? '' : format_as_indented_block_html('div', {:class => "footnotes"}, convert(ol, 2), 0))
+        if ol.children.empty?
+          ''
+        else
+          format_as_indented_block_html('div', {class: "footnotes"}, convert(ol, 2), 0)
+        end
       end
 
     end