kramdown 2.3.2 → 2.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f656918e531f3b20c720da64a979ea287aabf5e9b2a9bb18599d968957926e41
-  data.tar.gz: 3150331b7195b4dd186984a301a174ca0943e85ad11c3aef47b797b9dd46ad58
+  metadata.gz: 53bf662f86052a77c6c9867b0c4e6808c9cee58ce1ed2b5e77c74b9d015b8e73
+  data.tar.gz: 3f56c0fd6fb00b867ade3ce1df9ddda4380aa01307b37420fbc5a4f1330a8156
 SHA512:
-  metadata.gz: d7b0d8a7af60a0c72a6a49b26a05fc8fe3c348458802f489891166711aad985d0240d8b8882432d81ba6a061fd78e5c598ddc6dd6fa5f5de7b1aa57cf5fc9514
-  data.tar.gz: bec4c4bb4705f28db0bc4b7f3e4f6105e531a8711870edd90f36b0e270a653305bef544116760d3748ef91f112d51e666b468c38cddc116bcdf8c1267fbab1d3
+  metadata.gz: 813301a1fe55b9eeb724fec2e482a694e74852d2509cd74e64cbc5f5df5122f77d63b317439c4184b9ebc7651584d5c777d58eb29183e07ab941d453c12a6ed4
+  data.tar.gz: 610257c36fc35ac7195114f1a5458d02939116696e799ab257a821a2ab5025ecf94470a4800cd767dfdc0730da0a0ec599276ac2ac745ef64b07d2bc4d20ba9b

data/CONTRIBUTERS

@@ -1,6 +1,6 @@
   Count Name
 ======= ====
-    960 Thomas Leitner <t_leitner@gmx.at>
+    986 Thomas Leitner <t_leitner@gmx.at>
      18 Ashwin Maroli <ashmaroli@gmail.com>
       7 Christian Cornelssen <ccorn@1tein.de>
       6 Gioele Barabucci <gioele@svario.it>
@@ -11,6 +11,7 @@
       4 Arne Brasseur <arne@arnebrasseur.net>
       3 Henning Perl <perl@fast-sicher.de>
       3 gettalong <t_leitner@gmx.at>
+      3 Carsten Bormann <cabo@tzi.org>
       3 Brandur <brandur@mutelight.org>
       3 Ben Armston <ben.armston@googlemail.com>
       3 Ashwin Maroli <ashmaroli@users.noreply.github.com>
@@ -20,10 +21,11 @@
       2 Nathanael Jones <nathanael.jones@gmail.com>
       2 Max Meyer <dev@fedux.org>
       2 Jo Hund <jhund@clearcove.ca>
-      2 Carsten Bormann <cabo@tzi.org>
+      2 Bryce Willey <Bryce.Steven.Willey@gmail.com>
       2 Bran <m.versum@gmail.com>
       1 winniehell <git@winniehell.de>
       1 William <suttonwilliamd@gmail.com>
+      1 Virgil Ierubino <30758921+Convincible@users.noreply.github.com>
       1 Uwe Kubosch <donv@users.noreply.github.com>
       1 utenmiki <utenmiki@gmail.com>
       1 Trevor Wennblom <trevor@well.com>
@@ -45,6 +47,8 @@
       1 scherr <maximilianscherr@gmail.com>
       1 Postmodern <postmodern.mod3@gmail.com>
       1 Pete Michaud <michaudp@gmail.com>
+      1 Paul McMahon <paul@tokyodev.com>
+      1 Paul <2528280+psfrolov@users.noreply.github.com>
       1 Noah Doersing <doersino@gmail.com>
       1 myqlarson <myqlarson@gmail.com>
       1 milo.simpson <milo.simpson@bazaarvoice.com>
@@ -59,9 +63,11 @@
       1 Kir Kolyshkin <kolyshkin@gmail.com>
       1 Jun Aruga <jaruga@redhat.com>
       1 Jonathan Hooper <jonathan.hooper@gsa.gov>
+      1 John Haugeland <stonecypher@gmail.com>
       1 John Croisant <jacius@gmail.com>
       1 Joe Fiorini <joe@faithfulgeek.org>
       1 Jens Kraemer <jk@jkraemer.net>
