review 2.3.0 → 2.4.0

data/lib/review/htmltoc.rb

@@ -1,16 +1,14 @@
 module ReVIEW
   class HTMLToc
     def initialize(basedir)
-      @tochtmltxt = "toc-html.txt"
+      @tochtmltxt = 'toc-html.txt'
       @basedir = basedir
     end
 
     def add_item(level, filename, title, args)
       args_str = encode_args(args)
       line = [level, filename, title, args_str].join("\t")
-      File.open(tocfilename, "a") do |f|
-        f.write "#{line}\n"
-      end
+      File.open(tocfilename, 'a') { |f| f.write "#{line}\n" }
     end
 
     def each_item
@@ -30,15 +28,14 @@ module ReVIEW
     def decode_args(args_str)
       args = {}
       args_str.split(/,\s*/).each do |pair|
-        key, val = pair.split("=")
+        key, val = pair.split('=')
         args[key.to_sym] = val
       end
       args
     end
 
     def encode_args(args)
-      args.delete_if {|k,v| v.nil? }.map{|k,v| "#{k}=#{v}"}.join(",")
+      args.delete_if { |_k, v| v.nil? }.map { |k, v| "#{k}=#{v}" }.join(',')
     end
   end
 end
-

data/lib/review/htmlutils.rb

@@ -1,7 +1,6 @@
 #
-# $Id: htmlutils.rb 2227 2006-05-13 00:09:08Z aamine $
-#
-# Copyright (c) 2002-2006 Minero Aoki
+# Copyright (c) 2006-2017 Minero Aoki, Kenshi Muto
+#               2002-2006 Minero Aoki
 #
 # This program is free software.
 # You can distribute or modify this program under the terms of
@@ -10,18 +9,17 @@
 
 require 'cgi/util'
 module ReVIEW
-
   module HTMLUtils
     ESC = {
       '&' => '&',
       '<' => '&lt;',
       '>' => '&gt;',
       '"' => '&quot;'
-    }
+    } # .freeze
 
     def escape_html(str)
       t = ESC
