review 3.1.0 → 5.0.0

data/lib/review/init.rb

@@ -1,5 +1,5 @@
 #
-# Copyright (c) 2018 Masanori Kado, Masayoshi Takahashi, Kenshi Muto
+# Copyright (c) 2018-2019 Masanori Kado, Masayoshi Takahashi, Kenshi Muto
 #
 # This program is free software.
 # You can distribute or modify this program under the terms of
@@ -18,12 +18,12 @@ module ReVIEW
       new.execute(*args)
     end
 
-    TEX_DOCUMENTCLASS_OPTS = {
-      'review-jsbook' => 'media=print,paper=a5',
-      'review-jlreq' => 'media=print,paper=a5'
-    }
-
     def initialize
+      @tex_documentclass_opts = {
+        'review-jsbook' => 'media=print,paper=a5',
+        'review-jlreq' => 'media=print,paper=a5'
+      }
+
       @template = 'review-jsbook'
       @logger = ReVIEW.logger
       @review_dir = File.dirname(File.expand_path('..', __dir__))
@@ -32,15 +32,25 @@ module ReVIEW
     def execute(*args)
       initdir = parse_options(args)
 
+      if @webui
+        start_webui
+        unless @web_result
+          @logger.error 'Aborted.'
+          exit 1
+        end
+        @template = @web_result[0]
+        @tex_documentclass_opts[@template] = @web_result[1]
+      end
+
       generate_dir(initdir) do |dir|
         generate_catalog_file(dir)
         generate_sample(dir)
         generate_images_dir(dir)
         generate_cover_image(dir)
-        generate_layout(dir)
         generate_style(dir)
         generate_texmacro(dir)
         generate_config(dir)
+        generate_gitignore(dir)
         generate_locale(dir) if @locale
         generate_rakefile(dir)
         generate_gemfile(dir)
@@ -50,6 +60,9 @@ module ReVIEW
     end
 
     def parse_options(args)
+      @port = 18000
+      @bind = '0'
+
       opts = OptionParser.new
       opts.version = ReVIEW::VERSION
       opts.banner = "Usage: #{File.basename($PROGRAM_NAME)} [option] dirname"
@@ -63,23 +76,35 @@ module ReVIEW
       opts.on('-l', '--locale', 'generate locale.yml file.') do
         @locale = true
       end
-      opts.on('--latex-template name', 'specify LaTeX template name. (default: review-jsbook)') do |tname|
+      opts.on('-t', '--latex-template name', 'specify LaTeX template name. (default: review-jsbook)') do |tname|
         @template = tname
       end
       opts.on('', '--epub-version VERSION', 'define EPUB version.') do |version|
         @epub_version = version
       end
+      opts.on('', '--without-config-comment', "don't include comments in config.yml.") do
+        @without_config_comment = true
+      end
       opts.on('', '--without-doc', "don't generate doc files.") do
         @without_doc = true
       end
       opts.on('-p', '--package archivefile', 'extract from local or network archive.') do |archive|
         @archive = archive
       end
+      opts.on('-w', '--wizard', 'launch Web based layout configuration.') do
+        @webui = true
+      end
+      opts.on('', '--port port', 'port to use for Web based layout configuration. (default: 18000)') do |port|
+        @port = port
+      end
+      opts.on('', '--bind bindaddress', 'address to use for Web based layout configuration. (default: 0 (any))') do |bind|
+        @bind = bind
+      end
 
       begin
         opts.parse!(args)
-      rescue OptionParser::ParseError => err
-        @logger.error err.message
+      rescue OptionParser::ParseError => e
+        @logger.error e.message
         $stderr.puts opts.help
         exit 1
       end
@@ -89,9 +114,7 @@ module ReVIEW
         exit 1
       end
 
-      initdir = File.expand_path(args[0])
-
-      initdir
+      File.expand_path(args[0])
     end
 
     def generate_dir(dir)
@@ -99,7 +122,7 @@ module ReVIEW
         @logger.error "#{dir} already exists."
         exit 1
       end
-      FileUtils.mkdir_p dir
+      FileUtils.mkdir_p(dir)
       yield dir
     end
 
@@ -109,13 +132,8 @@ module ReVIEW
       end
     end
 
-    def generate_layout(dir)
-      FileUtils.mkdir_p File.join(dir, 'layouts')
-    end
-
     def generate_catalog_file(dir)
-      File.open(File.join(dir, 'catalog.yml'), 'w') do |file|
-        file.write <<-EOS
+      File.write(File.join(dir, 'catalog.yml'), <<-EOS)
 PREDEF:
 
 CHAPS:
@@ -126,7 +144,6 @@ APPENDIX:
 POSTDEF:
 
 EOS
-      end
     end
 
     def generate_images_dir(dir)