+      1 Jay Stramel <js@ionactual.com>
       1 Hirofumi Wakasugi <baenej@gmail.com>
       1 Hector Correa <hector@hectorcorrea.com>
       1 Florian Klampfer <f.klampfer@gmail.com>
@@ -72,13 +78,16 @@
       1 Daniel Bair <daniel@danielbair.com>
       1 Damien Pollet <damien.pollet@gmail.com>
       1 Christopher Jefferson <caj21@st-andrews.ac.uk>
+      1 Christian Boos <christian.boos@bct-technology.com>
       1 Cédric Boutillier <cedric.boutillier@gmail.com>
       1 Bob Lail <lail@squareup.com>
+      1 Ashesh Kumar Singh <user501254@gmail.com>
       1 Ashe Connor <ashe@kivikakk.ee>
       1 aschmitz <29508+aschmitz@users.noreply.github.com>
       1 Antoine Cotten <hello@acotten.com>
       1 Andrew <andrew.dale.wylie@gmail.com>
       1 Alpha Chen <alpha.chen@gmail.com>
       1 Alex Tomlins <alex.tomlins@digital.cabinet-office.gov.uk>
+      1 Alex Taylor <alex@alextaylor.ca>
       1 Alexey Vasiliev <le0pard@users.noreply.github.com>
       1 284km <k.furuhashi10@gmail.com>

data/VERSION

@@ -1 +1 @@
-2.3.2
+2.5.0

data/bin/kramdown

@@ -1,5 +1,5 @@
 #!/usr/bin/env ruby
-# -*- coding: utf-8 -*-
+# -*- coding: utf-8; frozen_string_literal: true -*-
 #
 #--
 # Copyright (C) 2009-2019 Thomas Leitner <t_leitner@gmx.at>
@@ -36,7 +36,7 @@ def add_kramdown_options(opts, parsed_options, banner: [], ignore: [])
       opts.on("--#{no} ARG", type) {|v| parsed_options[n] = Kramdown::Options.parse(n, v) }
     end
 
-    definition.desc.split(/\n/).each do |line|
+    definition.desc.split("\n").each do |line|
       opts.separator opts.summary_indent + ' ' * 6 + line
     end
     opts.separator ''
@@ -78,14 +78,12 @@ OptionParser.new do |opts|
           "prefix) separated", "by commas (e.g. parser-gfm,syntax-coderay)",
           "Note: Use this option before other options!") do |exts|
     exts.each do |ext|
-      begin
-        require "kramdown-#{ext}"
-        new_options = add_kramdown_options(opts, options, banner: ["#{ext} options:"],
-                                           ignore: defined_options)
-        defined_options.concat(new_options)
-      rescue LoadError
-        $stderr.puts "Couldn't load extension #{ext}, ignoring"
-      end
+      require "kramdown-#{ext}"
+      new_options = add_kramdown_options(opts, options, banner: ["#{ext} options:"],
+                                         ignore: defined_options)
+      defined_options.concat(new_options)
+    rescue LoadError
+      $stderr.puts "Couldn't load extension #{ext}, ignoring"
     end
   end
   opts.separator ""
@@ -101,7 +99,7 @@ OptionParser.new do |opts|
     exit
   end
   opts.on("-h", "--help", "Show the help") do
-    puts opts.summarize('', 5, 72)
+    puts opts.summarize(+'', 5, 72)
     exit
   end
 
@@ -111,7 +109,12 @@ end.parse!
 
 begin
   if config_file && File.exist?(config_file)
-    config_file_options = YAML.safe_load(File.read(config_file), [Symbol])
+    config_file_options =
+      if YAML.method(:safe_load).parameters.include?(%i[key permitted_classes])
+        YAML.safe_load(File.read(config_file), permitted_classes: [Symbol])
+      else # compatibility with Psych < 3.1.0
+        YAML.safe_load(File.read(config_file), [Symbol])
+      end
     case config_file_options
     when nil # empty configuration file except perhaps YAML header and comments
       # Nothing to do

data/lib/kramdown/converter/base.rb

@@ -137,7 +137,7 @@ module Kramdown
 
       # Return the template specified by +template+.
       def self.get_template(template) # :nodoc:
-        format_ext = '.' + ::Kramdown::Utils.snake_case(self.name.split(/::/).last)
+        format_ext = '.' + ::Kramdown::Utils.snake_case(name.split("::").last)
         shipped = File.join(::Kramdown.data_dir, template + format_ext)
         if File.exist?(template)
           File.read(template)
@@ -146,7 +146,7 @@ module Kramdown
         elsif File.exist?(shipped)
           File.read(shipped)
         elsif template.start_with?('string://')
-          template.sub(/\Astring:\/\//, '')
+          template.delete_prefix("string://")
         else
           raise "The specified template file #{template} does not exist"
         end

data/lib/kramdown/converter/html.rb

@@ -100,8 +100,12 @@ module Kramdown
         figure_attr = el.attr.dup
         image_attr = el.children.first.attr.dup
 
-        figure_attr['class'] = image_attr.delete('class') if image_attr.key?('class') and not figure_attr.key?('class')
-        figure_attr['id'] = image_attr.delete('id') if image_attr.key?('id') and not figure_attr.key?('id')
+        if image_attr.key?('class') && !figure_attr.key?('class')
+          figure_attr['class'] = image_attr.delete('class')
+        end
+        if image_attr.key?('id') && !figure_attr.key?('id')
+          figure_attr['id'] = image_attr.delete('id')
+        end
 
         body = "#{' ' * (indent + @indent)}<img#{html_attributes(image_attr)} />\n" \
           "#{' ' * (indent + @indent)}<figcaption>#{image_attr['alt']}</figcaption>\n"
@@ -128,7 +132,7 @@ module Kramdown
                 when "\t" then "<span class=\"ws-tab#{suffix}\">\t</span>"
                 when " " then "<span class=\"ws-space#{suffix}\">&#8901;</span>"
                 end
-              end.join('')
+              end.join
             end
           end
           code_attr = {}
@@ -147,6 +151,13 @@ module Kramdown
         if @options[:auto_ids] && !attr['id']
           attr['id'] = generate_id(el.options[:raw_text])
         end
+
+        if @options[:header_links] && attr['id'].to_s.length > 0
+          link = Element.new(:a, nil, nil)
+          link.attr['href'] = "##{attr['id']}"
+          el.children.unshift(link)
+        end
+
         @toc << [el.options[:level], attr['id'], el.children] if attr['id'] && in_toc?(el)
         level = output_header_level(el.options[:level])
         format_as_block_html("h#{level}", attr, inner(el, indent), indent)
@@ -179,7 +190,7 @@ module Kramdown
         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.match?(/\n\Z/) ? ' ' * indent : '')
         else
           output << "\n" << res << ' ' * indent
         end
@@ -283,7 +294,7 @@ module Kramdown
         hl_opts = {}
         result = highlight_code(el.value, lang, :span, hl_opts)
         if result
-          add_syntax_highlighter_to_class_attr(attr, hl_opts[:default_lang])
+          add_syntax_highlighter_to_class_attr(attr, lang || hl_opts[:default_lang])
         else
           result = escape_html(el.value)
         end
@@ -303,9 +314,10 @@ module Kramdown
           @footnotes << [name, el.value, number, 0]
           @footnotes_by_name[name] = @footnotes.last
         end
-        "<sup id=\"fnref:#{name}#{repeat}\" role=\"doc-noteref\">" \
-          "<a href=\"#fn:#{name}\" class=\"footnote\" rel=\"footnote\">" \
-          "#{number}</a></sup>"
+        formatted_link_text = sprintf(@options[:footnote_link_text], number)
+        "<sup id=\"fnref:#{name}#{repeat}\">" \
+          "<a href=\"#fn:#{name}\" class=\"footnote\" rel=\"footnote\" role=\"doc-noteref\">" \
+          "#{formatted_link_text}</a></sup>"
       end
 
       def convert_raw(el, _indent)
@@ -340,7 +352,7 @@ module Kramdown
         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
 
@@ -378,11 +390,7 @@ module Kramdown
         end
         if @toc_code
           toc_tree = generate_toc_tree(@toc, @toc_code[0], @toc_code[1] || {})