-      str.gsub(/[&"<>]/) {|c| t[c] }
+      str.gsub(/[&"<>]/) { |c| t[c] }
     end
 
     alias_method :escape, :escape_html
@@ -35,7 +33,7 @@ module ReVIEW
     alias_method :unescape, :unescape_html
 
     def strip_html(str)
-      str.gsub(/<\/?[^>]*>/, "")
+      str.gsub(%r{</?[^>]*>}, '')
     end
 
     def escape_comment(str)
@@ -43,40 +41,40 @@ module ReVIEW
     end
 
     def highlight?
-      @book.config["highlight"] &&
-        @book.config["highlight"]["html"]
+      @book.config['highlight'] &&
+        @book.config['highlight']['html']
     end
 
     def highlight(ops)
-      if @book.config["pygments"].present?
-        raise ReVIEW::ConfigError, "'pygments:' in config.yml is obsoleted."
+      if @book.config['pygments'].present?
+        raise ReVIEW::ConfigError, %Q('pygments:' in config.yml is obsoleted.)
       end
-      return ops[:body].to_s if !highlight?
+      return ops[:body].to_s unless highlight?
 
-      if @book.config["highlight"]["html"] == "pygments"
+      if @book.config['highlight']['html'] == 'pygments'
         highlight_pygments(ops)
-      elsif @book.config["highlight"]["html"] == "rouge"
+      elsif @book.config['highlight']['html'] == 'rouge'
         highlight_rouge(ops)
       else
-        raise ReVIEW::ConfigError, "unknown highlight method #{@book.config["highlight"]["html"]} in config.yml."
+        raise ReVIEW::ConfigError, "unknown highlight method #{@book.config['highlight']['html']} in config.yml."
       end
     end
 
     def highlight_pygments(ops)
       body = ops[:body] || ''
-      if @book.config["highlight"] && @book.config["highlight"]["lang"]
-        lexer = @book.config["highlight"]["lang"] # default setting
+      if @book.config['highlight'] && @book.config['highlight']['lang']
+        lexer = @book.config['highlight']['lang'] # default setting
       else
         lexer = 'text'
       end
       lexer = ops[:lexer] if ops[:lexer].present?
       format = ops[:format] || ''
-      options = {:nowrap => true, :noclasses => true}
+      options = { nowrap: true, noclasses: true }
       if ops[:linenum]
         options[:nowrap] = false
         options[:linenos] = 'inline'
       end
-      if ops[:options] && ops[:options].kind_of?(Hash)
+      if ops[:options] && ops[:options].is_a?(Hash)
         options.merge!(ops[:options])
       end
 
@@ -84,9 +82,9 @@ module ReVIEW
         require 'pygments'
         begin
           Pygments.highlight(unescape_html(body),
-                             :options => options,
-                             :formatter => format,
-                             :lexer => lexer)
+                             options: options,
+                             formatter: format,
+                             lexer: lexer)
         rescue MentosError
           body
         end
@@ -99,12 +97,12 @@ module ReVIEW
       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
+      elsif @book.config['highlight'] && @book.config['highlight']['lang']
+        lexer = @book.config['highlight']['lang'] # default setting
       else
         lexer = 'text'
       end
-      format = ops[:format] || ''
+      # format = ops[:format] || ''
 
       first_line_num = 1 ## default
       if ops[:options] && ops[:options][:linenostart]
@@ -115,11 +113,11 @@ module ReVIEW
       lexer = Rouge::Lexer.find(lexer)
       raise "unknown lexer #{lexer}" unless lexer
 
-      formatter = Rouge::Formatters::HTML.new(:css_class => 'highlight')
+      formatter = Rouge::Formatters::HTML.new(css_class: 'highlight')
       if ops[:linenum]
         formatter = Rouge::Formatters::HTMLTable.new(formatter,
-                                                     :table_class => 'highlight rouge-table',
-                                                     :start_line => first_line_num)
+                                                     table_class: 'highlight rouge-table',
+                                                     start_line: first_line_num)
       end
       raise "unknown formatter #{formatter}" unless formatter
 
@@ -129,13 +127,12 @@ module ReVIEW
 
     def normalize_id(id)
       if id =~ /\A[a-z][a-z0-9_.-]*\Z/i
-        return id
+        id
       elsif id =~ /\A[0-9_.-][a-z0-9_.-]*\Z/i
-        return "id_#{id}" # dummy prefix
+        "id_#{id}" # dummy prefix
       else
-        return "id_#{CGI.escape(id.gsub("_", "__")).gsub("%", "_").gsub("+", "-")}" # escape all
+        "id_#{CGI.escape(id.gsub('_', '__')).gsub('%', '_').gsub('+', '-')}" # escape all
       end
     end
   end
-
 end # module ReVIEW

data/lib/review/i18n.rb

@@ -1,20 +1,19 @@
-# -*- coding: utf-8 -*-
 require 'yaml'
 
 module ReVIEW
   class I18n
-    ALPHA_U = %w[0 A B C D E F G H I J K L M N O P Q R S T U V W X Y Z]
-    ALPHA_L = %w[0 a b c d e f g h i j k l m n o p q r s t u v w x y z]
-    ROMAN_U = %w[0 I II III IV V VI VII VIII IX X XI XII XIII XIV XV XVI XVII XVIII XIX XX XXI XXII XXIII XXIV XXV XXVI XXVII]
-    ROMAN_L = %w[0 i ii iii iv v vi vii viii ix x xi xii xiii xiv xv xvi xvii xviii xix xx xxi xxii xxiii xxiv xxv xxvi xxvii]
-    ALPHA_UW = %w[０ Ａ Ｂ Ｃ Ｄ Ｅ Ｆ Ｇ Ｈ Ｉ Ｊ Ｋ Ｌ Ｍ Ｎ Ｏ Ｐ Ｑ Ｒ Ｓ Ｔ Ｕ Ｖ Ｗ Ｘ Ｙ Ｚ]
-    ALPHA_LW = %w[０ ａ ｂ ｃ ｄ ｅ ｆ ｇ ｈ ｉ ｊ ｋ ｌ ｍ ｎ ｏ ｐ ｑ ｒ ｓ ｔ ｕ ｖ ｗ ｘ ｙ ｚ]
-    ROMAN_UW = %w[０ Ⅰ Ⅱ Ⅲ Ⅳ Ｖ Ⅵ Ⅶ Ⅷ Ⅸ Ｘ Ⅺ Ⅻ]
-    ARABIC_UW = %w[〇 １ ２ ３ ４ ５ ６ ７ ８ ９ １０ １１ １２ １３ １４ １５ １６ １７ １８ １９ ２０ ２１ ２２ ２３ ２４ ２５ ２６ ２７]
-    ARABIC_LW = %w[〇 １ ２ ３ ４ ５ ６ ７ ８ ９ 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27]
-    JAPAN = %w[〇 一 二 三 四 五 六 七 八 九 十 十一 十二 十三 十四 十五 十六 十七 十八 十九 二十 二十一 二十二 二十三 二十四 二十五 二十六 二十七]
-
-    def self.setup(locale="ja", ymlfile = "locale.yml")
+    ALPHA_U = %w[0 A B C D E F G H I J K L M N O P Q R S T U V W X Y Z].freeze
+    ALPHA_L = %w[0 a b c d e f g h i j k l m n o p q r s t u v w x y z].freeze
+    ROMAN_U = %w[0 I II III IV V VI VII VIII IX X XI XII XIII XIV XV XVI XVII XVIII XIX XX XXI XXII XXIII XXIV XXV XXVI XXVII].freeze
+    ROMAN_L = %w[0 i ii iii iv v vi vii viii ix x xi xii xiii xiv xv xvi xvii xviii xix xx xxi xxii xxiii xxiv xxv xxvi xxvii].freeze
+    ALPHA_UW = %w[０ Ａ Ｂ Ｃ Ｄ Ｅ Ｆ Ｇ Ｈ Ｉ Ｊ Ｋ Ｌ Ｍ Ｎ Ｏ Ｐ Ｑ Ｒ Ｓ Ｔ Ｕ Ｖ Ｗ Ｘ Ｙ Ｚ].freeze
+    ALPHA_LW = %w[０ ａ ｂ ｃ ｄ ｅ ｆ ｇ ｈ ｉ ｊ ｋ ｌ ｍ ｎ ｏ ｐ ｑ ｒ ｓ ｔ ｕ ｖ ｗ ｘ ｙ ｚ].freeze
+    ROMAN_UW = %w[０ Ⅰ Ⅱ Ⅲ Ⅳ Ｖ Ⅵ Ⅶ Ⅷ Ⅸ Ｘ Ⅺ Ⅻ].freeze
+    ARABIC_UW = %w[〇 １ ２ ３ ４ ５ ６ ７ ８ ９ １０ １１ １２ １３ １４ １５ １６ １７ １８ １９ ２０ ２１ ２２ ２３ ２４ ２５ ２６ ２７].freeze
+    ARABIC_LW = %w[〇 １ ２ ３ ４ ５ ６ ７ ８ ９ 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27].freeze
+    JAPAN = %w[〇 一 二 三 四 五 六 七 八 九 十 十一 十二 十三 十四 十五 十六 十七 十八 十九 二十 二十一 二十二 二十三 二十四 二十五 二十六 二十七].freeze
+
+    def self.setup(locale = 'ja', ymlfile = 'locale.yml')
       @i18n = ReVIEW::I18n.new(locale)
 
       lfile = nil
@@ -22,18 +21,14 @@ module ReVIEW
         lfile = File.expand_path(ymlfile, Dir.pwd)
 
         # backward compatibility
-        if !File.exist?(lfile) && (ymlfile == "locale.yml") && File.exist?(File.expand_path("locale.yaml", Dir.pwd))
-          raise ReVIEW::ConfigError, "locale.yaml is obsoleted.  Please use locale.yml."
-        end
+        raise ReVIEW::ConfigError, 'locale.yaml is obsoleted. Please use locale.yml.' if !File.exist?(lfile) && (ymlfile == 'locale.yml') && File.exist?(File.expand_path('locale.yaml', Dir.pwd))
       end
 
-      if lfile && File.file?(lfile)
-        @i18n.update_localefile(lfile)
-      end
+      @i18n.update_localefile(lfile) if lfile && File.file?(lfile)
     end
 
-    def self.i18n(*args)
-      raise NotImplementedError, "I18n.i18n is obsoleted. Please use I18n.setup(locale, [ymlfile])"
+    def self.i18n(*_args)
+      raise NotImplementedError, 'I18n.i18n is obsoleted. Please use I18n.setup(locale, [ymlfile])'
     end
 
     def self.t(str, args = nil)
@@ -68,7 +63,7 @@ module ReVIEW
     end
 
     def load_default
-      load_file(File.expand_path("i18n.yml", File.dirname(__FILE__)))
+      load_file(File.expand_path('i18n.yml', File.dirname(__FILE__)))
     end
 
     def load_file(path)
@@ -77,9 +72,9 @@ module ReVIEW
 
     def update_localefile(path)
       user_i18n = YAML.load_file(path)
-      locale = user_i18n["locale"]
+      locale = user_i18n['locale']
       if locale
-        user_i18n.delete("locale")
+        user_i18n.delete('locale')
         if @store[locale]
           @store[locale].merge!(user_i18n)
         else
@@ -87,7 +82,7 @@ module ReVIEW
         end
       else
         user_i18n.each do |key, values|
-          raise KeyError, "Invalid locale file: #{path}" unless values.kind_of? Hash
+          raise KeyError, "Invalid locale file: #{path}" unless values.is_a? Hash
           @store[key].merge!(values)
         end
       end
@@ -111,56 +106,50 @@ module ReVIEW
       frmt = @store[@locale][str].dup
       frmt.gsub!('%%', '##')
 
-      if !args.is_a?(Array)
-        if args.nil? && frmt !~ /\%/
-          args = []
-        else
-          args = [args]
-        end
+      unless args.is_a?(Array)
+        args = args.nil? && frmt !~ /\%/ ? [] : [args]
       end
 
       percents = frmt.scan(/%[A-Za-z]{1,3}/)
       remove_args = []
       percents.each_with_index do |i, idx|
         case i
-        when "%pA"
+        when '%pA'
           frmt.sub!(i, ALPHA_U[args[idx]])
           remove_args << idx
-        when "%pa"
+        when '%pa'
           frmt.sub!(i, ALPHA_L[args[idx]])
           remove_args << idx
-        when "%pAW"
+        when '%pAW'
           frmt.sub!(i, ALPHA_UW[args[idx]])
           remove_args << idx
-        when "%paW"
+        when '%paW'
           frmt.sub!(i, ALPHA_LW[args[idx]])
           remove_args << idx
-        when "%pR"
+        when '%pR'
           frmt.sub!(i, ROMAN_U[args[idx]])
           remove_args << idx
-        when "%pr"
+        when '%pr'
           frmt.sub!(i, ROMAN_L[args[idx]])
           remove_args << idx
-        when "%pRW"
+        when '%pRW'
           frmt.sub!(i, ROMAN_UW[args[idx]])
           remove_args << idx
-        when "%pJ"
+        when '%pJ'
           frmt.sub!(i, JAPAN[args[idx]])
           remove_args << idx
-        when "%pdW"
+        when '%pdW'
           frmt.sub!(i, ARABIC_LW[args[idx]])
           remove_args << idx
-        when "%pDW"
+        when '%pDW'
           frmt.sub!(i, ARABIC_UW[args[idx]])
           remove_args << idx
-        else
-          # do nothing
         end
       end
       remove_args.reverse_each do |idx|
         args.delete_at idx
       end
-      args_matched = (frmt.count("%") <= args.size)
+      args_matched = (frmt.count('%') <= args.size)
       frmt.gsub!('##', '%%')
       args_matched ? (frmt % args) : frmt
     rescue

data/lib/review/i18n.yml

@@ -5,7 +5,7 @@ ja:
   column: "コラム「%s」"
   columnname: "コラム"
   column_head: "■コラム"
-  part: 第%d部
+  part: 第%pR部
   part_short: "%pR"
   chapter: 第%d章
   chapter_short: "%d"
@@ -67,7 +67,7 @@ en:
   column: "Column %s"
   columnname: "Column"
   column_head: "Column"
-  part: "Part %s"
+  part: "Part %pR"
   part_short: "%pR"
   chapter: Chapter %d
   chapter_short: "%d"
@@ -126,7 +126,7 @@ zh-TW:
   image: 圖
   table: 表
   list: List
-  part: 第%d部份
+  part: 第%pR部份
   part_short: "%pR"
   chapter: 第%d章
   chapter_short: "%d"

data/lib/review/idgxmlbuilder.rb

@@ -1,7 +1,5 @@
-# encoding: utf-8
-
-# Copyright (c) 2002-2007 Minero Aoki
-#               2008-2017 Minero Aoki, Kenshi Muto
+# Copyright (c) 2008-2017 Minero Aoki, Kenshi Muto
+#               2002-2007 Minero Aoki
 #
 # This program is free software.
 # You can distribute or modify this program under the terms of
@@ -14,13 +12,11 @@ require 'review/textutils'
 require 'nkf'
 
 module ReVIEW
-
   class IDGXMLBuilder < Builder
-
     include TextUtils
     include HTMLUtils
 
-    [:ttbold, :hint, :maru, :keytop, :labelref, :ref, :pageref, :balloon].each {|e| Compiler.definline(e) }
+    %i[ttbold hint maru keytop labelref ref balloon].each { |e| Compiler.definline(e) }
     Compiler.defsingle(:dtp, 1)
 
     Compiler.defblock(:insn, 0..1)
@@ -65,19 +61,19 @@ module ReVIEW
       @noindent = nil
       @ol_num = nil
       @first_line_num = nil
-      @rootelement = "doc"
+      @rootelement = 'doc'
       @secttags = nil
       @tsize = nil
       @texblockequation = 0
       @texinlineequation = 0
       print %Q(<?xml version="1.0" encoding="UTF-8"?>\n)
       print %Q(<#{@rootelement} xmlns:aid="http://ns.adobe.com/AdobeInDesign/4.0/">)
-      @secttags = true unless @book.config["structuredxml"].nil?
+      @secttags = true unless @book.config['structuredxml'].nil?
     end
     private :builder_init_file
 
     def puts(arg)
-      if @book.config["nolf"].present?
+      if @book.config['nolf'].present?
         print arg
       else
         super
@@ -85,43 +81,25 @@ module ReVIEW
     end
 
     def result
-      s = ""
-      unless @secttags.nil?
-        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 = ''
+      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
       end
       @output.string + s + "</#{@rootelement}>\n"
     end
 
-    def warn(msg)
-      if @no_error
-        @warns.push [@location.filename, @location.lineno, msg]
-        puts "----WARNING: #{escape_html(msg)}----"
-      else
-        $stderr.puts "#{@location}: warning: #{msg}"
-      end
-    end
-
-    def error(msg)
-      if @no_error
-        @errors.push [@location.filename, @location.lineno, msg]
-        puts "----ERROR: #{escape_html(msg)}----"
-      else
-        $stderr.puts "#{@location}: error: #{msg}"
-      end
-    end
-
     def headline(level, label, caption)
       case level
       when 1
-        unless @secttags.nil?
-          print "</sect4>" if @subsubsubsection > 0
-          print "</sect3>" if @subsubsection > 0
-          print "</sect2>" if @subsection > 0
-          print "</sect>" if @section > 0
+        if @secttags
+          print '</sect4>' if @subsubsubsection > 0
+          print '</sect3>' if @subsubsection > 0
+          print '</sect2>' if @subsection > 0
+          print '</sect>' if @section > 0
         end
 
         print %Q(<chapter id="chap:#{@chapter.number}">) unless @secttags.nil?
@@ -131,11 +109,11 @@ module ReVIEW
         @subsubsection = 0
         @subsubsubsection = 0
       when 2
-        unless @secttags.nil?
-          print "</sect4>" if @subsubsubsection > 0
-          print "</sect3>" if @subsubsection > 0
-          print "</sect2>" if @subsection > 0
-          print "</sect>" if @section > 0
+        if @secttags
+          print '</sect4>' if @subsubsubsection > 0
+          print '</sect3>' if @subsubsection > 0
+          print '</sect2>' if @subsection > 0
+          print '</sect>' if @section > 0
         end
         @section += 1
         print %Q(<sect id="sect:#{@chapter.number}.#{@section}">) unless @secttags.nil?
@@ -144,10 +122,10 @@ module ReVIEW
         @subsubsection = 0
         @subsubsubsection = 0
       when 3
-        unless @secttags.nil?
-          print "</sect4>" if @subsubsubsection > 0
-          print "</sect3>" if @subsubsection > 0
-          print "</sect2>" if @subsection > 0
+        if @secttags
+          print '</sect4>' if @subsubsubsection > 0
+          print '</sect3>' if @subsubsection > 0
+          print '</sect2>' if @subsection > 0
         end
 
         @subsection += 1
@@ -156,35 +134,33 @@ module ReVIEW
         @subsubsection = 0
         @subsubsubsection = 0
       when 4
-        unless @secttags.nil?
-          print "</sect4>" if @subsubsubsection > 0
-          print "</sect3>" if @subsubsection > 0
+        if @secttags
+          print '</sect4>' if @subsubsubsection > 0
+          print '</sect3>' if @subsubsection > 0
         end
 
         @subsubsection += 1
-        print %Q(<sect3 id="sect:#{@chapter.number}.#{@section}.#{@subsection}.#{@subsubsection}">) unless @secttags.nil?
+        print %Q(<sect3 id="sect:#{@chapter.number}.#{@section}.#{@subsection}.#{@subsubsection}">) if @secttags
 
         @subsubsubsection = 0
       when 5
-        unless @secttags.nil?
-          print "</sect4>" if @subsubsubsection > 0
-        end
+        print '</sect4>' if @secttags && @subsubsubsection > 0
 
         @subsubsubsection += 1
-        print %Q(<sect4 id="sect:#{@chapter.number}.#{@section}.#{@subsection}.#{@subsubsection}.#{@subsubsubsection}">) unless @secttags.nil?
+        print %Q(<sect4 id="sect:#{@chapter.number}.#{@section}.#{@subsection}.#{@subsubsection}.#{@subsubsubsection}">) if @secttags
       else
         raise "caption level too deep or unsupported: #{level}"
       end
 
-      prefix, anchor = headline_prefix(level)
+      prefix, _anchor = headline_prefix(level)
 
-      label = label.nil? ? "" : " id=\"#{label}\""
+      label = label.nil? ? '' : %Q( id="#{label}")
       toccaption = escape_html(compile_inline(caption.gsub(/@<fn>\{.+?\}/, '')).gsub(/<[^>]+>/, ''))
       puts %Q(<title#{label} aid:pstyle="h#{level}">#{prefix}#{compile_inline(caption)}</title><?dtp level="#{level}" section="#{prefix}#{toccaption}"?>)
     end
 
     def ul_begin
-      level = block_given? ? yield : ""
+      level = block_given? ? yield : ''
       level = nil if level == 1
       puts "<ul#{level == 1 ? nil : level}>"
     end
@@ -194,36 +170,34 @@ module ReVIEW
     end
 
     def ul_item_end
-      puts "</li>"
+      puts '</li>'
     end
 
     def choice_single_begin
-      puts "<choice type='single'>"
+      puts %Q(<choice type='single'>)
     end
 
     def choice_multi_begin
-      puts "<choice type='multi'>"
+      puts %Q(<choice type='multi'>)
     end
 
     def choice_single_end
-      puts "</choice>"
+      puts '</choice>'
     end
 
     def choice_multi_end
-      puts "</choice>"
+      puts '</choice>'
     end
 
     def ul_end
-      level = block_given? ? yield : ""
+      level = block_given? ? yield : ''
       level = nil if level == 1
       puts "</ul#{level}>"
     end
 
     def ol_begin
       puts '<ol>'
-      if !@ol_num
-        @ol_num = 1
-      end
+      @ol_num ||= 1
     end
 
     def ol_item(lines, num)
@@ -259,7 +233,7 @@ module ReVIEW
     def paragraph(lines)
       if @noindent.nil?
         if lines[0] =~ /\A(\t+)/
-          puts %Q(<p inlist="#{$1.size}">#{lines.join('').sub(/\A\t+/, "")}</p>)
+          puts %Q(<p inlist="#{$1.size}">#{lines.join.sub(/\A\t+/, '')}</p>)
         else
           puts "<p>#{lines.join}</p>"
         end
@@ -270,95 +244,96 @@ module ReVIEW
     end
 
     def read(lines)
-      puts %Q[<lead>#{split_paragraph(lines).join}</lead>]
+      puts %Q(<lead>#{split_paragraph(lines).join}</lead>)
     end
 
     alias_method :lead, :read
 
-    def column_label(id)
-      num = @chapter.column(id).number
+    def column_label(id, chapter = @chapter)
+      num = chapter.column(id).number
       "column-#{num}"
     end
     private :column_label
 
     def inline_column_chap(chapter, id)
-      if @book.config["chapterlink"]
-        %Q(<link href="#{column_label(id)}">#{I18n.t("column", compile_inline(chapter.column(id).caption))}</link>)
+      if @book.config['chapterlink']
+        %Q(<link href="#{column_label(id, chapter)}">#{I18n.t('column', compile_inline(chapter.column(id).caption))}</link>)
       else
-        I18n.t("column", compile_inline(chapter.column(id).caption))
+        I18n.t('column', compile_inline(chapter.column(id).caption))
       end
     end
 
     def inline_list(id)
       chapter, id = extract_chapter_id(id)
       if get_chap(chapter).nil?
-        "<span type='list'>#{I18n.t("list")}#{I18n.t("format_number_without_chapter", [chapter.list(id).number])}</span>"
+        "<span type='list'>#{I18n.t('list')}#{I18n.t('format_number_without_chapter', [chapter.list(id).number])}</span>"
       else
-        "<span type='list'>#{I18n.t("list")}#{I18n.t("format_number", [get_chap(chapter), chapter.list(id).number])}</span>"
+        "<span type='list'>#{I18n.t('list')}#{I18n.t('format_number', [get_chap(chapter), chapter.list(id).number])}</span>"
       end
     end
 
-    def list_header(id, caption, lang)
-      puts %Q[<codelist>]
+    def list_header(id, caption, _lang)
+      puts '<codelist>'
+      return true unless caption.present?
       if get_chap.nil?
-        puts %Q[<caption>#{I18n.t("list")}#{I18n.t("format_number_without_chapter", [@chapter.list(id).number])}#{I18n.t("caption_prefix_idgxml")}#{compile_inline(caption)}</caption>]
+        puts %Q(<caption>#{I18n.t('list')}#{I18n.t('format_number_without_chapter', [@chapter.list(id).number])}#{I18n.t('caption_prefix_idgxml')}#{compile_inline(caption)}</caption>)
       else
-        puts %Q[<caption>#{I18n.t("list")}#{I18n.t("format_number", [get_chap, @chapter.list(id).number])}#{I18n.t("caption_prefix_idgxml")}#{compile_inline(caption)}</caption>]
+        puts %Q(<caption>#{I18n.t('list')}#{I18n.t('format_number', [get_chap, @chapter.list(id).number])}#{I18n.t('caption_prefix_idgxml')}#{compile_inline(caption)}</caption>)
       end
     end
 
     def codelines_body(lines)
       no = 1
       lines.each do |line|
-        unless @book.config["listinfo"].nil?
-          print "<listinfo line=\"#{no}\""
-          print " begin=\"1\"" if no == 1
-          print " end=\"#{no}\"" if no == lines.size
-          print ">"
+        if @book.config['listinfo']
+          print %Q(<listinfo line="#{no}")
+          print %Q( begin="1") if no == 1
+          print %Q( end="#{no}") if no == lines.size
+          print '>'
         end
         print detab(line)
         print "\n"
-        print "</listinfo>" unless @book.config["listinfo"].nil?
+        print '</listinfo>' if @book.config['listinfo']
         no += 1
       end
     end
 
-    def list_body(id, lines, lang)
-      print %Q(<pre>)
+    def list_body(_id, lines, _lang)
+      print '<pre>'
       codelines_body(lines)
-      puts "</pre></codelist>"
+      puts '</pre></codelist>'
     end
 
-    def emlist(lines, caption = nil, lang = nil)
+    def emlist(lines, caption = nil, _lang = nil)
       quotedlist lines, 'emlist', caption
     end
 
-    def emlistnum(lines, caption = nil, lang = nil)
-      _lines = []
-      first_line_num = get_line_num
+    def emlistnum(lines, caption = nil, _lang = nil)
+      lines2 = []
+      first_line_num = line_num
       lines.each_with_index do |line, i|
-        _lines << detab("<span type='lineno'>" + (i + first_line_num).to_s.rjust(2) + ": </span>" + line)
+        lines2 << detab(%Q(<span type='lineno'>) + (i + first_line_num).to_s.rjust(2) + ': </span>' + line)
       end
-      quotedlist _lines, 'emlistnum', caption
+      quotedlist lines2, 'emlistnum', caption
     end
 
-    def listnum_body(lines, lang)
-      print %Q(<pre>)
+    def listnum_body(lines, _lang)
+      print '<pre>'
       no = 1
-      first_line_num = get_line_num
+      first_line_num = line_num
       lines.each_with_index do |line, i|
-        unless @book.config["listinfo"].nil?
-          print "<listinfo line=\"#{no}\""
-          print " begin=\"1\"" if no == 1
-          print " end=\"#{no}\"" if no == lines.size
-          print ">"
+        if @book.config['listinfo']
+          print %Q(<listinfo line="#{no}")
+          print %Q( begin="1") if no == 1
+          print %Q( end="#{no}") if no == lines.size
+          print '>'
         end
-        print detab("<span type='lineno'>" + (i + first_line_num).to_s.rjust(2) + ": </span>" + line)
+        print detab(%Q(<span type='lineno'>) + (i + first_line_num).to_s.rjust(2) + ': </span>' + line)
         print "\n"
-        print "</listinfo>" unless @book.config["listinfo"].nil?
+        print '</listinfo>' if @book.config['listinfo']
         no += 1
       end
-      puts "</pre></codelist>"
+      puts '</pre></codelist>'
     end
 
     def cmd(lines, caption = nil)
@@ -366,20 +341,20 @@ module ReVIEW
     end
 
     def quotedlist(lines, css_class, caption)
-      print %Q[<list type='#{css_class}'>]
+      print %Q(<list type='#{css_class}'>)
       puts "<caption aid:pstyle='#{css_class}-title'>#{compile_inline(caption)}</caption>" if caption.present?
-      print %Q[<pre>]
+      print '<pre>'
       no = 1
       lines.each do |line|
-        unless @book.config["listinfo"].nil?
-          print "<listinfo line=\"#{no}\""
-          print " begin=\"1\"" if no == 1
-          print " end=\"#{no}\"" if no == lines.size
-          print ">"
+        if @book.config['listinfo']
+          print %Q(<listinfo line="#{no}")
+          print %Q( begin="1") if no == 1
+          print %Q( end="#{no}") if no == lines.size
+          print '>'
         end
         print detab(line)
         print "\n"
-        print "</listinfo>" unless @book.config["listinfo"].nil?
+        print '</listinfo>' if @book.config['listinfo']
         no += 1
       end
       puts '</pre></list>'
@@ -388,24 +363,24 @@ module ReVIEW
 
     def quote(lines)
       blocked_lines = split_paragraph(lines)
-      puts "<quote>#{blocked_lines.join("")}</quote>"
+      puts "<quote>#{blocked_lines.join}</quote>"
     end
 
     def inline_table(id)
       chapter, id = extract_chapter_id(id)
       if get_chap(chapter).nil?
-        "<span type='table'>#{I18n.t("table")}#{I18n.t("format_number_without_chapter", [chapter.table(id).number])}</span>"
+        "<span type='table'>#{I18n.t('table')}#{I18n.t('format_number_without_chapter', [chapter.table(id).number])}</span>"
       else
-        "<span type='table'>#{I18n.t("table")}#{I18n.t("format_number", [get_chap(chapter), chapter.table(id).number])}</span>"
+        "<span type='table'>#{I18n.t('table')}#{I18n.t('format_number', [get_chap(chapter), chapter.table(id).number])}</span>"
       end
     end
 
     def inline_img(id)
       chapter, id = extract_chapter_id(id)
       if get_chap(chapter).nil?
-        "<span type='image'>#{I18n.t("image")}#{I18n.t("format_number_without_chapter", [chapter.image(id).number])}</span>"
+        "<span type='image'>#{I18n.t('image')}#{I18n.t('format_number_without_chapter', [chapter.image(id).number])}</span>"
       else
-        "<span type='image'>#{I18n.t("image")}#{I18n.t("format_number", [get_chap(chapter), chapter.image(id).number])}</span>"
+        "<span type='image'>#{I18n.t('image')}#{I18n.t('format_number', [get_chap(chapter), chapter.image(id).number])}</span>"
       end
     end
 
@@ -413,69 +388,66 @@ module ReVIEW
       chapter, id = extract_chapter_id(id)
       if chapter.image(id).caption.blank?
         inline_img(id)
+      elsif get_chap(chapter).nil?
+        "<span type='image'>#{I18n.t('image')}#{I18n.t('format_number_without_chapter', [chapter.image(id).number])}#{I18n.t('image_quote', chapter.image(id).caption)}</span>"
       else
-        if get_chap(chapter).nil?
-          "<span type='image'>#{I18n.t("image")}#{I18n.t("format_number_without_chapter", [chapter.image(id).number])}#{I18n.t('image_quote', chapter.image(id).caption)}</span>"
-        else
-          "<span type='image'>#{I18n.t("image")}#{I18n.t("format_number", [get_chap(chapter), chapter.image(id).number])}#{I18n.t('image_quote', chapter.image(id).caption)}</span>"
-        end
+        "<span type='image'>#{I18n.t('image')}#{I18n.t('format_number', [get_chap(chapter), chapter.image(id).number])}#{I18n.t('image_quote', chapter.image(id).caption)}</span>"
       end
     end
 
     def handle_metric(str)
       k, v = str.split('=', 2)
-      return %Q|#{k}=\"#{v.sub(/\A["']/, '').sub(/["']\Z/, '')}\"|
+      %Q(#{k}="#{v.sub(/\A["']/, '').sub(/["']\Z/, '')}")
     end
 
     def result_metric(array)
       " #{array.join(' ')}"
     end
 
-    def image_image(id, caption, metric=nil)
-      metrics = parse_metric("idgxml", metric)
-      puts "<img>"
-      puts %Q[<Image href="file://#{@chapter.image(id).path.sub(/\A.\//, "")}"#{metrics} />]
+    def image_image(id, caption, metric = nil)
+      metrics = parse_metric('idgxml', metric)
+      puts '<img>'
+      puts %Q(<Image href="file://#{@chapter.image(id).path.sub(%r{\A./}, '')}"#{metrics} />)
       image_header id, caption
-      puts "</img>"
+      puts '</img>'
     end
 
     def image_dummy(id, caption, lines)
-      puts "<img>"
-      print %Q[<pre aid:pstyle="dummyimage">]
+      puts '<img>'
+      print %Q(<pre aid:pstyle="dummyimage">)
       lines.each do |line|
         print detab(line)
         print "\n"
       end
-      print %Q[</pre>]
+      print '</pre>'
       image_header id, caption
-      puts "</img>"
-      warn "no such image: #{id}"
+      puts '</img>'
+      warn "image not bound: #{id}"
     end
 
     def image_header(id, caption)
+      return true unless caption.present?
       if get_chap.nil?
-        puts %Q[<caption>#{I18n.t("image")}#{I18n.t("format_number_without_chapter", [@chapter.image(id).number])}#{I18n.t("caption_prefix_idgxml")}#{compile_inline(caption)}</caption>]
+        puts %Q(<caption>#{I18n.t('image')}#{I18n.t('format_number_without_chapter', [@chapter.image(id).number])}#{I18n.t('caption_prefix_idgxml')}#{compile_inline(caption)}</caption>)
       else
-        puts %Q[<caption>#{I18n.t("image")}#{I18n.t("format_number", [get_chap, @chapter.image(id).number])}#{I18n.t("caption_prefix_idgxml")}#{compile_inline(caption)}</caption>]
+        puts %Q(<caption>#{I18n.t('image')}#{I18n.t('format_number', [get_chap, @chapter.image(id).number])}#{I18n.t('caption_prefix_idgxml')}#{compile_inline(caption)}</caption>)
       end
     end
 
     def texequation(lines)
       @texblockequation += 1
-      puts %Q[<replace idref="texblock-#{@texblockequation}">]
+      puts %Q(<replace idref="texblock-#{@texblockequation}">)
       puts '<pre>'
-      puts "#{lines.join("\n")}"
+      puts lines.join("\n")
       puts '</pre>'
       puts '</replace>'
     end
 
     def table(lines, id = nil, caption = nil)
-      tablewidth = nil
+      tablewidth = @book.config['tableopt'] ? @book.config['tableopt'].split(',')[0].to_f / @book.config['pt_to_mm_unit'].to_f : nil
       col = 0
-      if @book.config["tableopt"]
-        tablewidth = @book.config["tableopt"].split(",")[0].to_f / @book.config["pt_to_mm_unit"].to_f
-      end
-      puts "<table>"
+
+      puts '<table>'
       rows = []
       sepidx = nil
       lines.each_with_index do |line, idx|
@@ -483,94 +455,90 @@ module ReVIEW
           sepidx ||= idx
           next
         end
-        if tablewidth.nil?
-          rows.push(line.gsub(/\t\.\t/, "\t\t").gsub(/\t\.\.\t/, "\t.\t").gsub(/\t\.\Z/, "\t").gsub(/\t\.\.\Z/, "\t.").gsub(/\A\./, ""))
+        if tablewidth
+          rows.push(line.gsub(/\t\.\t/, "\tDUMMYCELLSPLITTER\t").gsub(/\t\.\.\t/, "\t.\t").gsub(/\t\.\Z/, "\tDUMMYCELLSPLITTER").gsub(/\t\.\.\Z/, "\t.").gsub(/\A\./, ''))
         else
-          rows.push(line.gsub(/\t\.\t/, "\tDUMMYCELLSPLITTER\t").gsub(/\t\.\.\t/, "\t.\t").gsub(/\t\.\Z/, "\tDUMMYCELLSPLITTER").gsub(/\t\.\.\Z/, "\t.").gsub(/\A\./, ""))
+          rows.push(line.gsub(/\t\.\t/, "\t\t").gsub(/\t\.\.\t/, "\t.\t").gsub(/\t\.\Z/, "\t").gsub(/\t\.\.\Z/, "\t.").gsub(/\A\./, ''))
         end
-        _col = rows[rows.length - 1].split(/\t/).length
-        col = _col if _col > col
+        col2 = rows[rows.length - 1].split(/\t/).length
+        col = col2 if col2 > col
       end
 
       cellwidth = []
-      unless tablewidth.nil?
+      if tablewidth
         if @tsize.nil?
-          col.times {|n| cellwidth[n] = tablewidth / col }
+          col.times { |n| cellwidth[n] = tablewidth / col }
         else
           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
             totallength += cellwidth[n]
             warn "total length exceeds limit for table: #{id}" if totallength > tablewidth
           end
           if cellwidth.size < col
             cw = (tablewidth - totallength) / (col - cellwidth.size)
             warn "auto cell sizing exceeds limit for table: #{id}" if cw <= 0
-            (cellwidth.size..(col - 1)).each do |i|
-              cellwidth[i] = cw
-            end
+            (cellwidth.size..(col - 1)).each { |i| cellwidth[i] = cw }
           end
         end
       end
 
       begin
-        table_header id, caption unless caption.nil?
+        table_header id, caption if caption.present?
       rescue KeyError
         error "no such table: #{id}"
       end
       return if rows.empty?
 
       if tablewidth.nil?
-        print "<tbody>"
+        print '<tbody>'
       else
-        print %Q[<tbody xmlns:aid5="http://ns.adobe.com/AdobeInDesign/5.0/" aid:table="table" aid:trows="#{rows.length}" aid:tcols="#{col}">]
+        print %Q(<tbody xmlns:aid5="http://ns.adobe.com/AdobeInDesign/5.0/" aid:table="table" aid:trows="#{rows.length}" aid:tcols="#{col}">)
       end
 
       if sepidx
         sepidx.times do |y|
           if tablewidth.nil?
-            puts %Q[<tr type="header">#{rows.shift}</tr>]
+            puts %Q(<tr type="header">#{rows.shift}</tr>)
           else
             i = 0
-            rows.shift.split(/\t/).each_with_index do |cell, x|
-              print %Q[<td xyh="#{x + 1},#{y + 1},#{sepidx}" aid:table="cell" aid:theader="1" aid:crows="1" aid:ccols="1" aid:ccolwidth="#{sprintf("%.3f", cellwidth[i])}">#{cell.sub("DUMMYCELLSPLITTER", "")}</td>]
+            rows.shift.split("\t").each_with_index do |cell, x|
+              print %Q(<td xyh="#{x + 1},#{y + 1},#{sepidx}" aid:table="cell" aid:theader="1" aid:crows="1" aid:ccols="1" aid:ccolwidth="#{sprintf('%.3f', cellwidth[i])}">#{cell.sub('DUMMYCELLSPLITTER', '')}</td>)
               i += 1
             end
           end
         end
       end
       trputs(tablewidth, rows, cellwidth, sepidx)
-      puts "</tbody></table>"
+      puts '</tbody></table>'
       @tsize = nil
     end
 
     def trputs(tablewidth, rows, cellwidth, sepidx)
       sepidx = 0 if sepidx.nil?
-      if tablewidth.nil?
-        lastline = rows.pop
-        rows.each {|row| puts %Q[<tr>#{row}</tr>] }
-        puts %Q[<tr type="lastline">#{lastline}</tr>] unless lastline.nil?
-      else
+      if tablewidth
         rows.each_with_index do |row, y|
           i = 0
-          row.split(/\t/).each_with_index do |cell, x|
-            print %Q[<td xyh="#{x + 1},#{y + 1 + sepidx},#{sepidx}" aid:table="cell" aid:crows="1" aid:ccols="1" aid:ccolwidth="#{sprintf("%.3f", cellwidth[i])}">#{cell.sub("DUMMYCELLSPLITTER", "")}</td>]
+          row.split("\t").each_with_index do |cell, x|
+            print %Q(<td xyh="#{x + 1},#{y + 1 + sepidx},#{sepidx}" aid:table="cell" aid:crows="1" aid:ccols="1" aid:ccolwidth="#{sprintf('%.3f', cellwidth[i])}">#{cell.sub('DUMMYCELLSPLITTER', '')}</td>)
             i += 1
           end
         end
+      else
+        lastline = rows.pop
+        rows.each { |row| puts "<tr>#{row}</tr>" }
+        puts %Q(<tr type="lastline">#{lastline}</tr>) if lastline
       end
     end
 
     def table_header(id, caption)
       if id.nil?
-        puts %Q[<caption>#{compile_inline(caption)}</caption>]
+        puts %Q(<caption>#{compile_inline(caption)}</caption>)
+      elsif get_chap
+        puts %Q(<caption>#{I18n.t('table')}#{I18n.t('format_number', [get_chap, @chapter.table(id).number])}#{I18n.t('caption_prefix_idgxml')}#{compile_inline(caption)}</caption>)
       else
-        if get_chap.nil?
-          puts %Q[<caption>#{I18n.t("table")}#{I18n.t("format_number_without_chapter", [@chapter.table(id).number])}#{I18n.t("caption_prefix_idgxml")}#{compile_inline(caption)}</caption>]
-        else
-          puts %Q[<caption>#{I18n.t("table")}#{I18n.t("format_number", [get_chap, @chapter.table(id).number])}#{I18n.t("caption_prefix_idgxml")}#{compile_inline(caption)}</caption>]
-        end
+        puts %Q(<caption>#{I18n.t('table')}#{I18n.t('format_number_without_chapter', [@chapter.table(id).number])}#{I18n.t('caption_prefix_idgxml')}#{compile_inline(caption)}</caption>)
       end
     end
 
@@ -578,11 +546,11 @@ module ReVIEW
     end
 
     def tr(rows)
-      puts %Q[<tr>#{rows.join("\t")}</tr>]
+      puts %Q(<tr>#{rows.join("\t")}</tr>)
     end
 
     def th(str)
-      %Q[<?dtp tablerow header?>#{str}]
+      %Q(<?dtp tablerow header?>#{str})
     end
 
     def td(str)
@@ -590,7 +558,7 @@ module ReVIEW
     end
 
     def table_end
-      print "<?dtp tablerow last?>"
+      print '<?dtp tablerow last?>'
     end
 
     def emtable(lines, caption = nil)
@@ -599,11 +567,11 @@ module ReVIEW
 
     def imgtable(lines, id, caption = nil, metric = nil)
       if @chapter.image(id).bound?
-        metrics = parse_metric("idgxml", metric)
-        puts "<table>"
-        table_header id, caption
-        puts %Q[<imgtable><Image href="file://#{@chapter.image(id).path.sub(/\A.\//, "")}"#{metrics} /></imgtable>]
-        puts "</table>"
+        metrics = parse_metric('idgxml', metric)
+        puts '<table>'
+        table_header id, caption if caption.present?
+        puts %Q(<imgtable><Image href="file://#{@chapter.image(id).path.sub(%r{\A./}, '')}"#{metrics} /></imgtable>)
+        puts '</table>'
       else
         warn "image not bound: #{id}" if @strict
         image_dummy id, caption, lines
@@ -611,16 +579,15 @@ module ReVIEW
     end
 
     def comment(lines, comment = nil)
-      if @book.config["draft"]
-        lines ||= []
-        lines.unshift comment unless comment.blank?
-        str = lines.join("\n")
-        print "<msg>#{escape_html(str)}</msg>"
-      end
+      return true unless @book.config['draft']
+      lines ||= []
+      lines.unshift comment unless comment.blank?
+      str = lines.join("\n")
+      print "<msg>#{escape_html(str)}</msg>"
     end
 
     def inline_comment(str)
-      if @book.config["draft"]
+      if @book.config['draft']
         %Q(<msg>#{escape_html(str)}</msg>)
       else
         ''
@@ -632,11 +599,11 @@ module ReVIEW
     end
 
     def inline_fn(id)
-      %Q[<footnote>#{compile_inline(@chapter.footnote(id).content.strip)}</footnote>]
+      %Q(<footnote>#{compile_inline(@chapter.footnote(id).content.strip)}</footnote>)
     end
 
     def compile_ruby(base, ruby)
-      %Q[<GroupRuby><aid:ruby xmlns:aid="http://ns.adobe.com/AdobeInDesign/3.0/"><aid:rb>#{escape_html(base.sub(/\A\s+/, "").sub(/\s+$/, ""))}</aid:rb><aid:rt>#{escape_html(ruby.sub(/\A\s+/, "").sub(/\s+$/, ""))}</aid:rt></aid:ruby></GroupRuby>]
+      %Q(<GroupRuby><aid:ruby xmlns:aid="http://ns.adobe.com/AdobeInDesign/3.0/"><aid:rb>#{escape_html(base.strip)}</aid:rb><aid:rt>#{escape_html(ruby.strip)}</aid:rt></aid:ruby></GroupRuby>)
     end
 
     def compile_kw(word, alt)
@@ -646,55 +613,55 @@ module ReVIEW
         else escape_html(word)
         end +
       '</keyword>' +
-        %Q[<index value="#{escape_html(word)}" />] +
+        %Q(<index value="#{escape_html(word)}" />) +
         if alt
-          alt.split(/\s*,\s*/).collect! {|e| %Q[<index value="#{escape_html(e.strip)}" />] }.join
+          alt.split(/\s*,\s*/).collect! { |e| %Q(<index value="#{escape_html(e.strip)}" />) }.join
         else
-          ""
+          ''
         end
     end
 
     def compile_href(url, label)
-      %Q[<a linkurl='#{escape_html(url)}'>#{label.nil? ? escape_html(url) : escape_html(label)}</a>]
+      %Q(<a linkurl='#{escape_html(url)}'>#{label.nil? ? escape_html(url) : escape_html(label)}</a>)
     end
 
     def inline_sup(str)
-      %Q[<sup>#{escape_html(str)}</sup>]
+      %Q(<sup>#{escape_html(str)}</sup>)
     end
 
     def inline_sub(str)
-      %Q[<sub>#{escape_html(str)}</sub>]
+      %Q(<sub>#{escape_html(str)}</sub>)
     end
 
     def inline_raw(str)
-      %Q[#{super(str).gsub("\\n", "\n")}]
+      super(str).gsub('\\n', "\n")
     end
 
     def inline_hint(str)
-      if @book.config["nolf"].nil?
-        %Q[\n<hint>#{escape_html(str)}</hint>]
+      if @book.config['nolf']
+        %Q(<hint>#{escape_html(str)}</hint>)
       else
-        %Q[<hint>#{escape_html(str)}</hint>]
+        %Q(\n<hint>#{escape_html(str)}</hint>)
       end
     end
 
     def inline_maru(str)
       if str =~ /\A\d+\Z/
-        sprintf("&#x%x;", 9311 + str.to_i)
+        sprintf('&#x%x;', 9311 + str.to_i)
       elsif str =~ /\A[A-Z]\Z/
         begin
-          sprintf("&#x%x;", 9398 + str.codepoints.to_a[0] - 65)
+          sprintf('&#x%x;', 9398 + str.codepoints.to_a[0] - 65)
         rescue NoMethodError
-          sprintf("&#x%x;", 9398 + str[0] - 65)
+          sprintf('&#x%x;', 9398 + str[0] - 65)
         end
       elsif str =~ /\A[a-z]\Z/
         begin
-          sprintf("&#x%x;", 9392 + str.codepoints.to_a[0] - 65)
+          sprintf('&#x%x;', 9392 + str.codepoints.to_a[0] - 65)
         rescue NoMethodError
-          sprintf("&#x%x;", 9392 + str[0] - 65)
+          sprintf('&#x%x;', 9392 + str[0] - 65)
         end
       else
-        raise "can't parse maru: #{str}"
+        error "can't parse maru: #{str}"
       end
     end
 
@@ -738,42 +705,42 @@ module ReVIEW
 
     def inline_icon(id)
       begin
-        %Q[<Image href="file://#{@chapter.image(id).path.sub(/\A\.\//, "")}" type="inline" />]
+        %Q(<Image href="file://#{@chapter.image(id).path.sub(%r{\A\./}, '')}" type="inline" />)
       rescue
-        warn "no such icon image: #{id}"
-        ""
+        warn "image not bound: #{id}"
+        ''
       end
     end
 
     def inline_bou(str)
-      %Q[<bou>#{escape_html(str)}</bou>]
+      %Q(<bou>#{escape_html(str)}</bou>)
     end
 
     def inline_keytop(str)
-      %Q[<keytop>#{escape_html(str)}</keytop>]
+      %Q(<keytop>#{escape_html(str)}</keytop>)
     end
 
     def inline_labelref(idref)
-      %Q[<ref idref='#{escape_html(idref)}'>「#{I18n.t("label_marker")}#{escape_html(idref)}」</ref>] # FIXME: 節名とタイトルも込みで要出力
+      %Q(<ref idref='#{escape_html(idref)}'>「#{I18n.t('label_marker')}#{escape_html(idref)}」</ref>) # FIXME: 節名とタイトルも込みで要出力
     end
 
     alias_method :inline_ref, :inline_labelref
 
     def inline_pageref(idref)
-      %Q[<pageref idref='#{escape_html(idref)}'>●●</pageref>] # ページ番号を参照
+      %Q(<pageref idref='#{escape_html(idref)}'>●●</pageref>) # ページ番号を参照
     end
 
     def inline_balloon(str)
-      %Q[<balloon>#{escape_html(str).gsub(/@maru\[(\d+)\]/) {|m| inline_maru($1)}}</balloon>]
+      %Q(<balloon>#{escape_html(str).gsub(/@maru\[(\d+)\]/) { inline_maru($1) }}</balloon>)
     end
 
     def inline_uchar(str)
-      %Q[&#x#{str};]
+      %Q(&#x#{str};)
     end
 
     def inline_m(str)
       @texinlineequation += 1
-      %Q[<replace idref="texinline-#{@texinlineequation}"><pre>#{escape_html(str)}</pre></replace>]
+      %Q(<replace idref="texinline-#{@texinlineequation}"><pre>#{escape_html(str)}</pre></replace>)
     end
 
     def noindent
@@ -782,22 +749,22 @@ module ReVIEW
 
     def linebreak
       # FIXME: pが閉じちゃってるので一度戻らないといけないが、難しい…。
-      puts "<br />"
+      puts '<br />'
     end
 
     def pagebreak
-      puts "<pagebreak />"
+      puts '<pagebreak />'
     end
 
-    def nonum_begin(level, label, caption)
-      puts %Q[<title aid:pstyle="h#{level}">#{compile_inline(caption)}</title><?dtp level="#{level}" section="#{escape_html(compile_inline(caption))}"?>]
+    def nonum_begin(level, _label, caption)
+      puts %Q(<title aid:pstyle="h#{level}">#{compile_inline(caption)}</title><?dtp level="#{level}" section="#{escape_html(compile_inline(caption))}"?>)
     end
 
     def nonum_end(level)
     end
 
-    def circle_begin(level, label, caption)
-      puts %Q[<title aid:pstyle="smallcircle">&#x2022;#{compile_inline(caption)}</title>]
+    def circle_begin(_level, _label, caption)
+      puts %Q(<title aid:pstyle="smallcircle">&#x2022;#{compile_inline(caption)}</title>)
     end
 
     def circle_end(level)
@@ -805,233 +772,234 @@ module ReVIEW
 
     def common_column_begin(type, caption)
       @column += 1
-      a_id = %Q[id="column-#{@column}"]
+      a_id = %Q(id="column-#{@column}")
       print "<#{type}column #{a_id}>"
-      puts %Q[<title aid:pstyle="#{type}column-title">#{compile_inline(caption)}</title><?dtp level="9" section="#{escape_html(compile_inline(caption))}"?>]
+      puts %Q(<title aid:pstyle="#{type}column-title">#{compile_inline(caption)}</title><?dtp level="9" section="#{escape_html(compile_inline(caption))}"?>)
     end
 
     def common_column_end(type)
       puts "</#{type}column>"
     end
 
-    def column_begin(level, label, caption)
-      common_column_begin("", caption)
+    def column_begin(_level, _label, caption)
+      common_column_begin('', caption)
     end
 
-    def column_end(level)
-      common_column_end("")
+    def column_end(_level)
+      common_column_end('')
     end
 
-    def xcolumn_begin(level, label, caption)
-      common_column_begin("x", caption)
+    def xcolumn_begin(_level, _label, caption)
+      common_column_begin('x', caption)
     end
 
-    def xcolumn_end(level)
-      common_column_end("x")
+    def xcolumn_end(_level)
+      common_column_end('x')
     end
 
-    def world_begin(level, label, caption)
-      common_column_begin("world", caption)
+    def world_begin(_level, _label, caption)
+      common_column_begin('world', caption)
     end
 
-    def world_end(level)
-      common_column_end("world")
+    def world_end(_level)
+      common_column_end('world')
     end
 
-    def hood_begin(level, label, caption)
-      common_column_begin("hood", caption)
+    def hood_begin(_level, _label, caption)
+      common_column_begin('hood', caption)
     end
 
-    def hood_end(level)
-      common_column_end("hood")
+    def hood_end(_level)
+      common_column_end('hood')
     end
 
-    def edition_begin(level, label, caption)
-      common_column_begin("edition", caption)
+    def edition_begin(_level, _label, caption)
+      common_column_begin('edition', caption)
     end
 
-    def edition_end(level)
-      common_column_end("edition")
+    def edition_end(_level)
+      common_column_end('edition')
     end
 
-    def insideout_begin(level, label, caption)
-      common_column_begin("insideout", caption)
+    def insideout_begin(_level, _label, caption)
+      common_column_begin('insideout', caption)
     end
 
-    def insideout_end(level)
-      common_column_end("insideout")
+    def insideout_end(_level)
+      common_column_end('insideout')
     end
 
-    def ref_begin(level, label, caption)
-      if !label.nil?
+    def ref_begin(_level, label, _caption)
+      if label
         puts "<reference id='#{label}'>"
       else
-        puts "<reference>"
+        puts '<reference>'
       end
     end
 
-    def ref_end(level)
-      puts "</reference>"
+    def ref_end(_level)
+      puts '</reference>'
     end
 
-    def sup_begin(level, label, caption)
-      if !label.nil?
+    def sup_begin(_level, label, _caption)
+      if label
         puts "<supplement id='#{label}'>"
       else
-        puts "<supplement>"
+        puts '<supplement>'
       end
     end
 
-    def sup_end(level)
-      puts "</supplement>"
+    def sup_end(_level)
+      puts '</supplement>'
     end
 
     def flushright(lines)
-      puts split_paragraph(lines).join.gsub("<p>", "<p align='right'>")
+      puts split_paragraph(lines).join.gsub('<p>', %Q(<p align='right'>))
     end
 
     def centering(lines)
-      puts split_paragraph(lines).join.gsub("<p>", "<p align='center'>")
+      puts split_paragraph(lines).join.gsub('<p>', %Q(<p align='center'>))
     end
 
     def captionblock(type, lines, caption, specialstyle = nil)
       print "<#{type}>"
       style = specialstyle.nil? ? "#{type}-title" : specialstyle
-      puts "<title aid:pstyle='#{style}'>#{compile_inline(caption)}</title>" unless caption.nil?
+      puts "<title aid:pstyle='#{style}'>#{compile_inline(caption)}</title>" if caption.present?
       blocked_lines = split_paragraph(lines)
       puts "#{blocked_lines.join}</#{type}>"
     end
 
     def note(lines, caption = nil)
-      captionblock("note", lines, caption)
+      captionblock('note', lines, caption)
     end
 
     def memo(lines, caption = nil)
-      captionblock("memo", lines, caption)
+      captionblock('memo', lines, caption)
     end
 
     def tip(lines, caption = nil)
-      captionblock("tip", lines, caption)
+      captionblock('tip', lines, caption)
     end
 
     def info(lines, caption = nil)
-      captionblock("info", lines, caption)
+      captionblock('info', lines, caption)
     end
 
     def planning(lines, caption = nil)
-      captionblock("planning", lines, caption)
+      captionblock('planning', lines, caption)
     end
 
     def best(lines, caption = nil)
-      captionblock("best", lines, caption)
+      captionblock('best', lines, caption)
     end
 
     def important(lines, caption = nil)
-      captionblock("important", lines, caption)
+      captionblock('important', lines, caption)
     end
 
     def security(lines, caption = nil)
-      captionblock("security", lines, caption)
+      captionblock('security', lines, caption)
     end
 
     def caution(lines, caption = nil)
-      captionblock("caution", lines, caption)
+      captionblock('caution', lines, caption)
     end
 
     def warning(lines, caption = nil)
-      captionblock("warning", lines, caption)
+      captionblock('warning', lines, caption)
     end
 
     def term(lines)
-      captionblock("term", lines, nil)
+      captionblock('term', lines, nil)
     end
 
     def link(lines, caption = nil)
-      captionblock("link", lines, caption)
+      captionblock('link', lines, caption)
     end
 
     def notice(lines, caption = nil)
-      if caption.nil?
-        captionblock("notice", lines, nil)
+      if caption
+        captionblock('notice-t', lines, caption, 'notice-title')
       else
-        captionblock("notice-t", lines, caption, "notice-title")
+        captionblock('notice', lines, nil)
       end
     end
 
     def point(lines, caption = nil)
-      if caption.nil?
-        captionblock("point", lines, nil)
+      if caption
+        captionblock('point-t', lines, caption, 'point-title')
       else
-        captionblock("point-t", lines, caption, "point-title")
+        captionblock('point', lines, nil)
       end
     end
 
     def shoot(lines, caption = nil)
-      if caption.nil?
-        captionblock("shoot", lines, nil)
+      if caption
+        captionblock('shoot-t', lines, caption, 'shoot-title')
       else
-        captionblock("shoot-t", lines, caption, "shoot-title")
+        captionblock('shoot', lines, nil)
       end
     end
 
     def reference(lines)
-      captionblock("reference", lines, nil)
+      captionblock('reference', lines, nil)
     end
 
     def practice(lines)
-      captionblock("practice", lines, nil)
+      captionblock('practice', lines, nil)
     end
 
     def expert(lines)
-      captionblock("expert", lines, nil)
+      captionblock('expert', lines, nil)
     end
 
     def syntaxblock(type, lines, caption)
-      if caption.nil?
-        puts %Q[<#{type}>]
-      else
-        titleopentag = %Q[caption aid:pstyle="#{type}-title"]
-        titleclosetag = "caption"
-        if type == "insn"
-          titleopentag = %Q[floattitle type="insn"]
-          titleclosetag = "floattitle"
+      if caption.present?
+        titleopentag = %Q(caption aid:pstyle="#{type}-title")
+        titleclosetag = 'caption'
+        if type == 'insn'
+          titleopentag = %Q(floattitle type="insn")
+          titleclosetag = 'floattitle'
         end
-        puts %Q[<#{type}><#{titleopentag}>#{compile_inline(caption)}</#{titleclosetag}>]
+        puts %Q(<#{type}><#{titleopentag}>#{compile_inline(caption)}</#{titleclosetag}>)
+      else
+        puts "<#{type}>"
       end
+
       no = 1
       lines.each do |line|
-        unless @book.config["listinfo"].nil?
-          print %Q[<listinfo line="#{no}"]
-          print %Q[ begin="1"] if no == 1
-          print %Q[ end="#{no}"] if no == lines.size
-          print %Q[>]
+        if @book.config['listinfo']
+          print %Q(<listinfo line="#{no}")
+          print %Q( begin="1") if no == 1
+          print %Q( end="#{no}") if no == lines.size
+          print '>'
         end
         print detab(line)
         print "\n"
-        print "</listinfo>" unless @book.config["listinfo"].nil?
+        print '</listinfo>' if @book.config['listinfo']
         no += 1
       end
       puts "</#{type}>"
     end
 
     def insn(lines, caption = nil)
-      syntaxblock("insn", lines, caption)
+      syntaxblock('insn', lines, caption)
     end
 
     def box(lines, caption = nil)
-      syntaxblock("box", lines, caption)
+      syntaxblock('box', lines, caption)
     end
 
-    def indepimage(id, caption=nil, metric=nil)
-      metrics = parse_metric("idgxml", metric)
-      puts "<img>"
+    def indepimage(_lines, id, caption = nil, metric = nil)
+      metrics = parse_metric('idgxml', metric)
+      puts '<img>'
       begin
-        puts %Q[<Image href="file://#{@chapter.image(id).path.sub(/\A\.\//, "")}"#{metrics} />]
+        puts %Q(<Image href="file://#{@chapter.image(id).path.sub(%r{\A\./}, '')}"#{metrics} />)
       rescue
-        warn %Q[no such image: #{id}]
+        warn %Q(image not bound: #{id})
       end
-      puts %Q[<caption>#{compile_inline(caption)}</caption>] if caption.present?
-      puts "</img>"
+      puts %Q(<caption>#{compile_inline(caption)}</caption>) if caption.present?
+      puts '</img>'
     end
 
     alias_method :numberlessimage, :indepimage
@@ -1046,11 +1014,11 @@ module ReVIEW
     end
 
     def hr
-      print "<hr />"
+      print '<hr />'
     end
 
     def bpo(lines)
-      puts %Q[<bpo>#{lines.join("\n")}</bpo>]
+      puts %Q(<bpo>#{lines.join("\n")}</bpo>)
     end
 
     def inline_dtp(str)
@@ -1058,17 +1026,17 @@ module ReVIEW
     end
 
     def inline_code(str)
-      %Q[<tt type='inline-code'>#{escape_html(str)}</tt>]
+      %Q(<tt type='inline-code'>#{escape_html(str)}</tt>)
     end
 
-    def inline_br(str)
+    def inline_br(_str)
       "\n"
     end
 
     def rawblock(lines)
       no = 1
       lines.each do |l|
-        print l.gsub("&lt;", "<").gsub("&gt;", ">").gsub("&quot;", "\"").gsub("&amp;", "&")
+        print l.gsub('&lt;', '<').gsub('&gt;', '>').gsub('&quot;', '"').gsub('&amp;', '&')
         print "\n" unless lines.length == no
         no += 1
       end
@@ -1079,17 +1047,17 @@ module ReVIEW
     end
 
     def inline_chapref(id)
-      chs = ["", "「", "」"]
-      if @book.config["chapref"]
-        _chs = @book.config["chapref"].split(",")
-        if _chs.size != 3
-          error "--chapsplitter must have exactly 3 parameters with comma."
+      chs = ['', '「', '」']
+      if @book.config['chapref']
+        chs2 = @book.config['chapref'].split(',')
+        if chs2.size != 3
+          error '--chapsplitter must have exactly 3 parameters with comma.'
         else
-          chs = _chs
+          chs = chs2
         end
       end
       s = "#{chs[0]}#{@book.chapter_index.number(id)}#{chs[1]}#{@book.chapter_index.title(id)}#{chs[2]}"
-      if @book.config["chapterlink"]
+      if @book.config['chapterlink']
         %Q(<link href="#{id}">#{s}</link>)
       else
         s
@@ -1100,7 +1068,7 @@ module ReVIEW
     end
 
     def inline_chap(id)
-      if @book.config["chapterlink"]
+      if @book.config['chapterlink']
         %Q(<link href="#{id}">#{@book.chapter_index.number(id)}</link>)
       else
         @book.chapter_index.number(id)
@@ -1112,7 +1080,7 @@ module ReVIEW
 
     def inline_title(id)
       title = super
-      if @book.config["chapterlink"]
+      if @book.config['chapterlink']
         %Q(<link href="#{id}">#{title}</link>)
       else
         title
@@ -1123,31 +1091,29 @@ module ReVIEW
     end
 
     def source_header(caption)
-      puts %Q[<source>]
-      puts %Q[<caption>#{compile_inline(caption)}</caption>]
+      puts '<source>'
+      puts %Q(<caption>#{compile_inline(caption)}</caption>) if caption.present?
     end
 
-    def source_body(lines, lang)
-      puts %Q[<pre>]
+    def source_body(lines, _lang)
+      puts '<pre>'
       codelines_body(lines)
-      puts %Q[</pre></source>]
+      puts '</pre></source>'
     end
 
     def bibpaper(lines, id, caption)
       bibpaper_header id, caption
-      unless lines.empty?
-        bibpaper_bibpaper id, caption, lines
-      end
-      puts %Q(</bibitem>)
+      bibpaper_bibpaper id, caption, lines unless lines.empty?
+      puts '</bibitem>'
     end
 
     def bibpaper_header(id, caption)
       puts %Q(<bibitem id="bib-#{id}">)
-      puts "<caption><span type='bibno'>[#{@chapter.bibpaper(id).number}] </span>#{compile_inline(caption)}</caption>"
+      puts "<caption><span type='bibno'>[#{@chapter.bibpaper(id).number}] </span>#{compile_inline(caption)}</caption>" if caption.present?
     end
 
-    def bibpaper_bibpaper(id, caption, lines)
-      print split_paragraph(lines).join("")
+    def bibpaper_bibpaper(_id, _caption, lines)
+      print split_paragraph(lines).join
     end
 
     def inline_bib(id)
@@ -1157,11 +1123,11 @@ module ReVIEW
     def inline_hd_chap(chap, id)
       if chap.number
         n = chap.headline_index.number(id)
-        if @book.config["secnolevel"] >= n.split('.').size
-          return I18n.t("chapter_quote", "#{n}　#{compile_inline(chap.headline(id).caption)}")
+        if @book.config['secnolevel'] >= n.split('.').size
+          return I18n.t('chapter_quote', "#{n}　#{compile_inline(chap.headline(id).caption)}")
         end
       end
-      I18n.t("chapter_quote", compile_inline(chap.headline(id).caption))
+      I18n.t('chapter_quote', compile_inline(chap.headline(id).caption))
     end
 
     def inline_recipe(id)
@@ -1174,9 +1140,7 @@ module ReVIEW
     end
 
     def image_ext
-      "eps"
+      'eps'
     end
-
   end
-
 end # module ReVIEW