@@ -154,64 +171,82 @@ EOS
         content.gsub!(/^#.*htmlversion:.*$/, 'htmlversion: 4')
       end
 
-      if TEX_DOCUMENTCLASS_OPTS[@template]
-        content.gsub!(/^#\s*texdocumentclass:.*$/, %Q(texdocumentclass: ["#{@template}", "#{TEX_DOCUMENTCLASS_OPTS[@template]}"]))
+      if @tex_documentclass_opts[@template]
+        content.gsub!(/^#\s*texdocumentclass:.*$/, %Q(texdocumentclass: ["#{@template}", "#{@tex_documentclass_opts[@template]}"]))
+      end
+
+      if @without_config_comment
+        content = content.split("\n").delete_if { |l| l.strip.start_with?('#') || l.strip.empty? }.join("\n")
+      end
+
+      File.open(File.join(dir, 'config.yml'), 'w') { |f| f.write content }
+      if @webui && !@web_result[2].empty?
+        File.open(File.join(dir, 'config-ebook.yml'), 'w') do |f|
+          f.puts <<EOT
+# for ebook PDF
+# REVIEW_CONFIG_FILE=config.yml rake pdf
+inherit: ["config.yml"]
+# bookname: book-ebook
+texdocumentclass: ["#{@web_result[0]}", "#{@web_result[2]}"]
+EOT
+        end
       end
+    end
 
-      File.open(File.join(dir, 'config.yml'), 'w') { |f| f.write(content) }
+    def generate_gitignore(dir)
+      FileUtils.cp(File.join(@review_dir, 'samples/sample-book/src/.gitignore'), dir)
     end
 
     def generate_style(dir)
-      FileUtils.cp File.join(@review_dir, 'samples/sample-book/src/style.css'), dir
+      FileUtils.cp(File.join(@review_dir, 'samples/sample-book/src/style.css'), dir)
     end
 
     def generate_texmacro(dir)
       texmacrodir = File.join(dir, 'sty')
-      FileUtils.mkdir_p texmacrodir
+      FileUtils.mkdir_p(texmacrodir)
       tdir = File.join(@review_dir, 'templates/latex', @template)
       @logger.error "#{tdir} not found." unless File.exist?(tdir)
-      FileUtils.cp Dir.glob(File.join(tdir, '*.*')), texmacrodir
-      # provide jsbook from vendor/. current version is 2018/06/23
-      FileUtils.cp File.join(@review_dir, 'vendor/jsclasses/jsbook.cls'), File.join(texmacrodir, 'jsbook.cls')
-      # provide gentombow from vendor/. current version is 2018/08/30 v0.9j
-      FileUtils.cp File.join(@review_dir, 'vendor/gentombow/gentombow.sty'), File.join(texmacrodir, 'gentombow.sty')
+      FileUtils.cp(Dir.glob(File.join(tdir, '*.*')), texmacrodir)
+
+      if @template == 'review-jsbook'
+        # provide jsbook from vendor/. current version is 2018/06/23
+        FileUtils.cp(File.join(@review_dir, 'vendor/jsclasses/jsbook.cls'), File.join(texmacrodir, 'jsbook.cls'))
+        # provide gentombow from vendor/. current version is 2018/08/30 v0.9j
+        FileUtils.cp(File.join(@review_dir, 'vendor/gentombow/gentombow.sty'), File.join(texmacrodir, 'gentombow.sty'))
+      end
     end
 
     def generate_rakefile(dir)
-      FileUtils.mkdir_p File.join(dir, 'lib/tasks')
+      FileUtils.mkdir_p(File.join(dir, 'lib/tasks'))
 
-      File.open(File.join(dir, 'Rakefile'), 'w') do |file|
-        file.write <<-EOS
+      File.write(File.join(dir, 'Rakefile'), <<-EOS)
 Dir.glob('lib/tasks/*.rake').sort.each do |file|
   load(file)
 end
 EOS
-      end
 
       FileUtils.cp(File.join(@review_dir, 'samples/sample-book/src/lib/tasks/review.rake'),
                    File.join(dir, 'lib/tasks/review.rake'))
     end
 
     def generate_locale(dir)
-      FileUtils.cp File.join(@review_dir, 'lib/review/i18n.yml'), File.join(dir, 'locale.yml')
+      FileUtils.cp(File.join(@review_dir, 'lib/review/i18n.yml'), File.join(dir, 'locale.yml'))
     end
 
     def generate_gemfile(dir)
-      File.open(File.join(dir, 'Gemfile'), 'w') do |file|
-        file.write <<-EOS
+      File.write(File.join(dir, 'Gemfile'), <<-EOS)
 source 'https://rubygems.org'
 
 gem 'rake'
 gem 'review', '#{ReVIEW::VERSION}'
 EOS
-      end
     end
 
     def generate_doc(dir)
       docdir = File.join(dir, 'doc')
-      FileUtils.mkdir_p docdir
+      FileUtils.mkdir_p(docdir)
       md_files = Dir.glob(File.join(@review_dir, 'doc/*.md')).map.to_a
-      FileUtils.cp md_files, docdir
+      FileUtils.cp(md_files, docdir)
     end
 
     def download_and_extract_archive(dir, filename)
@@ -226,8 +261,8 @@ EOS
         begin
           @logger.info "Downloading from #{filename}"
           zipdata = Net::HTTP.get(URI.parse(filename))
-        rescue StandardError => err
-          @logger.error "Failed to download #{filename}: #{err.message}"
+        rescue StandardError => e
+          @logger.error "Failed to download #{filename}: #{e.message}"
           exit 1
         end
 
@@ -256,7 +291,7 @@ EOS
               next
             end
 
-            if fname =~ %r{\A/} || fname =~ /\.\./ # simple fool proof
+            if fname.start_with?('/') || fname =~ /\.\./ # simple fool proof
               made = nil
               break
             end
@@ -267,8 +302,56 @@ EOS
           made = true
         end
         raise Zip::Error unless made
-      rescue Zip::Error => err
-        @logger.error "#{originalfilename} seems invalid or broken zip file: #{err.message}"
+      rescue Zip::Error => e
+        @logger.error "#{originalfilename} seems invalid or broken zip file: #{e.message}"
+      end
+    end
+
+    def start_webui
+      require 'webrick'
+      web_config = {
+        BindAddress: @bind,
+        Port: @port,
+        HTTPVersion: WEBrick::HTTPVersion.new('1.1'),
+        AccessLog: [[File.open(IO::NULL, 'w'), '']]
+      }
+
+      bind_address = (@bind == '0') ? '<thishost>' : @bind
+      puts "Please access http://#{bind_address}:#{web_config[:Port]} from Web browser."
+      begin
+        @web_server = WEBrick::HTTPServer.new(web_config)
+      rescue StandardError => e
+        @logger.error "Error: #{e}"
+        exit 1
+      end
+
+      @web_result = nil
+      web_mounts
+
+      trap(:INT) { @web_server.shutdown }
+      @web_server.start
+
+      # validation
+      if @web_result
+        @web_result.each do |s|
+          if s !~ /\A[a-z0-9=_,.-]*\Z/i
+            @web_result = nil
+            break
+          end
+        end
+      end
+    end
+
+    def web_mounts
+      htmldir = File.join(@review_dir, 'lib', 'review', 'init-web')
+      @web_server.mount('/', WEBrick::HTTPServlet::FileHandler, File.join(htmldir, 'index.html'))
+      @web_server.mount('/review-layout-design.js', WEBrick::HTTPServlet::FileHandler, File.join(htmldir, 'review-layout-design.js'))
+
+      @web_server.mount_proc('/finish') do |req, res|
+        @web_result = [req.query['cls'], req.query['result_print'], req.query['result_ebook']]
+        res.content_type = 'text/html'
+        res.body = File.read(File.join(htmldir, 'finish.html'))
+        @web_server.shutdown
       end
     end
   end

data/lib/review/latexbuilder.rb

@@ -1,6 +1,6 @@
 # Copyright (c) 2002-2007 Minero Aoki
 #               2008-2009 Minero Aoki, Kenshi Muto
-#               2010-2019 Minero Aoki, Kenshi Muto, TAKAHASHI Masayoshi
+#               2010-2020 Minero Aoki, Kenshi Muto, TAKAHASHI Masayoshi
 #
 # This program is free software.
 # You can distribute or modify this program under the terms of
@@ -32,7 +32,6 @@ module ReVIEW
       @blank_needed = false
       @latex_tsize = nil
       @tsize = nil
-      @table_caption = nil
       @cellwidth = nil
       @ol_num = nil
       @first_line_num = nil
@@ -61,7 +60,7 @@ module ReVIEW
         require 'nkf'
         @index_mecab = MeCab::Tagger.new(@book.config['pdfmaker']['makeindex_mecab_opts'])
       rescue LoadError
-        error 'not found MeCab'
+        warn 'not found MeCab'
       end
     end
 
@@ -101,7 +100,21 @@ module ReVIEW
       if @chapter.is_a?(ReVIEW::Book::Part) && !@book.config.check_version('2', exception: false)
         puts '\end{reviewpart}'
       end
-      @output.string
+      solve_nest(@output.string)
+    end
+
+    def solve_nest(s)
+      check_nest
+      s.gsub("\\end{description}\n\n\x01→dl←\x01\n", "\n").
+        gsub("\x01→/dl←\x01", "\\end{description}←END\x01").
+        gsub("\\end{itemize}\n\n\x01→ul←\x01\n", "\n").
+        gsub("\x01→/ul←\x01", "\\end{itemize}←END\x01").
+        gsub("\\end{enumerate}\n\n\x01→ol←\x01\n", "\n").
+        gsub("\x01→/ol←\x01", "\\end{enumerate}←END\x01").
+        gsub("\\end{description}←END\x01\n\n\\begin{description}", '').
+        gsub("\\end{itemize}←END\x01\n\n\\begin{itemize}", '').
+        gsub("\\end{enumerate}←END\x01\n\n\\begin{enumerate}", '').
+        gsub("←END\x01", '')
     end
 
     HEADLINE = {
@@ -213,7 +226,53 @@ module ReVIEW
       @doc_status[:column] = nil
     end
 
+    def common_block_begin(type, caption = nil)
+      check_nested_minicolumn
+      if @book.config.check_version('2', exception: false)
+        type = 'minicolumn'
+      end
+
+      @doc_status[:minicolumn] = type
+      print "\\begin{review#{type}}"
+
+      @doc_status[:caption] = true
+      if @book.config.check_version('2', exception: false)
+        puts
+        if caption.present?
+          puts "\\reviewminicolumntitle{#{compile_inline(caption)}}"
+        end
+      else
+        if caption.present?
+          print "[#{compile_inline(caption)}]"
+        end
+        puts
+      end
+      @doc_status[:caption] = nil
+    end
+
+    def common_block_end(type)
+      if @book.config.check_version('2', exception: false)
+        type = 'minicolumn'
+      end
+
+      puts "\\end{review#{type}}"
+      @doc_status[:minicolumn] = nil
+    end
+
+    CAPTION_TITLES.each do |name|
+      class_eval %Q(
+        def #{name}_begin(caption = nil)
+          common_block_begin('#{name}', caption)
+        end
+
+        def #{name}_end
+          common_block_end('#{name}')
+        end
+      ), __FILE__, __LINE__ - 8
+    end
+
     def captionblock(type, lines, caption)
+      check_nested_minicolumn
       if @book.config.check_version('2', exception: false)
         type = 'minicolumn'
       end
@@ -257,7 +316,11 @@ module ReVIEW
     end
 
     def ul_item(lines)
-      str = lines.join
+      str = join_lines_to_paragraph(lines)
+      unless @book.config['join_lines_by_lang']
+        str = lines.map(&:chomp).join("\n")
+      end
+
       str.sub!(/\A(\[)/) { '\lbrack{}' }
       puts '\item ' + str
     end
@@ -276,7 +339,11 @@ module ReVIEW
     end
 
     def ol_item(lines, _num)
-      str = lines.join
+      str = join_lines_to_paragraph(lines)
+      unless @book.config['join_lines_by_lang']
+        str = lines.map(&:chomp).join("\n")
+      end
+
       str.sub!(/\A(\[)/) { '\lbrack{}' }
       puts '\item ' + str
     end
@@ -292,13 +359,16 @@ module ReVIEW
     end
 
     def dt(str)
-      str.sub!(/\[/) { '\lbrack{}' }
-      str.sub!(/\]/) { '\rbrack{}' }
+      str = str.gsub('[', '\lbrack{}').gsub(']', '\rbrack{}')
       puts '\item[' + str + '] \mbox{} \\\\'
     end
 
     def dd(lines)
-      puts lines.join
+      if @book.config['join_lines_by_lang']
+        puts join_lines_to_paragraph(lines)
+      else
+        puts lines.map(&:chomp).join("\n")
+      end
     end
 
     def dl_end
@@ -308,8 +378,10 @@ module ReVIEW
 
     def paragraph(lines)
       blank
-      lines.each do |line|
-        puts line
+      if @book.config['join_lines_by_lang']
+        puts join_lines_to_paragraph(lines)
+      else
+        lines.each { |line| puts line }
       end
       blank
     end
@@ -319,22 +391,35 @@ module ReVIEW
     end
 
     def read(lines)
-      latex_block 'quotation', lines
+      latex_block('quotation', lines)
     end
 
     alias_method :lead, :read
 
+    def highlight?
+      @book.config['highlight'] &&
+        @book.config['highlight']['latex']
+    end
+
     def highlight_listings?
       @book.config['highlight'] && @book.config['highlight']['latex'] == 'listings'
     end
     private :highlight_listings?
 
+    def code_line(_type, line, _idx, _id, _caption, _lang)
+      detab(line) + "\n"
+    end
+
+    def code_line_num(_type, line, first_line_num, idx, _id, _caption, _lang)
+      detab((idx + first_line_num).to_s.rjust(2) + ': ' + line) + "\n"
+    end
+
     def emlist(lines, caption = nil, lang = nil)
       blank
       if highlight_listings?
         common_code_block_lst(nil, lines, 'reviewemlistlst', 'title', caption, lang)
       else
-        common_code_block(nil, lines, 'reviewemlist', caption, lang) { |line, _idx| detab(line) + "\n" }
+        common_code_block(nil, lines, 'reviewemlist', caption, lang) { |line, idx| code_line('emlist', line, idx, nil, caption, lang) }
       end
     end
 
@@ -344,7 +429,7 @@ module ReVIEW
       if highlight_listings?
         common_code_block_lst(nil, lines, 'reviewemlistnumlst', 'title', caption, lang, first_line_num: first_line_num)
       else
-        common_code_block(nil, lines, 'reviewemlist', caption, lang) { |line, idx| detab((idx + first_line_num).to_s.rjust(2) + ': ' + line) + "\n" }
+        common_code_block(nil, lines, 'reviewemlist', caption, lang) { |line, idx| code_line_num('emlistnum', line, first_line_num, idx, nil, caption, lang) }
       end
     end
 
@@ -353,7 +438,7 @@ module ReVIEW
       if highlight_listings?
         common_code_block_lst(id, lines, 'reviewlistlst', 'caption', caption, lang)
       else
-        common_code_block(id, lines, 'reviewlist', caption, lang) { |line, _idx| detab(line) + "\n" }
+        common_code_block(id, lines, 'reviewlist', caption, lang) { |line, idx| code_line('list', line, idx, id, caption, lang) }
       end
     end
 
@@ -363,7 +448,7 @@ module ReVIEW
       if highlight_listings?
         common_code_block_lst(id, lines, 'reviewlistnumlst', 'caption', caption, lang, first_line_num: first_line_num)
       else
-        common_code_block(id, lines, 'reviewlist', caption, lang) { |line, idx| detab((idx + first_line_num).to_s.rjust(2) + ': ' + line) + "\n" }
+        common_code_block(id, lines, 'reviewlist', caption, lang) { |line, idx| code_line_num('listnum', line, first_line_num, idx, id, caption, lang) }
       end
     end
 
@@ -372,24 +457,25 @@ module ReVIEW
         common_code_block_lst(nil, lines, 'reviewcmdlst', 'title', caption, lang)
       else
         blank
-        common_code_block(nil, lines, 'reviewcmd', caption, lang) { |line, _idx| detab(line) + "\n" }
+        common_code_block(nil, lines, 'reviewcmd', caption, lang) { |line, idx| code_line('cmd', line, idx, nil, caption, lang) }
       end
     end
 
     def common_code_block(id, lines, command, caption, _lang)
       @doc_status[:caption] = true
+      captionstr = nil
       unless @book.config.check_version('2', exception: false)
         puts '\\begin{reviewlistblock}'
       end
       if caption.present?
         if command =~ /emlist/ || command =~ /cmd/ || command =~ /source/
-          puts macro(command + 'caption', compile_inline(caption))
+          captionstr = macro(command + 'caption', compile_inline(caption))
         else
           begin
             if get_chap.nil?
-              puts macro('reviewlistcaption', "#{I18n.t('list')}#{I18n.t('format_number_header_without_chapter', [@chapter.list(id).number])}#{I18n.t('caption_prefix')}#{compile_inline(caption)}")
+              captionstr = macro('reviewlistcaption', "#{I18n.t('list')}#{I18n.t('format_number_header_without_chapter', [@chapter.list(id).number])}#{I18n.t('caption_prefix')}#{compile_inline(caption)}")
             else
-              puts macro('reviewlistcaption', "#{I18n.t('list')}#{I18n.t('format_number_header', [get_chap, @chapter.list(id).number])}#{I18n.t('caption_prefix')}#{compile_inline(caption)}")
+              captionstr = macro('reviewlistcaption', "#{I18n.t('list')}#{I18n.t('format_number_header', [get_chap, @chapter.list(id).number])}#{I18n.t('caption_prefix')}#{compile_inline(caption)}")
             end
           rescue KeyError
             error "no such list: #{id}"
@@ -397,6 +483,11 @@ module ReVIEW
         end
       end
       @doc_status[:caption] = nil
+
+      if caption_top?('list') && captionstr
+        puts captionstr
+      end
+
       body = ''
       lines.each_with_index do |line, idx|
         body.concat(yield(line, idx))
@@ -404,6 +495,11 @@ module ReVIEW
       puts macro('begin', command)
       print body
       puts macro('end', command)
+
+      if !caption_top?('list') && captionstr
+        puts captionstr
+      end
+
       unless @book.config.check_version('2', exception: false)
         puts '\\end{reviewlistblock}'
       end
@@ -414,7 +510,7 @@ module ReVIEW
       if title == 'title' && caption.blank? && @book.config.check_version('2', exception: false)
         print '\vspace{-1.5em}'
       end
-      body = lines.inject('') { |i, j| i + detab(unescape(j)) + "\n" }
+      body = lines.inject('') { |i, j| i + detab(j) + "\n" }
       args = make_code_block_args(title, caption, lang, first_line_num: first_line_num)
       puts %Q(\\begin{#{command}}[#{args}])
       print body
@@ -449,15 +545,23 @@ module ReVIEW
       if highlight_listings?
         common_code_block_lst(nil, lines, 'reviewsourcelst', 'title', caption, lang)
       else
-        common_code_block(nil, lines, 'reviewsource', caption, lang) { |line, _idx| detab(line) + "\n" }
+        common_code_block(nil, lines, 'reviewsource', caption, lang) { |line, idx| code_line('source', line, idx, nil, caption, lang) }
       end
     end
 
     def image_header(id, caption)
     end
 
+    def parse_metric(type, metric)
+      s = super(type, metric)
+      if @book.config['pdfmaker']['use_original_image_size'] && s.empty? && !metric.present?
+        return ' ' # pass empty to \reviewincludegraphics
+      end
+      s
+    end
+
     def handle_metric(str)
-      if @book.config['image_scale2width'] && str =~ /\Ascale=([\d.]+)\Z/
+      if @book.config['pdfmaker']['image_scale2width'] && str =~ /\Ascale=([\d.]+)\Z/
         return "width=#{$1}\\maxwidth"
       end
       str
@@ -468,23 +572,39 @@ module ReVIEW
     end
 
     def image_image(id, caption, metric)
+      captionstr = nil
+      @doc_status[:caption] = true
+      if @book.config.check_version('2', exception: false)
+        captionstr = macro('caption', compile_inline(caption)) + "\n" if caption.present?
+      else
+        captionstr = macro('reviewimagecaption', compile_inline(caption)) + "\n" if caption.present?
+      end
+      captionstr << macro('label', image_label(id))
+      @doc_status[:caption] = nil
+
       metrics = parse_metric('latex', metric)
       # image is always bound here
       puts "\\begin{reviewimage}%%#{id}"
-      if metrics.present?
-        puts "\\includegraphics[#{metrics}]{#{@chapter.image(id).path}}"
-      else
-        puts "\\includegraphics[width=\\maxwidth]{#{@chapter.image(id).path}}"
+
+      if caption_top?('image') && captionstr
+        puts captionstr
       end
-      @doc_status[:caption] = true
 
+      command = 'reviewincludegraphics'
       if @book.config.check_version('2', exception: false)
-        puts macro('caption', compile_inline(caption)) if caption.present?
+        command = 'includegraphics'
+      end
+
+      if metrics.present?
+        puts "\\#{command}[#{metrics}]{#{@chapter.image(id).path}}"
       else
-        puts macro('reviewimagecaption', compile_inline(caption)) if caption.present?
+        puts "\\#{command}[width=\\maxwidth]{#{@chapter.image(id).path}}"
       end
-      @doc_status[:caption] = nil
-      puts macro('label', image_label(id))
+
+      if !caption_top?('image') && captionstr
+        puts captionstr
+      end
+
       puts '\end{reviewimage}'
     end
 
@@ -508,7 +628,7 @@ module ReVIEW
     end
 
     def existence(id)
-      @chapter.image(id).bound? ? 'exist' : 'not exist'
+      @chapter.image_bound?(id) ? 'exist' : 'not exist'
     end
     private :existence
 
@@ -539,7 +659,8 @@ module ReVIEW
     end
     private :bib_label
 
-    def column_label(id, chapter = @chapter)
+    def column_label(id, chapter = nil)
+      chapter ||= @chapter
       filename = chapter.id
       num = chapter.column(id).number
       "column:#{filename}:#{num}"
@@ -549,28 +670,43 @@ module ReVIEW
     def indepimage(lines, id, caption = nil, metric = nil)
       metrics = parse_metric('latex', metric)
 
+      captionstr = nil
+      if caption.present?
+        @doc_status[:caption] = true
+        captionstr = macro('reviewindepimagecaption',
+                           %Q(#{I18n.t('numberless_image')}#{I18n.t('caption_prefix')}#{compile_inline(caption)}))
+        @doc_status[:caption] = nil
+      end
+
       if @chapter.image(id).path
         puts "\\begin{reviewimage}%%#{id}"
+
+        if caption_top?('image') && captionstr
+          puts captionstr
+        end
+
+        command = 'reviewincludegraphics'
+        if @book.config.check_version('2', exception: false)
+          command = 'includegraphics'
+        end
+
         if metrics.present?
-          puts "\\includegraphics[#{metrics}]{#{@chapter.image(id).path}}"
+          puts "\\#{command}[#{metrics}]{#{@chapter.image(id).path}}"
         else
-          puts "\\includegraphics[width=\\maxwidth]{#{@chapter.image(id).path}}"
+          puts "\\#{command}[width=\\maxwidth]{#{@chapter.image(id).path}}"
         end
       else
         warn "image not bound: #{id}"
         puts '\begin{reviewdummyimage}'
-        puts "--[[path = #{id} (#{existence(id)})]]--"
+        puts "--[[path = #{escape(id)} (#{existence(id)})]]--"
         lines.each do |line|
           puts detab(line.rstrip)
         end
       end
 
-      @doc_status[:caption] = true
-      if caption.present?
-        puts macro('reviewindepimagecaption',
-                   %Q(#{I18n.t('numberless_image')}#{I18n.t('caption_prefix')}#{compile_inline(caption)}))
+      if !caption_top?('image') && captionstr
+        puts captionstr
       end
-      @doc_status[:caption] = nil
 
       if @chapter.image(id).path
         puts '\end{reviewimage}'
@@ -582,26 +718,35 @@ module ReVIEW
     alias_method :numberlessimage, :indepimage
 
     def table(lines, id = nil, caption = nil)
-      rows = []
-      sepidx = nil
-      lines.each_with_index do |line, idx|
-        if /\A[\=\{\-\}]{12}/ =~ line
-          # just ignore
-          # error "too many table separator" if sepidx
-          sepidx ||= idx
-          next
+      if caption.present?
+        if @book.config.check_version('2', exception: false)
+          puts "\\begin{table}[h]%%#{id}"
+        else
+          puts "\\begin{table}%%#{id}"
         end
-        rows.push(line.strip.split(/\t+/).map { |s| s.sub(/\A\./, '') })
       end
-      rows = adjust_n_cols(rows)
 
+      sepidx, rows = parse_table_rows(lines)
       begin
-        table_header(id, caption) if caption.present?
+        if caption_top?('table') && caption.present?
+          table_header(id, caption)
+        end
       rescue KeyError
         error "no such table: #{id}"
       end
-      return if rows.empty?
       table_begin(rows.first.size)
+      table_rows(sepidx, rows)
+      table_end
+      if caption.present?
+        unless caption_top?('table')
+          table_header(id, caption)
+        end
+        puts '\end{table}'
+      end
+      blank
+    end
+
+    def table_rows(sepidx, rows)
       if sepidx
         sepidx.times do
           cno = -1
@@ -628,31 +773,18 @@ module ReVIEW
              end)
         end
       end
-      table_end
     end
 
     def table_header(id, caption)
       if id.nil?
         if caption.present?
-          @table_caption = true
           @doc_status[:caption] = true
-          if @book.config.check_version('2', exception: false)
-            puts "\\begin{table}[h]%%#{id}"
-          else
-            puts "\\begin{table}%%#{id}"
-          end
           puts macro('reviewtablecaption*', compile_inline(caption))
           @doc_status[:caption] = nil
         end
       else
         if caption.present?
-          @table_caption = true
           @doc_status[:caption] = true
-          if @book.config.check_version('2', exception: false)
-            puts "\\begin{table}[h]%%#{id}"
-          else
-            puts "\\begin{table}%%#{id}"
-          end
           puts macro('reviewtablecaption', compile_inline(caption))
           @doc_status[:caption] = nil
         end
@@ -755,12 +887,9 @@ module ReVIEW
 
     def table_end
       puts macro('end', 'reviewtable')
-      puts '\end{table}' if @table_caption
-      @table_caption = nil
       @tsize = nil
       @latex_tsize = nil
       @cellwidth = nil
-      blank
     end
 
     def emtable(lines, caption = nil)
@@ -768,19 +897,22 @@ module ReVIEW
     end
 
     def imgtable(lines, id, caption = nil, metric = nil)
-      unless @chapter.image(id).bound?
+      unless @chapter.image_bound?(id)
         warn "image not bound: #{id}"
-        image_dummy id, caption, lines
+        image_dummy(id, caption, lines)
         return
       end
 
+      captionstr = nil
       begin
         if caption.present?
-          @table_caption = true
-          @doc_status[:caption] = true
           puts "\\begin{table}[h]%%#{id}"
-          puts macro('reviewimgtablecaption', compile_inline(caption))
+          @doc_status[:caption] = true
+          captionstr = macro('reviewimgtablecaption', compile_inline(caption))
           @doc_status[:caption] = nil
+          if caption_top?('table')
+            puts captionstr
+          end
         end
         puts macro('label', table_label(id))
       rescue ReVIEW::KeyError
@@ -788,8 +920,12 @@ module ReVIEW
       end
       imgtable_image(id, caption, metric)
 
-      puts '\end{table}' if @table_caption
-      @table_caption = nil
+      if caption.present?
+        unless caption_top?('table')
+          puts captionstr
+        end
+        puts '\end{table}'
+      end
       blank
     end
 
@@ -797,46 +933,61 @@ module ReVIEW
       metrics = parse_metric('latex', metric)
       # image is always bound here
       puts "\\begin{reviewimage}%%#{id}"
+
+      command = 'reviewincludegraphics'
+      if @book.config.check_version('2', exception: false)
+        command = 'includegraphics'
+      end
+
       if metrics.present?
-        puts "\\includegraphics[#{metrics}]{#{@chapter.image(id).path}}"
+        puts "\\#{command}[#{metrics}]{#{@chapter.image(id).path}}"
       else
-        puts "\\includegraphics[width=\\maxwidth]{#{@chapter.image(id).path}}"
+        puts "\\#{command}[width=\\maxwidth]{#{@chapter.image(id).path}}"
       end
       puts '\end{reviewimage}'
     end
 
     def quote(lines)
-      latex_block 'quote', lines
+      latex_block('quote', lines)
     end
 
     def center(lines)
-      latex_block 'center', lines
+      latex_block('center', lines)
     end
 
     alias_method :centering, :center
 
     def flushright(lines)
-      latex_block 'flushright', lines
+      latex_block('flushright', lines)
     end
 
     def texequation(lines, id = nil, caption = '')
       blank
+      captionstr = nil
 
       if id
         puts macro('begin', 'reviewequationblock')
         if get_chap.nil?
-          puts macro('reviewequationcaption', "#{I18n.t('equation')}#{I18n.t('format_number_header_without_chapter', [@chapter.equation(id).number])}#{I18n.t('caption_prefix')}#{compile_inline(caption)}")
+          captionstr = macro('reviewequationcaption', "#{I18n.t('equation')}#{I18n.t('format_number_header_without_chapter', [@chapter.equation(id).number])}#{I18n.t('caption_prefix')}#{compile_inline(caption)}")
         else
-          puts macro('reviewequationcaption', "#{I18n.t('equation')}#{I18n.t('format_number_header', [get_chap, @chapter.equation(id).number])}#{I18n.t('caption_prefix')}#{compile_inline(caption)}")
+          captionstr = macro('reviewequationcaption', "#{I18n.t('equation')}#{I18n.t('format_number_header', [get_chap, @chapter.equation(id).number])}#{I18n.t('caption_prefix')}#{compile_inline(caption)}")
         end
       end
 
+      if caption_top?('equation') && captionstr
+        puts captionstr
+      end
+
       puts macro('begin', 'equation*')
       lines.each do |line|
-        puts unescape(line)
+        puts line
       end
       puts macro('end', 'equation*')
 
+      if !caption_top?('equation') && captionstr
+        puts captionstr
+      end
+
       if id
         puts macro('end', 'reviewequationblock')
       end
@@ -865,7 +1016,7 @@ module ReVIEW
       return true unless @book.config['draft']
       lines ||= []
       unless comment.blank?
-        lines.unshift escape(comment)
+        lines.unshift(escape(comment))
       end
       str = lines.join('\par ')
       puts macro('pdfcomment', str)
@@ -977,6 +1128,9 @@ module ReVIEW
 
     def footnote(id, content)
       if @book.config['footnotetext'] || @foottext[id]
+        if @doc_status[:column]
+          warn "//footnote[#{id}] is in the column block. It is recommended to move out of the column block."
+        end
         puts macro("footnotetext[#{@chapter.footnote(id).number}]", compile_inline(content.strip))
       end
     end
@@ -984,7 +1138,7 @@ module ReVIEW
     def inline_fn(id)
       if @book.config['footnotetext']
         macro("footnotemark[#{@chapter.footnote(id).number}]", '')
-      elsif @doc_status[:caption] || @doc_status[:table] || @doc_status[:column]
+      elsif @doc_status[:caption] || @doc_status[:table] || @doc_status[:column] || @doc_status[:dt]
         @foottext[id] = @chapter.footnote(id).number
         macro('protect\\footnotemark', '')
       else
@@ -997,7 +1151,7 @@ module ReVIEW
     BOUTEN = '・'.freeze
 
     def inline_bou(str)
-      str.split(//).map { |c| macro('ruby', escape(c), macro('textgt', BOUTEN)) }.join('\allowbreak')
+      macro('reviewbou', escape(str))
     end
 
     def compile_ruby(base, ruby)
@@ -1103,7 +1257,7 @@ module ReVIEW
 
     def inline_hd_chap(chap, id)
       n = chap.headline_index.number(id)
-      if chap.number and @book.config['secnolevel'] >= n.split('.').size
+      if n.present? && chap.number && over_secnolevel?(n)
         str = I18n.t('hd_quote', [chap.headline_index.number(id), compile_inline(chap.headline(id).caption)])
       else
         str = I18n.t('hd_quote_without_number', compile_inline(chap.headline(id).caption))
@@ -1124,7 +1278,7 @@ module ReVIEW
       error "unknown column: #{id}"
     end
 
-    def inline_raw(str)
+    def inline_raw(str) # rubocop:disable Lint/UselessMethodDefinition
       super(str)
     end
 
@@ -1154,7 +1308,11 @@ module ReVIEW
 
     def inline_icon(id)
       if @chapter.image(id).path
-        macro('includegraphics', @chapter.image(id).path)
+        command = 'reviewincludegraphics'
+        if @book.config.check_version('2', exception: false)
+          command = 'includegraphics'
+        end
+        macro(command, @chapter.image(id).path)
       else
         warn "image not bound: #{id}"
         "\\verb|--[[path = #{id} (#{existence(id)})]]--|"
@@ -1193,7 +1351,12 @@ module ReVIEW
     end
 
     def bibpaper_bibpaper(_id, _caption, lines)
-      print split_paragraph(lines).join
+      if @book.config['join_lines_by_lang']
+        print split_paragraph(lines).join("\n\n")
+      else
+        print split_paragraph(lines).map(&:chomp).join("\n")
+      end
+
       puts ''
     end