-          text = if !toc_tree.children.empty?
-                   convert(toc_tree, 0)
-                 else
-                   ''
-                 end
+          text = toc_tree.children.empty? ? '' : convert(toc_tree, 0)
           result.sub!(/#{@toc_code.last}/, text.gsub(/\\/, "\\\\\\\\"))
         end
         result
@@ -492,7 +500,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}", 'role' => 'doc-endnote')
+          li = Element.new(:li, nil, 'id' => "fn:#{name}")
           li.children = Marshal.load(Marshal.dump(data.children))
 
           para = nil

data/lib/kramdown/converter/kramdown.rb

@@ -27,6 +27,8 @@ module Kramdown
         @footnotes = []
         @abbrevs = []
         @stack = []
+        @list_indent = @options[:list_indent]
+        @list_spacing = ' ' * (@list_indent - 2)
       end
 
       def convert(el, opts = {indent: 0})
@@ -43,7 +45,8 @@ module Kramdown
         elsif el.block? &&
             ![:li, :dd, :dt, :td, :th, :tr, :thead, :tbody, :tfoot, :blank].include?(el.type) &&
             (el.type != :html_element || @stack.last.type != :html_element) &&
-            (el.type != :p || !el.options[:transparent])
+            (el.type != :p || !el.options[:transparent]) &&
+            !([:ul, :dl, :ol].include?(el.type) && @stack.last.type == :li)
           res << "\n"
         end
         res
@@ -69,24 +72,30 @@ module Kramdown
         ""
       end
 
