review 5.2.0 → 5.5.0

data/lib/review/epubmaker/epubv3.rb

@@ -1,6 +1,6 @@
 # = epubv3.rb -- EPUB version 3 producer.
 #
-# Copyright (c) 2010-2017 Kenshi Muto
+# Copyright (c) 2010-2022 Kenshi Muto
 #
 # This program is free software.
 # You can distribute or modify this program under the terms of
@@ -186,7 +186,7 @@ module ReVIEW
         @tocx_contents = []
         toc = nil
         contents.each do |item|
-          next if item.media !~ /xhtml\+xml/ # skip non XHTML
+          next unless /xhtml\+xml/.match?(item.media) # skip non XHTML
 
           @tocx_contents << item
         end
@@ -210,19 +210,20 @@ module ReVIEW
         @title = h(ReVIEW::I18n.t('toctitle'))
         @language = config['language']
         @stylesheets = config['stylesheet']
-        ReVIEW::Template.generate(path: './html/layout-html5.html.erb', binding: binding)
+        ReVIEW::Template.generate(path: template_name, binding: binding)
       end
 
       # Produce EPUB file +epubfile+.
       # +work_dir+ points the directory has contents.
       # +tmpdir+ defines temporary directory.
       def produce(epubfile, work_dir, tmpdir, base_dir:)
+        @workdir = base_dir
         produce_write_common(work_dir, tmpdir)
 
         toc_file = "#{tmpdir}/OEBPS/#{config['bookname']}-toc.#{config['htmlext']}"
         File.write(toc_file, ncx(config['epubmaker']['ncxindent']))
 
-        call_hook('hook_prepack', tmpdir, base_dir: base_dir)
+        call_hook('hook_prepack', tmpdir, base_dir: @workdir)
         expoter = ReVIEW::EPUBMaker::ZipExporter.new(tmpdir, config)
         expoter.export_zip(epubfile)
       end

data/lib/review/epubmaker/producer.rb

@@ -89,8 +89,8 @@ module ReVIEW
         Dir.foreach(path) do |f|
           next if f.start_with?('.')
 
