review 2.4.0 → 2.5.0

data/bin/review-epubmaker

@@ -33,11 +33,16 @@ rescue OptionParser::ParseError => err
   exit 1
 end
 
-if ARGV.size < 1 || !File.exist?(ARGV[0])
+if ARGV.size < 1
   puts opts.help
   exit 1
 end
 
+unless File.exist?(ARGV[0])
+  @logger.error "#{File.basename($PROGRAM_NAME, '.*')}: #{ARGV[0]} not found."
+  exit 1
+end
+
 yaml_file = ARGV[0]
 bookname = ARGV[1]
 rv.produce(yaml_file, bookname)

data/bin/review-init

@@ -29,6 +29,7 @@ def main
   opts.on('-f', '--force', 'generate files (except *.re) if directory has already existed.') { @force = true }
   opts.on('-l', '--locale', 'generate locale.yml file.') { @locale = true }
   opts.on('', '--epub-version VERSION', 'define EPUB version') { |version| @epub_version = version }
+  opts.on('', '--without-doc', "don't generate doc files") { @without_doc = true }
   begin
     opts.parse!
   rescue OptionParser::ParseError => err
@@ -57,6 +58,7 @@ def main
     generate_locale(dir) if @locale
     generate_rakefile(dir)
     generate_gemfile(dir)
+    generate_doc(dir) unless @without_doc
   end
 end
 
@@ -130,7 +132,18 @@ def generate_texmacro(dir)
 end
 
 def generate_rakefile(dir)
-  FileUtils.cp @review_dir + '/test/sample-book/src/Rakefile', dir
+  FileUtils.mkdir_p dir + '/lib/tasks'
+
+  File.open(dir + '/Rakefile', 'w') do |file|
+    file.write <<-EOS
+Dir.glob('lib/tasks/*.rake').each do |file|
+  load(file)
+end
+EOS
+  end
+
+  FileUtils.cp @review_dir + '/test/sample-book/src/Rakefile',
+               dir + '/lib/tasks/review.rake'
 end
 
 def generate_locale(dir)
@@ -148,4 +161,11 @@ EOS
   end
 end
 
+def generate_doc(dir)
+  docdir = dir + '/doc'
+  FileUtils.mkdir_p docdir
+  md_files = Dir.glob(@review_dir + '/doc/*.md').map.to_a
+  FileUtils.cp md_files, docdir
+end
+
 main if File.basename($PROGRAM_NAME) == File.basename(__FILE__)

data/bin/review-textmaker

@@ -0,0 +1,16 @@
+#!/usr/bin/env ruby
+# Copyright (c) 2018 Kenshi Muto
+#
+# This program is free software.
+# You can distribute or modify this program under the terms of
+# the GNU LGPL, Lesser General Public License version 2.1.
+# For details of the GNU LGPL, see the file "COPYING".
+#
+
+require 'pathname'
+bindir = Pathname.new(__FILE__).realpath.dirname
+$LOAD_PATH.unshift((bindir + '../lib').realpath)
+
+require 'review/textmaker'
+
+ReVIEW::TEXTMaker.execute(*ARGV)

data/doc/config.yml.sample

@@ -314,9 +314,14 @@ pdfmaker:
   # 渡される引数1=作業用展開ディレクトリ、引数2=呼び出しを実行したディレクトリ
   # hook_afterdvipdf: null
   #
-  # 画像のscale=X.Xという指定を画像拡大縮小率からページ最大幅の相対倍率に変換します。
+  # 画像のscale=X.Xという指定を画像拡大縮小率からページ最大幅の相対倍率に変換する
   # image_scale2width: true
   #
+  # PDFやIllustratorファイル(.ai)の画像のBoudingBoxの抽出に指定のボックスを採用する
+  # cropbox(デフォルト), mediabox, artbox, trimbox, bleedboxから選択する。
+  # Illustrator CC以降のIllustratorファイルに対してはmediaboxを指定する必要がある
+  # bbox: mediabox
+  #
   # 奥付を作成するか。trueを指定するとデフォルトの奥付、ファイル名を指定するとそれがcolophon.htmlとしてコピーされる
   colophon: true
   # pdfmaker:階層を使うものはここまで