-      ESCAPED_CHAR_RE = /(\$\$|[\\*_`\[\]\{"'|])|^[ ]{0,3}(:)/
+      ESCAPED_CHAR_RE = /(\$\$|[\\*_`\[\]{"'|])|^ {0,3}(:)/
 
       def convert_text(el, opts)
-        if opts[:raw_text]
+        if opts[:raw_text] || (@stack.last.type == :html_element && @stack.last.options[:content_model] == :raw)
           el.value
         else
-          el.value.gsub(/\A\n/) do
+          result = el.value.gsub(/\A\n/) do
             opts[:prev] && opts[:prev].type == :br ? '' : "\n"
-          end.gsub(/\s+/, ' ').gsub(ESCAPED_CHAR_RE) { "\\#{$1 || $2}" }
+          end
+          result.gsub!(/\s+/, ' ') unless el.options[:cdata]
+          result.gsub!(ESCAPED_CHAR_RE) do
+            $1 || !opts[:prev] || opts[:prev].type == :br ? "\\#{$1 || $2}" : $&
+          end
+          result
         end
       end
 
       def convert_p(el, opts)
         w = @options[:line_width] - opts[:indent].to_s.to_i
         first, second, *rest = inner(el, opts).strip.gsub(/(.{1,#{w}})( +|$\n?)/, "\\1\n").split(/\n/)
-        first&.gsub!(/^(?:(#|>)|(\d+)\.|([+-]\s))/) { $1 || $3 ? "\\#{$1 || $3}" : "#{$2}\\." }
+        first&.gsub!(/^(?:(#|>)|(\d+)\.(\s)|([+-]\s))/) { $1 || $4 ? "\\#{$1 || $4}" : "#{$2}\\.#{$3}" }
         second&.gsub!(/^([=-]+\s*?)$/, "\\\1")
         res = [first, second, *rest].compact.join("\n") + "\n"
+        res.gsub!(/^ {0,3}:/, "\\:")
         if el.children.length == 1 && el.children.first.type == :math
           res = "\\#{res}"
         elsif res.start_with?('\$$') && res.end_with?("\\$$\n")
@@ -96,15 +105,16 @@ module Kramdown
       end
 
       def convert_codeblock(el, _opts)
-        el.value.split(/\n/).map {|l| l.empty? ? "    " : "    #{l}" }.join("\n") + "\n"
+        el.value.split("\n").map {|l| l.empty? ? "    " : "    #{l}" }.join("\n") + "\n"
       end
 
       def convert_blockquote(el, opts)
         opts[:indent] += 2
-        inner(el, opts).chomp.split(/\n/).map {|l| "> #{l}" }.join("\n") << "\n"
+        inner(el, opts).chomp.split("\n").map {|l| "> #{l}" }.join("\n") << "\n"
       end
 
       def convert_header(el, opts)
+        opts[:in_header] = true
         res = +''
         res << "#{'#' * output_header_level(el.options[:level])} #{inner(el, opts)}"
         res[-1, 1] = "\\#" if res[-1] == '#'
@@ -124,7 +134,7 @@ module Kramdown
 
       def convert_li(el, opts)
         sym, width = if @stack.last.type == :ul
-                       [+'* ', el.children.first && el.children.first.type == :codeblock ? 4 : 2]
+                       ['* ' + @list_spacing, el.children.first && el.children.first.type == :codeblock ? 4 : @list_indent]
                      else
                        ["#{opts[:index] + 1}.".ljust(4), 4]
                      end
@@ -135,13 +145,13 @@ module Kramdown
         opts[:indent] += width
         text = inner(el, opts)
         newlines = text.scan(/\n*\Z/).first
-        first, *last = text.split(/\n/)
+        first, *last = text.split("\n")
         last = last.map {|l| " " * width + l }.join("\n")
         text = (first.nil? ? "\n" : first + (last.empty? ? "" : "\n") + last + newlines)
         if el.children.first && el.children.first.type == :p && !el.children.first.options[:transparent]
           res = +"#{sym}#{text}"
           res << "^\n" if el.children.size == 1 && @stack.last.children.last == el &&
-            (@stack.last.children.any? {|c| c.children.first.type != :p } || @stack.last.children.size == 1)
+            (@stack.last.children.any? {|c| !c.children.first || c.children.first.type != :p } || @stack.last.children.size == 1)
           res
         elsif el.children.first && el.children.first.type == :codeblock
           "#{sym}\n    #{text}"
@@ -151,7 +161,7 @@ module Kramdown
       end
 
       def convert_dd(el, opts)
-        sym, width = +": ", (el.children.first && el.children.first.type == :codeblock ? 4 : 2)
+        sym, width = ": " + @list_spacing, (el.children.first && el.children.first.type == :codeblock ? 4 : @list_indent)
         if (ial = ial_for_element(el))
           sym << ial << " "
         end
@@ -159,7 +169,7 @@ module Kramdown
         opts[:indent] += width
         text = inner(el, opts)
         newlines = text.scan(/\n*\Z/).first
-        first, *last = text.split(/\n/)
+        first, *last = text.split("\n")
         last = last.map {|l| " " * width + l }.join("\n")
         text = first.to_s + (last.empty? ? "" : "\n") + last + newlines
         text.chomp! if text =~ /\n\n\Z/ && opts[:next] && opts[:next].type == :dd
@@ -182,7 +192,7 @@ module Kramdown
         result << inner(el, opts) << "\n"
       end
 
-      HTML_TAGS_WITH_BODY = ['div', 'script', 'iframe', 'textarea', 'th', 'td']
+      HTML_TAGS_WITH_BODY = %w[div script iframe textarea th td]
 
       HTML_ELEMENT_TYPES = [:entity, :text, :html_element].freeze
       private_constant :HTML_ELEMENT_TYPES
@@ -238,7 +248,7 @@ module Kramdown
 
       def convert_thead(el, opts)
         rows = inner(el, opts)
-        if opts[:alignment].all? {|a| a == :default }
+        if opts[:alignment].all?(:default)
           "#{rows}|#{'-' * 10}\n"
         else
           "#{rows}| " + opts[:alignment].map do |a|
@@ -279,8 +289,8 @@ module Kramdown
         end
       end
 
-      def convert_br(_el, _opts)
-        "  \n"
+      def convert_br(_el, opts)
+        opts[:in_header] ? "<br />" : "  \n"
       end
 
       def convert_a(el, opts)
@@ -362,7 +372,7 @@ module Kramdown
       end
 
       def convert_smart_quote(el, _opts)
-        el.value.to_s =~ /[rl]dquo/ ? "\"" : "'"
+        el.value.to_s.match?(/[rl]dquo/) ? "\"" : "'"
       end
 
       def convert_math(el, _opts)
@@ -395,7 +405,7 @@ module Kramdown
         res = +''
         @footnotes.each do |name, data|
           res << "[^#{name}]:\n"
-          res << inner(data).chomp.split(/\n/).map {|l| "    #{l}" }.join("\n") + "\n\n"
+          res << inner(data).chomp.split("\n").map {|l| "    #{l}" }.join("\n") + "\n\n"
         end
         res
       end
@@ -417,14 +427,14 @@ module Kramdown
           next if el.type == :header && k == 'id' && !v.strip.empty?
           if v.nil?
             ''
-          elsif k == 'class' && !v.empty? && !v.index(/[\.#]/)
+          elsif k == 'class' && !v.empty? && !v.index(/[.#]/)
             " " + v.split(/\s+/).map {|w| ".#{w}" }.join(" ")
           elsif k == 'id' && !v.strip.empty?
             " ##{v}"
           else
             " #{k}=\"#{v}\""
           end
-        end.compact.join('')
+        end.compact.join
         res = "toc" + (res.strip.empty? ? '' : " #{res}") if (el.type == :ul || el.type == :ol) &&
           el.options.dig(:ial, :refs)&.include?('toc')
         res = "footnotes" + (res.strip.empty? ? '' : " #{res}") if (el.type == :ul || el.type == :ol) &&

data/lib/kramdown/converter/latex.rb

@@ -59,7 +59,7 @@ module Kramdown
       end
 
       def convert_blank(_el, opts)
-        opts[:result] =~ /\n\n\Z|\A\Z/ ? "" : "\n"
+        opts[:result].match?(/\n\n\Z|\A\Z/) ? "" : "\n"
       end
 
       def convert_text(el, _opts)
@@ -153,9 +153,10 @@ module Kramdown
       end
 
       def convert_html_element(el, opts)
-        if el.value == 'i' || el.value == 'em'
+        case el.value
+        when 'i', 'em'
           "\\emph{#{inner(el, opts)}}"
-        elsif el.value == 'b' || el.value == 'strong'
+        when 'b', 'strong'
           "\\textbf{#{inner(el, opts)}}"
         else
           warning("Can't convert HTML element")
@@ -164,7 +165,7 @@ module Kramdown
       end
 
       def convert_xml_comment(el, _opts)
-        el.value.split(/\n/).map {|l| "% #{l}" }.join("\n") + "\n"
+        el.value.split("\n").map {|l| "% #{l}" }.join("\n") + "\n"
       end
 
       def convert_xml_pi(_el, _opts)
@@ -203,7 +204,7 @@ module Kramdown
       end
 
       def convert_comment(el, _opts)
-        el.value.split(/\n/).map {|l| "% #{l}" }.join("\n") << "\n"
+        el.value.split("\n").map {|l| "% #{l}" }.join("\n") << "\n"
       end
 
       def convert_br(_el, opts)
@@ -224,7 +225,7 @@ module Kramdown
 
       def convert_img(el, _opts)
         line = el.options[:location]
-        if el.attr['src'] =~ /^(https?|ftps?):\/\//
+        if el.attr['src'].match?(/^(https?|ftps?):\/\//)
           warning("Cannot include non-local image#{line ? " (line #{line})" : ''}")
           ''
         elsif !el.attr['src'].empty?
@@ -556,7 +557,7 @@ module Kramdown
       def convert_math(el, _opts)
         @data[:packages] += %w[amssymb amsmath amsthm amsfonts]
         if el.options[:category] == :block
-          if el.value =~ /\A\s*\\begin\{/
+          if el.value.match?(/\A\s*\\begin\{/)
             el.value
           else
             latex_environment('displaymath', el, el.value)
@@ -573,7 +574,7 @@ module Kramdown
 
       # Normalize the abbreviation key so that it only contains allowed ASCII character
       def normalize_abbreviation_key(key)
-        key.gsub(/\W/) {|m| m.unpack('H*').first }
+        key.gsub(/\W/) {|m| m.unpack1('H*') }
       end
 
       # Wrap the +text+ inside a LaTeX environment of type +type+. The element +el+ is passed on to
@@ -597,7 +598,7 @@ module Kramdown
 
       # Return a LaTeX comment containing all attributes as 'key="value"' pairs.
       def attribute_list(el)
-        attrs = el.attr.map {|k, v| v.nil? ? '' : " #{k}=\"#{v}\"" }.compact.sort.join('')
+        attrs = el.attr.map {|k, v| v.nil? ? '' : " #{k}=\"#{v}\"" }.compact.sort.join
         attrs = "   % #{attrs}" unless attrs.empty?
         attrs
       end
@@ -611,7 +612,7 @@ module Kramdown
         ">"  => "\\textgreater{}",
         "["  => "{[}",
         "]"  => "{]}",
-      }.merge(Hash[*("{}$%&_#".each_char.map {|c| [c, "\\#{c}"] }.flatten)]) # :nodoc:
+      }.merge(Hash[*"{}$%&_#".each_char.map {|c| [c, "\\#{c}"] }.flatten]) # :nodoc:
       ESCAPE_RE = Regexp.union(*ESCAPE_MAP.collect {|k, _v| k }) # :nodoc:
 
       # Escape the special LaTeX characters in the string +str+.

data/lib/kramdown/converter/man.rb

@@ -17,7 +17,7 @@ module Kramdown
     # man-pages(7) for information regarding the output.
     class Man < Base
 
-      def convert(el, opts = {indent: 0, result: +''}) #:nodoc:
+      def convert(el, opts = {indent: 0, result: +''}) # :nodoc:
         send("convert_#{el.type}", el, opts)
       end
 
@@ -139,7 +139,7 @@ module Kramdown
       def convert_table(el, opts)
         opts[:alignment] = el.options[:alignment].map {|a| TABLE_CELL_ALIGNMENT[a] }
         table_options = ["box"]
-        table_options << "center" if el.attr['class'] =~ /\bcenter\b/
+        table_options << "center" if el.attr['class']&.match?(/\bcenter\b/)
         opts[:result] << macro("TS") << "#{table_options.join(' ')} ;\n"
         inner(el, opts)
         opts[:result] << macro("TE") << macro("sp")
@@ -171,7 +171,7 @@ module Kramdown
         result = opts[:result]
         opts[:result] = +''
         inner(el, opts)
-        if opts[:result] =~ /\n/
+        if opts[:result].include?("\n")
           warning("Table cells using links are not supported")
           result << "\t"
         else

data/lib/kramdown/converter/math_engine/mathjax.rb

@@ -17,7 +17,7 @@ module Kramdown::Converter::MathEngine
 
     def self.call(converter, el, opts)
       value = converter.escape_html(el.value)
-      result = el.options[:category] == :block ?  "\\[#{value}\\]\n" : "\\(#{value}\\)"
+      result = el.options[:category] == :block ? "\\[#{value}\\]\n" : "\\(#{value}\\)"
       if el.attr.empty?
         result
       elsif el.options[:category] == :block

data/lib/kramdown/converter/remove_html_tags.rb

@@ -35,12 +35,13 @@ module Kramdown
         children = el.children.dup
         index = 0
         while index < children.length
-          if [:xml_pi].include?(children[index].type) ||
-              (children[index].type == :html_element && %w[style script].include?(children[index].value))
+          if children[index].type == :xml_pi ||
+              (children[index].type == :html_element && (children[index].value == 'style' ||
+                                                         children[index].value == 'script'))
             children[index..index] = []
           elsif children[index].type == :html_element &&
-            ((@options[:remove_block_html_tags] && children[index].options[:category] == :block) ||
-             (@options[:remove_span_html_tags] && children[index].options[:category] == :span))
+              ((@options[:remove_block_html_tags] && children[index].options[:category] == :block) ||
+               (@options[:remove_span_html_tags] && children[index].options[:category] == :span))
             children[index..index] = children[index].children
           else
             convert(children[index])

data/lib/kramdown/document.rb

@@ -121,7 +121,7 @@ module Kramdown
       end
     end
 
-    def inspect #:nodoc:
+    def inspect # :nodoc:
       "<KD:Document: options=#{@options.inspect} root=#{@root.inspect} warnings=#{@warnings.inspect}>"
     end
 

data/lib/kramdown/element.rb

@@ -508,7 +508,7 @@ module Kramdown
       @options ||= {}
     end
 
-    def inspect #:nodoc:
+    def inspect # :nodoc:
       "<kd:#{@type}" \
         "#{value.nil? ? '' : ' value=' + value.inspect}" \
         "#{attr.empty? ? '' : ' attr=' + attr.inspect}" \