-          if f =~ /\.(#{allow_exts.join('|')})\Z/i
-            path.chop! if path =~ %r{/\Z}
+          if /\.(#{allow_exts.join('|')})\Z/i.match?(f)
+            path.chop! if %r{/\Z}.match?(path)
             if base.nil?
               @contents.push(ReVIEW::EPUBMaker::Content.new(file: "#{path}/#{f}"))
             else
@@ -115,7 +115,7 @@ module ReVIEW
 
         # use Dir to solve a path for Windows (see #1011)
         new_tmpdir = Dir[File.join(tmpdir.nil? ? Dir.mktmpdir : tmpdir)][0]
-        if epubfile !~ %r{\A/}
+        unless epubfile.start_with?('/')
           epubfile = "#{current}/#{epubfile}"
         end
 

data/lib/review/epubmaker/reviewheaderlistener.rb

@@ -41,7 +41,7 @@ module ReVIEW
       end
 
       def tag_end(name)
-        if name =~ /\Ah\d+/
+        if /\Ah\d+/.match?(name)
           if @id.present?
             @headlines.push({ 'level' => @level,
                               'id' => @id,

data/lib/review/epubmaker.rb

@@ -1,4 +1,4 @@
-# Copyright (c) 2010-2021 Kenshi Muto and Masayoshi Takahashi
+# Copyright (c) 2010-2022 Kenshi Muto and Masayoshi Takahashi
 #
 # This program is free software.
 # You can distribute or modify this program under the terms of
@@ -75,9 +75,13 @@ module ReVIEW
       cmd_config, yamlfile, exportfile = parse_opts(args)
       error! "#{yamlfile} not found." unless File.exist?(yamlfile)
 
-      @config = ReVIEW::Configure.create(maker: 'epubmaker',
-                                         yamlfile: yamlfile,
-                                         config: cmd_config)
+      begin
+        @config = ReVIEW::Configure.create(maker: 'epubmaker',
+                                           yamlfile: yamlfile,
+                                           config: cmd_config)
+      rescue ReVIEW::ConfigError => e
+        error! e.message
+      end
       @producer = ReVIEW::EPUBMaker::Producer.new(@config)
       update_log_level
       debug("Loaded yaml file (#{yamlfile}).")
@@ -118,7 +122,9 @@ module ReVIEW
       booktmpname = "#{bookname}-epub"
 
       @img_math = ReVIEW::ImgMath.new(@config)
-      unless @config.check_version(ReVIEW::VERSION, exception: false)
+      begin
+        @config.check_version(ReVIEW::VERSION, exception: true)
+      rescue ReVIEW::ConfigError => e
         warn e.message
       end
       debug("#{bookname}.epub will be created.")
@@ -225,7 +231,7 @@ module ReVIEW
       if @config['epubmaker']['verify_target_images'].present?
         @config['epubmaker']['force_include_images'].each do |file|
           unless File.exist?(file)
-            if file !~ /\Ahttps?:/
+            unless /\Ahttps?:/.match?(file)
               warn "#{file} is not found, skip."
             end
             next
@@ -255,7 +261,7 @@ module ReVIEW
 
           if FileTest.directory?(File.join(resdir, fname))
             recursive_copy_files(File.join(resdir, fname), File.join(destdir, fname), allow_exts)
-          elsif fname =~ /\.(#{allow_exts.join('|')})\Z/i
+          elsif /\.(#{allow_exts.join('|')})\Z/i.match?(fname)
             FileUtils.mkdir_p(destdir)
             debug("Copy #{resdir}/#{fname} to the temporary directory.")
             FileUtils.cp(File.join(resdir, fname), destdir, preserve: true)
@@ -313,15 +319,25 @@ module ReVIEW
       File.open(File.join(basetmpdir, htmlfile), 'w') do |f|
         @part_number = part.number
         @part_title = part.name.strip
-        @body = ReVIEW::Template.generate(path: 'html/_part_body.html.erb', binding: binding)
-
+        @body = ReVIEW::Template.generate(path: template_name(localfile: '_part_body.html.erb', systemfile: 'html/_part_body.html.erb'), binding: binding)
         @language = @producer.config['language']
         @stylesheets = @producer.config['stylesheet']
         f.write ReVIEW::Template.generate(path: template_name, binding: binding)
       end
     end
 
-    def template_name
+    def template_name(localfile: 'layout.html.erb', systemfile: nil)
+      if @basedir
+        layoutfile = File.join(@basedir, 'layouts', localfile)
+        if File.exist?(layoutfile)
+          return layoutfile
+        end
+      end
+
+      if systemfile
+        return systemfile
+      end
+
       if @producer.config['htmlversion'].to_i == 5
         './html/layout-html5.html.erb'
       else
@@ -374,7 +390,7 @@ module ReVIEW
 
       if @config['params'].present?
         warn %Q('params:' in config.yml is obsoleted.)
-        if @config['params'] =~ /stylesheet=/
+        if /stylesheet=/.match?(@config['params'])
           warn %Q(stylesheets should be defined in 'stylesheet:', not in 'params:')
         end
       end
@@ -382,7 +398,7 @@ module ReVIEW
         @converter.convert(filename, File.join(basetmpdir, htmlfile))
         write_info_body(basetmpdir, id, htmlfile, ispart, chaptype)
         remove_hidden_title(basetmpdir, htmlfile)
-      rescue => e
+      rescue StandardError => e
         @compile_errors = true
         error "compile error in #{filename} (#{e.class})"
         error e.message
@@ -434,11 +450,11 @@ module ReVIEW
       end
 
       properties = detect_properties(path)
-      if properties.present?
-        prop_str = ',properties=' + properties.join(' ')
-      else
-        prop_str = ''
-      end
+      prop_str = if properties.present?
+                   ',properties=' + properties.join(' ')
+                 else
+                   ''
+                 end
       first = true
       headlines.each do |headline|
         if ispart.present? && headline['level'] == 1
@@ -543,25 +559,12 @@ module ReVIEW
     end
 
     def build_titlepage(basetmpdir, htmlfile)
-      # TODO: should be created via epubcommon
       @title = h(@config.name_of('booktitle'))
       File.open(File.join(basetmpdir, htmlfile), 'w') do |f|
-        @body = ''
-        @body << %Q(<div class="titlepage">\n)
-        @body << %Q(<h1 class="tp-title">#{h(@config.name_of('booktitle'))}</h1>\n)
-        if @config['subtitle']
-          @body << %Q(<h2 class="tp-subtitle">#{h(@config.name_of('subtitle'))}</h2>\n)
-        end
-        if @config['aut']
-          @body << %Q(<h2 class="tp-author">#{h(@config.names_of('aut').join(ReVIEW::I18n.t('names_splitter')))}</h2>\n)
-        end
-        if @config['pbl']
-          @body << %Q(<h3 class="tp-publisher">#{h(@config.names_of('pbl').join(ReVIEW::I18n.t('names_splitter')))}</h3>\n)
-        end
-        @body << '</div>'
-
+        @body = ReVIEW::Template.generate(path: template_name(localfile: '_titlepage.html.erb', systemfile: 'html/_titlepage.html.erb'), binding: binding)
         @language = @producer.config['language']
         @stylesheets = @producer.config['stylesheet']
+
         f.write ReVIEW::Template.generate(path: template_name, binding: binding)
       end
     end

data/lib/review/extentions/string.rb

@@ -1,4 +1,4 @@
-if defined?(Encoding) && Encoding.respond_to?('default_external') &&
+if defined?(Encoding) && Encoding.respond_to?(:default_external) &&
    Encoding.default_external != Encoding::UTF_8
   Encoding.default_external = 'UTF-8'
 end

data/lib/review/htmlbuilder.rb

@@ -1,4 +1,4 @@
-# Copyright (c) 2008-2021 Minero Aoki, Kenshi Muto, Masayoshi Takahashi,
+# Copyright (c) 2008-2022 Minero Aoki, Kenshi Muto, Masayoshi Takahashi,
 #                         KADO Masanori
 #               2002-2007 Minero Aoki
 #
@@ -67,11 +67,11 @@ module ReVIEW
         htmldir = 'html'
         localfilename = 'layout.html.erb'
       end
-      if @book.htmlversion == 5
-        htmlfilename = File.join(htmldir, 'layout-html5.html.erb')
-      else
-        htmlfilename = File.join(htmldir, 'layout-xhtml1.html.erb')
-      end
+      htmlfilename = if @book.htmlversion == 5
+                       File.join(htmldir, 'layout-html5.html.erb')
+                     else
+                       File.join(htmldir, 'layout-xhtml1.html.erb')
+                     end
 
       layout_file = File.join(@book.basedir, 'layouts', localfilename)
       if !File.exist?(layout_file) && File.exist?(File.join(@book.basedir, 'layouts', 'layout.erb'))
@@ -111,6 +111,8 @@ module ReVIEW
     end
 
     def result
+      check_printendnotes
+
       # flush all `</section>`
       if use_section?
         print close_sections
@@ -825,6 +827,22 @@ module ReVIEW
       end
     end
 
+    def endnote_begin
+      puts %Q(<div class="endnotes">)
+    end
+
+    def endnote_end
+      puts %Q(</div>)
+    end
+
+    def endnote_item(id)
+      back = ''
+      if @book.config['epubmaker'] && @book.config['epubmaker']['back_footnote']
+        back = %Q(<a href="#endnoteb-#{normalize_id(id)}">#{I18n.t('html_footnote_backmark')}</a>)
+      end
+      puts %Q(<div class="endnote" id="endnote-#{normalize_id(id)}"><p class="endnote">#{back}#{I18n.t('html_endnote_textmark', @chapter.endnote(id).number)}#{compile_inline(@chapter.endnote(id).content)}</p></div>)
+    end
+
     def indepimage(lines, id, caption = '', metric = nil)
       metrics = parse_metric('html', metric)
       caption = '' unless caption.present?
@@ -843,7 +861,7 @@ EOS
       end
       begin
         puts %Q(<img src="#{@chapter.image(id).path.sub(%r{\A\./}, '')}" alt="#{escape(compile_inline(caption))}"#{metrics} />)
-      rescue
+      rescue StandardError
         warn "image not bound: #{id}", location: location
         if lines
           puts %Q(<pre class="dummyimage">)
@@ -942,6 +960,12 @@ EOS
       app_error "unknown footnote: #{id}"
     end
 
+    def inline_endnote(id)
+      %Q(<a id="endnoteb-#{normalize_id(id)}" href="#endnote-#{normalize_id(id)}" class="noteref" epub:type="noteref">#{I18n.t('html_endnote_refmark', @chapter.endnote(id).number)}</a>)
+    rescue KeyError
+      app_error "unknown endnote: #{id}"
+    end
+
     def compile_ruby(base, ruby)
       if @book.htmlversion == 5
         %Q(<ruby>#{escape(base)}<rp>#{I18n.t('ruby_prefix')}</rp><rt>#{escape(ruby)}</rt><rp>#{I18n.t('ruby_postfix')}</rp></ruby>)
@@ -953,8 +977,9 @@ EOS
     def compile_kw(word, alt)
       %Q(<b class="kw">) +
         if alt
-        then escape(word + " (#{alt.strip})")
-        else escape(word)
+          escape(word + " (#{alt.strip})")
+        else
+          escape(word)
         end +
         "</b><!-- IDX:#{escape_comment(escape(word))} -->"
     end
@@ -1072,11 +1097,11 @@ EOS
 
     def inline_hd_chap(chap, id)
       n = chap.headline_index.number(id)
-      if n.present? && chap.number && over_secnolevel?(n)
-        str = I18n.t('hd_quote', [n, compile_inline(chap.headline(id).caption)])
-      else
-        str = I18n.t('hd_quote_without_number', compile_inline(chap.headline(id).caption))
-      end
+      str = if n.present? && chap.number && over_secnolevel?(n)
+              I18n.t('hd_quote', [n, compile_inline(chap.headline(id).caption)])
+            else
+              I18n.t('hd_quote_without_number', compile_inline(chap.headline(id).caption))
+            end
       if @book.config['chapterlink']
         anchor = 'h' + n.tr('.', '-')
         %Q(<a href="#{chap.id}#{extname}##{anchor}">#{str}</a>)
@@ -1087,6 +1112,31 @@ EOS
       app_error "unknown headline: #{id}"
     end
 
+    def inline_sec(id)
+      if @book.config['chapterlink']
+        chap, id2 = extract_chapter_id(id)
+        n = chap.headline_index.number(id2)
+        anchor = 'h' + n.tr('.', '-')
+        %Q(<a href="#{chap.id}#{extname}##{anchor}">#{super(id)}</a>)
+      else
+        super(id)
+      end
+    rescue KeyError
+      app_error "unknown headline: #{id}"
+    end
+
+    def inline_sectitle(id)
+      if @book.config['chapterlink']
+        chap, id2 = extract_chapter_id(id)
+        anchor = 'h' + chap.headline_index.number(id2).tr('.', '-')
+        %Q(<a href="#{chap.id}#{extname}##{anchor}">#{super(id)}</a>)
+      else
+        super(id)
+      end
+    rescue KeyError
+      app_error "unknown headline: #{id}"
+    end
+
     def column_label(id, chapter = @chapter)
       num = chapter.column(id).number
       "column-#{num}"
@@ -1226,7 +1276,7 @@ EOS
     def inline_icon(id)
       begin
         %Q(<img src="#{@chapter.image(id).path.sub(%r{\A\./}, '')}" alt="[#{id}]" />)
-      rescue
+      rescue StandardError
         warn "image not bound: #{id}", location: location
         %Q(<pre>missing image: #{id}</pre>)
       end

data/lib/review/htmlutils.rb

@@ -9,7 +9,7 @@
 
 begin
   require 'cgi/escape'
-rescue
+rescue StandardError
   require 'cgi/util'
 end
 
@@ -60,13 +60,13 @@ module ReVIEW
     def highlight_pygments(ops)
       body = ops[:body] || ''
       format = ops[:format] || ''
-      if ops[:lexer].present?
-        lexer = ops[:lexer]
-      elsif @book.config['highlight'] && @book.config['highlight']['lang']
-        lexer = @book.config['highlight']['lang'] # default setting
-      else
-        lexer = 'text'
-      end
+      lexer = if ops[:lexer].present?
+                ops[:lexer]
+              elsif @book.config['highlight'] && @book.config['highlight']['lang']
+                @book.config['highlight']['lang'] # default setting
+              else
+                'text'
+              end
       options = { nowrap: true, noclasses: true }
       if ops[:linenum]
         options[:nowrap] = false
@@ -93,13 +93,13 @@ module ReVIEW
 
     def highlight_rouge(ops)
       body = ops[:body] || ''
-      if ops[:lexer].present?
-        lexer = ops[:lexer]
-      elsif @book.config['highlight'] && @book.config['highlight']['lang']
-        lexer = @book.config['highlight']['lang'] # default setting
-      else
-        lexer = 'text'
-      end
+      lexer = if ops[:lexer].present?
+                ops[:lexer]
+              elsif @book.config['highlight'] && @book.config['highlight']['lang']
+                @book.config['highlight']['lang'] # default setting
+              else
+                'text'
+              end
       # format = ops[:format] || ''
 
       first_line_num = 1 ## default
@@ -129,9 +129,9 @@ module ReVIEW
     end
 
     def normalize_id(id)
-      if id =~ /\A[a-z][a-z0-9_.-]*\Z/i
+      if /\A[a-z][a-z0-9_.-]*\Z/i.match?(id)
         id
-      elsif id =~ /\A[0-9_.-][a-z0-9_.-]*\Z/i
+      elsif /\A[0-9_.-][a-z0-9_.-]*\Z/i.match?(id)
         "id_#{id}" # dummy prefix
       else
         "id_#{CGI.escape(id.gsub('_', '__')).tr('%', '_').tr('+', '-')}" # escape all

data/lib/review/i18n.rb

@@ -75,11 +75,11 @@ module ReVIEW
     end
 
     def load_file(path)
-      @store = YAML.load_file(path)
+      @store = YAMLLoader.safe_load_file(path)
     end
 
     def update_localefile(path)
-      user_i18n = YAML.load_file(path)
+      user_i18n = YAMLLoader.safe_load_file(path)
       locale = user_i18n['locale']
       if locale
         user_i18n.delete('locale')
@@ -171,7 +171,7 @@ module ReVIEW
       args_matched = (frmt.count('%') <= args.size)
       frmt.gsub!('##', '%%')
       args_matched ? (frmt % args) : frmt
-    rescue
+    rescue StandardError
       str
     end
   end

data/lib/review/i18n.yml

@@ -32,6 +32,8 @@ ja:
   html_footnote_refmark: "*%s"
   html_footnote_textmark: "[*%s] "
   html_footnote_backmark: "⏎"
+  html_endnote_refmark: "(%s)"
+  html_endnote_textmark: "(%s) "
   aut: "著　者"
   csl: "監　修"
   dsr: "デザイン"
@@ -120,6 +122,8 @@ en:
   html_footnote_refmark: "*%s"
   html_footnote_textmark: "[*%s] "
   html_footnote_backmark: "⏎"
+  html_endnote_refmark: "(%s)"
+  html_endnote_textmark: "(%s) "
   aut: "Author"
   csl: "Consultant"
   dsr: "Design"
@@ -188,6 +192,8 @@ zh-TW:
   html_footnote_refmark: "*%s"
   html_footnote_textmark: "[*%s] "
   html_footnote_backmark: "⏎"
+  html_endnote_refmark: "(%s)"
+  html_endnote_textmark: "(%s) "
   aut: "著作人"
   csl: "監　修"
   dsr: "美術編輯"

data/lib/review/idgxmlbuilder.rb

@@ -1,4 +1,4 @@
-# Copyright (c) 2008-2020 Minero Aoki, Kenshi Muto
+# Copyright (c) 2008-2022 Minero Aoki, Kenshi Muto
 #               2002-2007 Minero Aoki
 #
 # This program is free software.
@@ -78,13 +78,15 @@ module ReVIEW
     end
 
     def result
+      check_printendnotes
+
       s = ''
       if @secttags
         s += '</sect4>' if @subsubsubsection > 0
         s += '</sect3>' if @subsubsection > 0
         s += '</sect2>' if @subsection > 0
         s += '</sect>' if @section > 0
-        s += '</chapter>' if @chapter.number > 0
+        s += '</chapter>'
       end
       solve_nest(@output.string) + s + "</#{@rootelement}>\n"
     end
@@ -455,11 +457,11 @@ module ReVIEW
       caption_str = nil
       if id
         puts '<equationblock>'
-        if get_chap.nil?
-          caption_str = %Q(<caption>#{I18n.t('equation')}#{I18n.t('format_number_without_chapter', [@chapter.equation(id).number])}#{I18n.t('caption_prefix_idgxml')}#{compile_inline(caption)}</caption>)
-        else
-          caption_str = %Q(<caption>#{I18n.t('equation')}#{I18n.t('format_number', [get_chap, @chapter.equation(id).number])}#{I18n.t('caption_prefix_idgxml')}#{compile_inline(caption)}</caption>)
-        end
+        caption_str = if get_chap.nil?
+                        %Q(<caption>#{I18n.t('equation')}#{I18n.t('format_number_without_chapter', [@chapter.equation(id).number])}#{I18n.t('caption_prefix_idgxml')}#{compile_inline(caption)}</caption>)
+                      else
+                        %Q(<caption>#{I18n.t('equation')}#{I18n.t('format_number', [get_chap, @chapter.equation(id).number])}#{I18n.t('caption_prefix_idgxml')}#{compile_inline(caption)}</caption>)
+                      end
         puts caption_str if caption_top?('equation')
       end
 
@@ -478,7 +480,7 @@ module ReVIEW
     def table(lines, id = nil, caption = nil)
       @tablewidth = nil
       if @book.config['tableopt']
-        @tablewidth = @book.config['tableopt'].split(',')[0].to_f / @book.config['pt_to_mm_unit'].to_f
+        @tablewidth = @book.config['tableopt'].split(',')[0].to_f / @book.config['pt_to_mm_unit'].to_f # rubocop:disable Style/FloatDivision
       end
       @col = 0
 
@@ -513,7 +515,7 @@ module ReVIEW
       sepidx = nil
       rows = []
       lines.each_with_index do |line, idx|
-        if /\A[=\-]{12}/ =~ line
+        if /\A[=\-]{12}/.match?(line)
           sepidx ||= idx
           next
         end
@@ -538,7 +540,7 @@ module ReVIEW
           cellwidth = @tsize.split(/\s*,\s*/)
           totallength = 0
           cellwidth.size.times do |n|
-            cellwidth[n] = cellwidth[n].to_f / @book.config['pt_to_mm_unit'].to_f
+            cellwidth[n] = cellwidth[n].to_f / @book.config['pt_to_mm_unit'].to_f # rubocop:disable Style/FloatDivision
             totallength += cellwidth[n]
             warn "total length exceeds limit for table: #{@table_id}", location: location if totallength > @tablewidth
           end
@@ -660,6 +662,24 @@ module ReVIEW
       app_error "unknown footnote: #{id}"
     end
 
+    def inline_endnote(id)
+      %Q(<span type='endnoteref' idref='endnoteb-#{normalize_id(id)}'>(#{@chapter.endnote(id).number})</span>)
+    rescue KeyError
+      app_error "unknown endnote: #{id}"
+    end
+
+    def endnote_begin
+      puts '<endnotes>'
+    end
+
+    def endnote_end
+      puts '</endnotes>'
+    end
+
+    def endnote_item(id)
+      puts %Q(<endnote id='endnoteb-#{normalize_id(id)}'><span type='endnotenumber'>(#{@chapter.endnote(id).number})</span>\t#{compile_inline(@chapter.endnote(id).content)}</endnote>)
+    end
+
     def compile_ruby(base, ruby)
       %Q(<GroupRuby><aid:ruby xmlns:aid="http://ns.adobe.com/AdobeInDesign/3.0/"><aid:rb>#{escape(base.strip)}</aid:rb><aid:rt>#{escape(ruby.strip)}</aid:rt></aid:ruby></GroupRuby>)
     end
@@ -667,8 +687,9 @@ module ReVIEW
     def compile_kw(word, alt)
       '<keyword>' +
         if alt
-        then escape("#{word}（#{alt.strip}）")
-        else escape(word)
+          escape("#{word}（#{alt.strip}）")
+        else
+          escape(word)
         end +
         '</keyword>' +
         %Q(<index value="#{escape(word)}" />) +
@@ -704,15 +725,15 @@ module ReVIEW
     end
 
     def inline_maru(str)
-      if str =~ /\A\d+\Z/
+      if /\A\d+\Z/.match?(str)
         sprintf('&#x%x;', 9311 + str.to_i)
-      elsif str =~ /\A[A-Z]\Z/
+      elsif /\A[A-Z]\Z/.match?(str)
         begin
           sprintf('&#x%x;', 9398 + str.codepoints.to_a[0] - 65)
         rescue NoMethodError
           sprintf('&#x%x;', 9398 + str[0] - 65)
         end
-      elsif str =~ /\A[a-z]\Z/
+      elsif /\A[a-z]\Z/.match?(str)
         begin
           sprintf('&#x%x;', 9392 + str.codepoints.to_a[0] - 65)
         rescue NoMethodError
@@ -780,7 +801,7 @@ module ReVIEW
     def inline_icon(id)
       begin
         %Q(<Image href="file://#{@chapter.image(id).path.sub(%r{\A\./}, '')}" type="inline" />)
-      rescue
+      rescue StandardError
         warn "image not bound: #{id}", location: location
         ''
       end
@@ -1122,16 +1143,16 @@ module ReVIEW
     def indepimage(_lines, id, caption = nil, metric = nil)
       metrics = parse_metric('idgxml', metric)
       puts '<img>'
-      if caption_top?('image')
-        puts %Q(<caption>#{compile_inline(caption)}</caption>) if caption.present?
+      if caption_top?('image') && caption.present?
+        puts %Q(<caption>#{compile_inline(caption)}</caption>)
       end
       begin
         puts %Q(<Image href="file://#{@chapter.image(id).path.sub(%r{\A\./}, '')}"#{metrics} />)
-      rescue
+      rescue StandardError
         warn %Q(image not bound: #{id}), location: location
       end
-      unless caption_top?('image')
-        puts %Q(<caption>#{compile_inline(caption)}</caption>) if caption.present?
+      if !caption_top?('image') && caption.present?
+        puts %Q(<caption>#{compile_inline(caption)}</caption>)
       end
       puts '</img>'
     end