data/doc/format.ja.md

@@ -337,6 +337,12 @@ V1 --> V6 --|
 * `<id>` は //image[〜] の最初に入れた「〜」のことです（つまり、ID に日本語や空白交じりの文字を使ってしまうと、後で画像ファイル名の名前付けに苦労することになります！）。
 * `<ext>` は Re:VIEW が自動で判別する拡張子です。ビルダによってサポートおよび優先する拡張子は異なります。
 
+各ビルダでは、以下の拡張子から最初に発見した画像ファイルが使われます。
+
+* HTMLBuilder (EPUBMaker、WEBMaker)、MARKDOWNBuilder: .png、.jpg、.jpeg、.gif、.svg
+* LATEXBuilder (PDFMaker): .ai、.eps、.pdf、.tif、.tiff、.png、.bmp、.jpg、.jpeg、.gif
+* それ以外のビルダ: .ai、.psd、.eps、.pdf、.tif、.tiff、.png、.bmp、.jpg、.jpeg、.gif、.svg
+
 ### インラインの画像挿入
 
 段落途中などに画像を貼り込むには、インライン命令の `@<icon>{識別子}` を使います。ファイルの探索ルールは同じです。
@@ -569,6 +575,23 @@ LaTeX の数式が正常に整形されるかどうかは処理系に依存し
 
 段落の行頭字下げを制御するタグとして、`//noindent` があります。HTML では `noindent` が `class` 属性に設定されます。
 
+## 空行
+
+1行ぶんの空行を明示して入れるには、`//blankline` を使います。
+
+例:
+
+```
+この下に1行の空行が入る
+
+//blankline
+
+この下に2行の空行が入る
+
+//blankline
+//blankline
+```
+
 ## 見出し参照
 章に対する参照は、次の3つのインライン命令を利用できます。章 ID は、各章のファイル名から拡張子を除いたものです。たとえば `advanced.re` であれば `advanced` が章 ID です。
 

data/doc/format.md

@@ -380,6 +380,12 @@ The order of finding image is as follows.  The first matched one is used.
 * ``<id>`` is the ID of the first argument of `//image`.  You should use only printable ASCII characters as ID.
 * ``<ext>`` is file extensions of Re:VIEW.  They are different by the builder you use.
 
+For each builder, image files are searched in order of the following extensions, and the first hit file is adopted.
+
+* HTMLBuilder (EPUBMaker, WEBMaker), MARKDOWNBuilder: .png, .jpg, .jpeg, .gif, .svg
+* LATEXBuilder (PDFMaker): .ai, .eps, .pdf, .tif, .tiff, .png, .bmp, .jpg, .jpeg, .gif
+* Other builders: .ai, .psd, .eps, .pdf, .tif, .tiff, .png, .bmp, .jpg, .jpeg, .gif, .svg
+
 ### Inline Images
 
 When you want to use images in paragraph, you can use the inline command `@<icon>{ID}`.  The order of finding images are same as `//image`.
@@ -610,12 +616,24 @@ Usage:
 
 `//noindent` is a tag for spacing.
 
-
 * `//noindent` : ingore indentation immediately following line. (in HTML, add `noindent` class)
 
+## Blank line
+
+`//blankline` put an empty line.
+
+Usage:
+
+```
+Insert one blank line below.
 
-`//linebreak` and `//pagebreak` will be obsoleted.
+//blankline
 
+Insert two blank line below.
+
+//blankline
+//blankline
+```
 
 ## Referring headings
 

data/doc/quickstart.ja.md

@@ -113,17 +113,19 @@ review-init コマンドによらず、独自に作業フォルダを作成し
 
 テキストファイルの文字エンコーディングには、UTF-8 を使用してください。
 
-### PDF 化と EPUB 化
+### PDF 化と EPUB 化、プレインテキスト化
 
-review-pdfmaker コマンドで PDF ブックの作成、review-epubmaker コマンドで EPUB ファイルの作成ができます。
+review-pdfmaker コマンドで PDF ブックの作成、review-epubmaker コマンドで EPUB ファイルの作成、review-textmaker コマンドでプレインテキストの作成ができます。
 
 PDF を作成するには、TeXLive2012 以上の環境が必要です。EPUB を作成するには、rubyzip gem あるいは zip コマンドが必要です（MathML も使いたいときには、 [MathML ライブラリ](http://www.hinet.mydns.jp/?mathml.rb)も必要です）。
 
 いずれのコマンドも、必要な設定情報を記した YAML 形式ファイルを引数に指定して実行します。review-init コマンドで作成した環境には、デフォルトで config.yml として用意されているので、これを利用します。
 
 ```bash
-$ review-pdfmaker config.yml  ←PDFの作成
-$ review-epubmaker config.yml ←EPUBの作成
+$ review-pdfmaker config.yml     ←PDFの作成
+$ review-epubmaker config.yml    ←EPUBの作成
+$ review-textmaker config.yml    ←テキストの作成（装飾情報あり）
+$ review-textmaker -n config.yml ←テキストの作成（装飾情報なし）
 ```
 
 rake コマンドを利用できるなら、次のように実行することもできます。
@@ -131,6 +133,8 @@ rake コマンドを利用できるなら、次のように実行することも
 ```bash
 $ rake pdf  ←PDFの作成
 $ rake epub ←EPUBの作成
+$ rake text  ←テキストの作成（装飾情報あり）
+$ rake plaintext ←テキストの作成（装飾情報なし）
 ```
 
 config.yml のサンプルについては以下を参照してください。

data/doc/quickstart.md

@@ -115,28 +115,31 @@ The web site of Re:VIEW is @<tt>{https://reviewml.org/}.
 
 You should use UTF-8 as encodings in text files.
 
-### generating PDF and EPUB
+### generating PDF, EPUB, and plain-text
 
-You can generate a PDF file with `review-pdfmaker` command.  Also you can generate an EPUB file with `review-epubmaker` command.
+You can generate a PDF file with `review-pdfmaker` command.  Also you can generate an EPUB file with `review-epubmaker` command, and can generate an plain-text file with 'review-textmaker' command.
 
 To generate PDF, you should install TeXLive 2012 or later.  To generate EPUB, you should install zip command.
 When you want to use MathML, you should install [MathML library](http://www.hinet.mydns.jp/?mathml.rb)
 
-`review-pdfmaker` and `review-epubmaker` need `config.yml`, configuration YAML files.  `review-init` command generates `config.yml` in default.
+Each maker need `config.yml`, configuration YAML files.  `review-init` command generates `config.yml` in default.
 
 ```bash
-$ review-pdfmaker config.yml  ## generate PDF
-$ review-epubmaker config.yml ## generate EPUB
+$ review-pdfmaker config.yml     ## generate PDF
+$ review-epubmaker config.yml    ## generate EPUB
+$ review-textmaker config.yml    ## generate text with decoration
+$ review-textmaker -n config.yml ## generate text without decoration
 ```
 
 You also can generate them with Rake.
 
 ```bash
-$ rake pdf  ## generate PDF
-$ rake epub ## generate EPUB
+$ rake pdf       ## generate PDF
+$ rake epub      ## generate EPUB
+$ rake text      ## generate text with decoration
+$ rake plaintext ## generate text without decoration
 ```
 
-
 There is a sample YAML file [config.yml.sample](https://github.com/kmuto/review/blob/master/doc/config.yml.sample) in the same directory of this document.
 
 ### add chapters and modify them

data/lib/review/book/base.rb

@@ -32,11 +32,11 @@ module ReVIEW
 
       def self.update_rubyenv(dir)
         return if @basedir_seen.key?(dir)
-        if File.file?("#{dir}/review-ext.rb")
+        if File.file?(File.join(dir, 'review-ext.rb'))
           if ENV['REVIEW_SAFE_MODE'].to_i & 2 > 0
             ReVIEW.logger.warn 'review-ext.rb is prohibited in safe mode. ignored.'
           else
-            Kernel.load File.expand_path("#{dir}/review-ext.rb")
+            Kernel.load File.expand_path(File.join(dir, 'review-ext.rb'))
           end
         end
         @basedir_seen[dir] = true
@@ -183,8 +183,11 @@ module ReVIEW
       def catalog
         return @catalog if @catalog.present?
 
-        catalogfile_path = "#{basedir}/#{config['catalogfile']}"
-        @catalog = File.open(catalogfile_path) { |f| Catalog.new(f) } if File.file? catalogfile_path
+        catalogfile_path = filename_join(@basedir, config['catalogfile'])
+        @catalog = File.open(catalogfile_path, 'r:BOM|utf-8') { |f| Catalog.new(f) } if File.file? catalogfile_path
+        if @catalog
+          @catalog.validate!(basedir)
+        end
         @catalog
       end
 
@@ -226,7 +229,7 @@ module ReVIEW
         if catalog
           @read_part = catalog.parts
         else
-          @read_part = File.read("#{@basedir}/#{config['part_file']}")
+          @read_part = File.read(File.join(@basedir, config['part_file']))
         end
       end
 
@@ -234,23 +237,26 @@ module ReVIEW
         if catalog
           catalog.parts.present?
         else
-          File.exist?("#{@basedir}/#{config['part_file']}")
+          File.exist?(File.join(@basedir, config['part_file']))
         end
       end
 
       def read_bib
-        File.read("#{@basedir}/#{bib_file}")
+        File.read(File.join(@basedir, bib_file))
       end
 
       def bib_exist?
-        File.exist?("#{@basedir}/#{bib_file}")
+        File.exist?(File.join(@basedir, bib_file))
       end
 
       def prefaces
-        return mkpart_from_namelist(catalog.predef.split("\n")) if catalog
+        if catalog
+          return mkpart_from_namelist(catalog.predef.split("\n"))
+        end
 
         begin
-          mkpart_from_namelistfile("#{@basedir}/#{config['predef_file']}") if File.file?("#{@basedir}/#{config['predef_file']}")
+          predef_file = filename_join(@basedir, config['predef_file'])
+          mkpart_from_namelistfile(predef_file) if File.file?(predef_file)
         rescue FileNotFound => err
           raise FileNotFound, "preface #{err.message}"
         end
@@ -264,7 +270,8 @@ module ReVIEW
         end
 
         begin
-          mkpart_from_namelistfile("#{@basedir}/#{config['postdef_file']}") if File.file?("#{@basedir}/#{config['postdef_file']}")
+          postdef_file = filename_join(@basedir, config['postdef_file'])
+          mkpart_from_namelistfile(postdef_file) if File.file?(postdef_file)
         rescue FileNotFound => err
           raise FileNotFound, "postscript #{err.message}"
         end
@@ -301,12 +308,12 @@ module ReVIEW
           return catalog.parts_with_chaps.map do |entry|
             if entry.is_a?(Hash)
               chaps = entry.values.first.map do |chap|
-                chap = Chapter.new(self, num += 1, chap, "#{@basedir}/#{chap}")
+                chap = Chapter.new(self, num += 1, chap, File.join(@basedir, chap))
                 chap
               end
               Part.new(self, part += 1, chaps, read_part.split("\n")[part - 1])
             else
-              chap = Chapter.new(self, num += 1, entry, "#{@basedir}/#{entry}")
+              chap = Chapter.new(self, num += 1, entry, File.join(@basedir, entry))
               if chap.number
                 num = chap.number
               else
@@ -320,7 +327,7 @@ module ReVIEW
         chap = read_chaps.
                strip.lines.map(&:strip).join("\n").split(/\n{2,}/).
                map do |part_chunk|
-          chaps = part_chunk.split.map { |chapid| Chapter.new(self, num += 1, chapid, "#{@basedir}/#{chapid}") }
+          chaps = part_chunk.split.map { |chapid| Chapter.new(self, num += 1, chapid, File.join(@basedir, chapid)) }
           if part_exist? && read_part.split("\n").size > part
             Part.new(self, part += 1, chaps, read_part.split("\n")[part - 1])
           else
@@ -352,14 +359,14 @@ module ReVIEW
 
       def mkchap(name, number = nil)
         name += ext if File.extname(name).empty?
-        path = "#{@basedir}/#{name}"
+        path = File.join(@basedir, name)
         raise FileNotFound, "file not exist: #{path}" unless File.file?(path)
         Chapter.new(self, number, name, path)
       end
 
       def mkchap_ifexist(name, idx = nil)
         name += ext if File.extname(name).empty?
-        path = "#{@basedir}/#{name}"
+        path = File.join(@basedir, name)
         if File.file?(path)
           idx += 1 if idx
           Chapter.new(self, idx, name, path)
@@ -369,10 +376,10 @@ module ReVIEW
       def read_file(filename)
         unless @warn_old_files[filename]
           @warn_old_files[filename] = true
-          warn "!!! #{filename} is obsoleted. please use catalog.yml." if caller.none? { |item| item =~ %r{/review/test/test_} }
+          ReVIEW.logger.warn "!!! #{filename} is obsoleted. please use catalog.yml." if caller.none? { |item| item =~ %r{/review/test/test_} }
         end
         res = ''
-        File.open("#{@basedir}/#{filename}", 'r:BOM|utf-8') do |f|
+        File.open(filename_join(@basedir, filename), 'r:BOM|utf-8') do |f|
           f.each_line do |line|
             next if /\A#/ =~ line
             line.gsub!(/#.*\Z/, '')
@@ -385,6 +392,10 @@ module ReVIEW
       rescue Errno::EISDIR
         ''
       end
+
+      def filename_join(*args)
+        File.join(args.reject(&:nil?))
+      end
     end
   end
 end

data/lib/review/book/index.rb

@@ -88,9 +88,11 @@ module ReVIEW
 
       def number(id)
         chapter = @index.fetch(id)
-        chapter.format_number
-      rescue # part
-        I18n.t('part', chapter.number)
+        begin
+          chapter.format_number
+        rescue # part
+          I18n.t('part', chapter.number)
+        end
       end
 
       def title(id)
@@ -280,6 +282,7 @@ module ReVIEW
         headlines = []
         inside_column = false
         inside_block = nil
+        column_level = -1
         src.each do |line|
           if line =~ %r{\A//[a-z]+.*\{\Z}
             inside_block = true
@@ -293,19 +296,20 @@ module ReVIEW
 
           m = HEADLINE_PATTERN.match(line)
           next if m.nil? || m[1].size > 10 # Ignore too deep index
-          next if m[4].strip.empty? # no title
           index = m[1].size - 2
 
           # column
           if m[2] == 'column'
             inside_column = true
+            column_level = index
             next
           elsif m[2] == '/column'
             inside_column = false
             next
           end
-          inside_column = false if indexs.blank? || index <= indexs[-1]
+          inside_column = false if indexs.blank? || index <= column_level
           next if inside_column
+          next if m[4].strip.empty? # no title
 
           next unless index >= 0
           if indexs.size > (index + 1)
@@ -324,6 +328,7 @@ module ReVIEW
         @items = items
         @chap = chap
         @index = {}
+        @logger = ReVIEW.logger
         items.each do |i|
           @logger.warn "warning: duplicate ID: #{i.id}" if @index[i.id]
           @index[i.id] = i

data/lib/review/builder.rb

@@ -1,4 +1,4 @@
-# Copyright (c) 2002-2017 Minero Aoki, Kenshi Muto
+# Copyright (c) 2002-2018 Minero Aoki, Kenshi Muto
 #
 # This program is free software.
 # You can distribute or modify this program under the terms of
@@ -12,6 +12,7 @@ require 'review/compiler'
 require 'review/sec_counter'
 require 'stringio'
 require 'cgi'
+require 'fileutils'
 
 module ReVIEW
   class Builder
@@ -27,10 +28,13 @@ module ReVIEW
       nil
     end
 
+    attr_accessor :doc_status
+
     def initialize(strict = false, *args)
       @strict = strict
       @output = nil
       @logger = ReVIEW.logger
+      @doc_status = {}
       builder_init(*args)
     end
 
@@ -148,8 +152,12 @@ module ReVIEW
       return if rows.empty?
       table_begin rows.first.size
       if sepidx
-        sepidx.times { tr(rows.shift.map { |s| th(s) }) }
-        rows.each { |cols| tr(cols.map { |s| td(s) }) }
+        sepidx.times do
+          tr(rows.shift.map { |s| th(s) })
+        end
+        rows.each do |cols|
+          tr(cols.map { |s| td(s) })
+        end
       else
         rows.each do |cols|
           h, *cs = *cols
@@ -160,9 +168,15 @@ module ReVIEW
     end
 
     def adjust_n_cols(rows)
-      rows.each { |cols| cols.pop while cols.last and cols.last.strip.empty? }
+      rows.each do |cols|
+        while cols.last and cols.last.strip.empty?
+          cols.pop
+        end
+      end
       n_maxcols = rows.map(&:size).max
-      rows.each { |cols| cols.concat [''] * (n_maxcols - cols.size) }
+      rows.each do |cols|
+        cols.concat [''] * (n_maxcols - cols.size)
+      end
       rows
     end
     private :adjust_n_cols
@@ -183,6 +197,10 @@ module ReVIEW
     #   footnote_end
     # end
 
+    def blankline
+      puts ''
+    end
+
     def compile_inline(s)
       @compiler.text(s)
     end
@@ -191,35 +209,30 @@ module ReVIEW
       compile_inline @book.chapter_index.display_string(id)
     rescue KeyError
       error "unknown chapter: #{id}"
-      nofunc_text("[UnknownChapter:#{id}]")
     end
 
     def inline_chap(id)
       @book.chapter_index.number(id)
     rescue KeyError
       error "unknown chapter: #{id}"
-      nofunc_text("[UnknownChapter:#{id}]")
     end
 
     def inline_title(id)
       compile_inline @book.chapter_index.title(id)
     rescue KeyError
       error "unknown chapter: #{id}"
-      nofunc_text("[UnknownChapter:#{id}]")
     end
 
     def inline_list(id)
       "#{I18n.t('list')}#{@chapter.list(id).number}"
     rescue KeyError
       error "unknown list: #{id}"
-      nofunc_text("[UnknownList:#{id}]")
     end
 
     def inline_img(id)
       "#{I18n.t('image')}#{@chapter.image(id).number}"
     rescue KeyError
       error "unknown image: #{id}"
-      nofunc_text("[UnknownImage:#{id}]")
     end
 
     def inline_imgref(id)
@@ -236,14 +249,12 @@ module ReVIEW
       "#{I18n.t('table')}#{@chapter.table(id).number}"
     rescue KeyError
       error "unknown table: #{id}"
-      nofunc_text("[UnknownTable:#{id}]")
     end
 
     def inline_fn(id)
       @chapter.footnote(id).content
     rescue KeyError
       error "unknown footnote: #{id}"
-      nofunc_text("[UnknownFootnote:#{id}]")
     end
 
     def inline_bou(str)
@@ -252,8 +263,12 @@ module ReVIEW
 
     def inline_ruby(arg)
       base, *ruby = *arg.scan(/(?:(?:(?:\\\\)*\\,)|[^,\\]+)+/)
-      base = base.gsub(/\\,/, ',') if base
-      ruby = ruby.join(',').gsub(/\\,/, ',') if ruby
+      if base
+        base = base.gsub(/\\,/, ',')
+      end
+      if ruby
+        ruby = ruby.join(',').gsub(/\\,/, ',')
+      end
       compile_ruby(base, ruby)
     end
 
@@ -265,7 +280,9 @@ module ReVIEW
     def inline_href(arg)
       url, label = *arg.scan(/(?:(?:(?:\\\\)*\\,)|[^,\\]+)+/).map(&:lstrip)
       url = url.gsub(/\\,/, ',').strip
-      label = label.gsub(/\\,/, ',').strip if label
+      if label
+        label = label.gsub(/\\,/, ',').strip
+      end
       compile_href(url, label)
     end
 
@@ -284,20 +301,23 @@ module ReVIEW
 
     def inline_hd(id)
       m = /\A([^|]+)\|(.+)/.match(id)
-      chapter = @book.contents.detect { |chap| chap.id == m[1] } if m && m[1]
+      if m && m[1]
+        chapter = @book.contents.detect { |chap| chap.id == m[1] }
+      end
       if chapter
         inline_hd_chap(chapter, m[2])
       else
         inline_hd_chap(@chapter, id)
       end
     rescue KeyError
-      error "unknown hd: #{id}"
-      nofunc_text("[UnknownHeader:#{id}]")
+      error "unknown headline: #{id}"
     end
 
     def inline_column(id)
       m = /\A([^|]+)\|(.+)/.match(id)
-      chapter = @book.chapters.detect { |chap| chap.id == m[1] } if m && m[1]
+      if m && m[1]
+        chapter = @book.chapters.detect { |chap| chap.id == m[1] }
+      end
       if chapter
         inline_column_chap(chapter, m[2])
       else
@@ -305,7 +325,6 @@ module ReVIEW
       end
     rescue KeyError
       error "unknown column: #{id}"
-      nofunc_text("[UnknownColumn:#{id}]")
     end
 
     def inline_column_chap(chapter, id)
@@ -324,7 +343,9 @@ module ReVIEW
       if matched = str.match(/\|(.*?)\|(.*)/)
         builders = matched[1].split(',').map { |i| i.gsub(/\s/, '') }
         c = target_name
-        print matched[2].gsub('\\n', "\n") if builders.include?(c)
+        if builders.include?(c)
+          print matched[2].gsub('\\n', "\n")
+        end
       else
         print str.gsub('\\n', "\n")
       end
@@ -345,8 +366,11 @@ module ReVIEW
     end
 
     def error(msg)
-      raise ApplicationError, msg if msg =~ /:\d+: error: /
-      raise ApplicationError, "#{@location}: error: #{msg}"
+      if msg =~ /:\d+: error: /
+        raise ApplicationError, msg
+      else
+        raise ApplicationError, "#{@location}: error: #{msg}"
+      end
     end
 
     def handle_metric(str)
@@ -374,15 +398,20 @@ module ReVIEW
 
     def get_chap(chapter = @chapter)
       if @book.config['secnolevel'] > 0 && !chapter.number.nil? && !chapter.number.to_s.empty?
-        return I18n.t('part_short', chapter.number) if chapter.is_a?(ReVIEW::Book::Part)
-        return chapter.format_number(nil)
+        if chapter.is_a?(ReVIEW::Book::Part)
+          return I18n.t('part_short', chapter.number)
+        else
+          return chapter.format_number(nil)
+        end
       end
       nil
     end
 
     def extract_chapter_id(chap_ref)
       m = /\A([\w+-]+)\|(.+)/.match(chap_ref)
-      return [@book.contents.detect { |chap| chap.id == m[1] }, m[2]] if m
+      if m
+        return [@book.contents.detect { |chap| chap.id == m[1] }, m[2]]
+      end
       [@chapter, chap_ref]
     end
 
@@ -401,7 +430,7 @@ module ReVIEW
     def graph(lines, id, command, caption = nil)
       c = target_name
       dir = File.join(@book.basedir, @book.image_dir, c)
-      Dir.mkdir(dir) unless File.exist?(dir)
+      FileUtils.mkdir_p(dir)
       file = "#{id}.#{image_ext}"
       file_path = File.join(dir, file)
 
@@ -410,7 +439,7 @@ module ReVIEW
         graphviz: "echo '#{line}' | dot -T#{image_ext} -o#{file_path}",
         gnuplot: %Q(echo 'set terminal ) +
         "#{image_ext == 'eps' ? 'postscript eps' : image_ext}\n" +
-        %Q(" set output "#{file_path}"\n#{line}' | gnuplot),
+        %Q( set output "#{file_path}"\n#{line}' | gnuplot),
         blockdiag: "echo '#{line}' " +
         "| blockdiag -a -T #{image_ext} -o #{file_path} /dev/stdin",
         aafigure: "echo '#{line}' | aafigure -t#{image_ext} -o#{file_path}"
@@ -428,11 +457,7 @@ module ReVIEW
     end
 
     def inline_include(file_name)
-      compile_inline File.read(file_name)
-    end
-
-    def include(file_name)
-      File.foreach(file_name) { |line| paragraph([line]) }
+      compile_inline File.read(file_name, mode: 'rt:BOM|utf-8').chomp
     end
 
     def ul_item_begin(lines)