review 5.1.1 → 5.4.0

data/NEWS.ja.md

@@ -1,3 +1,152 @@
+# Version 5.4.0
+## 新機能
+* Re:VIEW に関する質問の受け付けに対応する [GitHub Discussions](https://github.com/kmuto/review/discussions) を開始しました
+
+## 非互換の変更
+* EPUBMaker: opf ファイルの `manifest` 内の `item` を ID 文字列の辞書順でソートするようにしました ([#1763])
+* TextMaker: 表の見出しセル行を太字表現（★〜☆）にするのではなく、見出しセル行と通常セル行の区切り線を入れるようにしました。従来の太字表現に戻すには `textmaker` セクションの `th_bold` パラメータを true に設定してください ([#1789])
+* TextMaker: `//indepimage` 命令の出力結果を `//image` に合わせました。開始・終了マークが入り、画像ファイルが見つからないときにはコメント内容を出力するようになります ([#1790])
+* TextMaker: `//imgtable` 命令の出力結果を `//image` および `//table` に合わせました。開始・終了マークが入り、画像ファイルが見つからないときにはコメント内容を出力するようになります ([#1791])
+* ハイライト有効時に、`//source` 命令もハイライト対象として中身をエスケープしないようにしました ([#1788])
+
+## バグ修正
+* Ruby 3.1 で YAML のエラーが発生するのを修正し、互換性も持たせました ([#1767], [#1775])
+* EPUBMaker: `epub:type=cover` が大扉や奥付に入るのを修正しました ([#1776])
+* 無効な urnid の例がサンプルとして示されているのを削除しました ([#1779])
+* config.yml の YAML 構文にエラーがあったときに例外ではなく妥当なエラーを返すようにしました ([#1797])
+* IDGXMLMaker: secttags を有効にしている状態で前付や後付がエラーになるのを修正しました ([#1800])
+
+## 機能強化
+* EPUBMaker, WebMaker: 表紙・大扉・奥付・部のベーステンプレートに通常の章と同じく `layout.html.erb` または `layout-web.html.erb` を使うようにしました ([#1780])
+* EPUBMaker, WebMaker: 表紙・大扉・奥付・部のカスタムテンプレートとして、`layouts` フォルダの `_cover.html.erb`、`_titlepage.html.erb`、`_colophon.html.erb`、`_colophon_history.html.erb`、`_part_body.html.erb` で上書きできるようにしました ([#1777])
+
+## ドキュメント
+* GitHub Discussions について README.md に記載しました ([#1772])
+
+## その他
+* RuboCop 1.25.1 の指摘を反映しました ([#1773], [#1782], [#1783], [#1784], [#1792])
+
+[#1763]: https://github.com/kmuto/review/pull/1763
+[#1767]: https://github.com/kmuto/review/pull/1767
+[#1772]: https://github.com/kmuto/review/pull/1772
+[#1773]: https://github.com/kmuto/review/pull/1773
+[#1775]: https://github.com/kmuto/review/pull/1775
+[#1776]: https://github.com/kmuto/review/pull/1776
+[#1777]: https://github.com/kmuto/review/issues/1777
+[#1779]: https://github.com/kmuto/review/pull/1779
+[#1780]: https://github.com/kmuto/review/pull/1780
+[#1782]: https://github.com/kmuto/review/pull/1782
+[#1783]: https://github.com/kmuto/review/pull/1783
+[#1784]: https://github.com/kmuto/review/pull/1784
+[#1788]: https://github.com/kmuto/review/pull/1788
+[#1789]: https://github.com/kmuto/review/issues/1789
+[#1790]: https://github.com/kmuto/review/issues/1790
+[#1791]: https://github.com/kmuto/review/issues/1791
+[#1792]: https://github.com/kmuto/review/pull/1792
+[#1797]: https://github.com/kmuto/review/issues/1797
+[#1800]: https://github.com/kmuto/review/pull/1800
+
+# Version 5.3.0
+## 新機能
+* 後注のサポートを追加しました。`//endnote` 命令で後注内容、`@<endnote>` 命令で後注参照、`//printendnotes` 命令で後注を配置する場所を指定します ([#1724])
+
+## バグ修正
+* 新しい jlreq において ifthen パッケージに非依存になったことによるエラーを修正しました ([#1718])
+* review-jsbook と新しい TeXLive 2020 以降の組み合わせにおいて、隠しノンブルがすべて1になってしまう問題を修正しました ([#1720])
+* coverimage パラメータに実際に存在しないファイルを指定したときに例外エラーが発生するのを修正しました ([#1726], [#1729])
+* titlefile・creditfile・profile パラメータに存在しないファイルを指定したときに警告を表示するようにしました ([#1730])
+* review-jlreq で `@<tcy>` 命令がエラーになるのを修正しました。縦中横の TeX 表現側での抽象名に `\reviewtcy` マクロを使うようにしました ([#1733])
+* review-vol および review-index の例外エラーを修正しました ([#1740])
+* 数式ビルドエラーが出たときに `__IMGMATH_BODY__.tex` のコピーを忘れているのを修正しました ([#1747])
+* `//beginchild`・`//endchild` 命令でエラーが起きたときにエラー位置の表示がされていなかったのを修正しました ([#1742])
+* `//graph` 命令を使うとビルドエラーになるのを修正しました ([#1744])
+* epubmaker.rb で未定義の変数を参照している箇所を修正しました ([#1755])
+* review-catalog-converter がエラーになるのを修正しました ([#1753])
+
+## 機能強化
+* 脚注 (`//footnote`)・後注(`//endnote`) を定義したけれども参照（`@<fn>`、`@<endnote>`）していないときに警告するようにしました ([#1725])
+* 紙面全体に画像を貼り込む `\includefullpagegraphics` マクロを縦書きにも対応するようにしました ([#1734])
+* plantuml の jar ファイル探索先を作業フォルダのほか、`/usr/share/plantuml`、`/usr/share/java` からも探すようにしました ([#1760])
+
+## ドキュメント
+* format.ja.md, format.md: SVG数式を作成するコマンドラインの間違いを修正しました ([#1748])
+
+## その他
+* Windows 版 Ruby 2.7 のテストを追加しました ([#1743])
+* Rubocop 1.22.1 に対応しました ([#1759])
+
+## コントリビューターのみなさん
+* [@munepi](https://github.com/munepi)
+* [@huideyeren](https://github.com/huideyeren)
+
+[#1718]: https://github.com/kmuto/review/issues/1718
+[#1720]: https://github.com/kmuto/review/issues/1720
+[#1724]: https://github.com/kmuto/review/issues/1724
+[#1725]: https://github.com/kmuto/review/issues/1725
+[#1726]: https://github.com/kmuto/review/issues/1726
+[#1729]: https://github.com/kmuto/review/pull/1729
+[#1730]: https://github.com/kmuto/review/pull/1730
+[#1733]: https://github.com/kmuto/review/issues/1733
+[#1734]: https://github.com/kmuto/review/issues/1734
+[#1740]: https://github.com/kmuto/review/pull/1740
+[#1742]: https://github.com/kmuto/review/pull/1742
+[#1743]: https://github.com/kmuto/review/pull/1743
+[#1744]: https://github.com/kmuto/review/issues/1744
+[#1747]: https://github.com/kmuto/review/pull/1747
+[#1748]: https://github.com/kmuto/review/pull/1748
+[#1753]: https://github.com/kmuto/review/issues/1753
+[#1755]: https://github.com/kmuto/review/issues/1755
+[#1759]: https://github.com/kmuto/review/pull/1759
+[#1760]: https://github.com/kmuto/review/pull/1760
+
+# Version 5.2.0
+## 新機能
+* EPUBMaker: CSS 組版向けに、見出しの存在に応じて `<section>` で階層化する機能を追加しました。config.yml で `epubmaker` セクションの `use_section` パラメータを `true` にすると有効化されます ([#1685])
+
+## バグ修正
+* PDFMaker: Ruby 2.6 以上でテンプレートの引数についての警告が出る問題を修正しました ([#1683])
+* EPUBMaker: Docker 環境においてファイルがコピーされず空になる問題を修正しました ([#1686])
+* 縦中横を正しく表示する CSS 設定を追加しました ([#1688])
+* PDFMaker: 新しい TeXLive との組み合わせで pxjahyper のオプション競合エラーが発生するのを修正しました ([#1690])
+* PDFMaker: 画像が見つからないときにコンパイルエラーになるのを修正しました ([#1706])
+
+## 機能強化
+* 警告とエラーを出力する際の処理を改善しました。`error!` (すぐに終了) および `app_error` (`ApplicationError` 例外を上げる) メソッドを導入しました ([#1674])
+* PDFMaker: ビルドのために画像ファイルをコピーする際、実コピーではなくシンボリックリンクを利用して処理を高速化するオプションを追加しました。`pdfmaker` セクションの `use_symlink` パラメータを `true` にすると、デフォルト挙動の実コピーの代わりにシンボリックリンクが使われます。Windows など一部の OS ではこれは動作しない可能性があります ([#1696])
+* PDFMaker: review-jlreq で `serial_pagination=true, openany` を指定したときには前付の後の空ページが入らないようにしました ([#1711])
+
+## その他
+* GitHub Actions まわりを修正しました ([#1684], [#1691])
+* review-preproc: リファクタリングを行いました ([#1697])
+* 入れ子の箇条書きの処理をリファクタリングしました ([#1698])
+* Rubocop 1.12 に対応しました ([#1689], [#1692], [#1699], [#1700])
+* 各ビルダの `builder_init_file` メソッドで最初に `super` で基底 builder の `builder_init_file` を実行するようにしました ([#1702])
+* PDFMaker: FileUtils ライブラリを内部で使う際に、明示記法の `FIleUtils.foobar` を使うようにしました ([#1704])
+
+## コントリビューターのみなさん
+* [@odaki](https://github.com/odaki)
+* [@imamurayusuke](https://github.com/imamurayusuke)
+
+[#1674]: https://github.com/kmuto/review/issues/1674
+[#1683]: https://github.com/kmuto/review/pulls/1683
+[#1684]: https://github.com/kmuto/review/pulls/1684
+[#1685]: https://github.com/kmuto/review/pulls/1685
+[#1686]: https://github.com/kmuto/review/issues/1686
+[#1688]: https://github.com/kmuto/review/pulls/1688
+[#1689]: https://github.com/kmuto/review/pulls/1689
+[#1690]: https://github.com/kmuto/review/pulls/1690
+[#1691]: https://github.com/kmuto/review/pulls/1691
+[#1692]: https://github.com/kmuto/review/pulls/1692
+[#1696]: https://github.com/kmuto/review/issues/1696
+[#1697]: https://github.com/kmuto/review/pulls/1697
+[#1698]: https://github.com/kmuto/review/pulls/1698
+[#1699]: https://github.com/kmuto/review/pulls/1699
+[#1700]: https://github.com/kmuto/review/pulls/1700
+[#1702]: https://github.com/kmuto/review/pulls/1702
+[#1704]: https://github.com/kmuto/review/pulls/1704
+[#1706]: https://github.com/kmuto/review/issues/1706
+[#1711]: https://github.com/kmuto/review/issues/1711
+
 # Version 5.1.1
 ## バグ修正
 * `review-preproc` がエラーになるのを修正しました ([#1679])

data/NEWS.md

@@ -1,3 +1,152 @@
+# Version 5.4.0
+## New Features
+* [GitHub Discussions](https://github.com/kmuto/review/discussions) has been opened to answer questions about Re:VIEW
+
+## Breaking Changes
+* EPUBMaker: `manifest/item` in the opf file is now sorted by the dictional order of ID strings ([#1763])
+* TextMaker: the separator line now put between the heading rows and the normal rows of the table. To revert this behavior to old version, set `textmaker/th_bold` parameter to true ([#1789])
+* TextMaker: the output result of the `//indepimage` op has been adjusted to `//image` ([#1790])
+* TextMaker: the output result of the `//imgtable` op has been adjusted to `//image` and `//table` ([#1791])
+* `//source` op is now also not escaped when highlighting is enabled ([#1788])
+
+## Bug Fixes
+* fixed YAML error in Ruby 3.1 and kept backward compatibility ([#1767], [#1775])
+* EPUBMaker: removed `epub:type=cover` from titlepage and colophon ([#1776])
+* removed invalid urnid sample ([#1779])
+* when there is a syntax error in config.yml, exit with proper error message instead of an exception ([#1797])
+* IDGXMLMaker: fixed an error to compile prefaces or postfaces when secttags is enabled ([#1800])
+
+## Enhancements
+* EPUBMaker, WebMaker: use `layout.html.erb` or `layout-web.html.erb` as the base template for cover, titlepage, colophon, and part, just like regular chapters ([#1780])
+* EPUBMaker, WebMaker: cover, titlepage, colophon, and part can now be overwritten with `_cover.html.erb`, `_titlepage.html.erb`, `_colophon.html.erb`, `_colophon_history.html.erb`, and `_part_body.html.erb` ([#1777])
+
+## Docs
+* mentioned GitHub Discussions in README.md ([#1772])
+
+## Others
+* refactor code with RuboCop 1.25.1 ([#1773], [#1782], [#1783], [#1784], [#1792])
+
+[#1763]: https://github.com/kmuto/review/pull/1763
+[#1767]: https://github.com/kmuto/review/pull/1767
+[#1772]: https://github.com/kmuto/review/pull/1772
+[#1773]: https://github.com/kmuto/review/pull/1773
+[#1775]: https://github.com/kmuto/review/pull/1775
+[#1776]: https://github.com/kmuto/review/pull/1776
+[#1777]: https://github.com/kmuto/review/issues/1777
+[#1779]: https://github.com/kmuto/review/pull/1779
+[#1780]: https://github.com/kmuto/review/pull/1780
+[#1782]: https://github.com/kmuto/review/pull/1782
+[#1783]: https://github.com/kmuto/review/pull/1783
+[#1784]: https://github.com/kmuto/review/pull/1784
+[#1788]: https://github.com/kmuto/review/pull/1788
+[#1789]: https://github.com/kmuto/review/issues/1789
+[#1790]: https://github.com/kmuto/review/issues/1790
+[#1791]: https://github.com/kmuto/review/issues/1791
+[#1792]: https://github.com/kmuto/review/pull/1792
+[#1797]: https://github.com/kmuto/review/issues/1797
+[#1800]: https://github.com/kmuto/review/pull/1800
+
+# Version 5.3.0
+## New Features
+* add the future of endnote. `//endnote` specifies the content of the endnote, `@<endnote>` specifies the reference to the endnote, and `//printendnotes` places endnotes ([#1724])
+
+## Bug Fixes
+* fixed an error in new jlreq that caused it to become independent of ifthen package ([#1718])
+* fixed an issue with hidden folio being set to all 1 when using review-jsbook with TeXLive 2020 or later ([#1720])
+* fixed an error that occurred when a non-existent file was specified in the coverimage parameter ([#1726], [#1729])
+* it now warns when a non-existent file is specified in the titlefile, creditfile, and profile parameters ([#1730])
+* fixed `@<tcy>` op error in review-jlreq. this op will be expanded into `\reviewtcy` macro ([#1733])
+* fixed exception errors in review-vol and review-index ([#1740])
+* fixed forgetting to copy `__IMGMATH_BODY__.tex` when math compiling error occurs ([#1747])
+* fixed the problem that the position of `//beginchild` and `//endchild` is not displayed when an error occurs ([#1742])
+* fixed a build error when using `//graph` op ([#1744])
+* fixed undefined variable in epubmaker.rb ([#1755])
+* fixed execution error in review-catalog-converter ([#1753])
+
+## Enhancements
+* warnings are now given when footnotes (`//footnote`) and endnotes (`//endnote`) are defined but not referenced (`@<fn>`, `@<endnote>`) ([#1725])
+* `\includefullpagegraphics` macro that pastes an image over the entire page now supports vertical writing ([#1734])
+* try to find plantuml.jar from the working folder, `/usr/share/plantuml`, or `/usr/share/java` ([#1760])
+
+## Docs
+* format.ja.md, format.md: fixed a mistake in the command line for creating SVG formulas ([#1748])
+
+## Others
+* added tests for Ruby 2.7 for Windows ([#1743])
+* refactor code with Rubocop 1.22.1 ([#1759])
+
+## Contributors
+* [@munepi](https://github.com/munepi)
+* [@huideyeren](https://github.com/huideyeren)
+
+[#1718]: https://github.com/kmuto/review/issues/1718
+[#1720]: https://github.com/kmuto/review/issues/1720
+[#1724]: https://github.com/kmuto/review/issues/1724
+[#1725]: https://github.com/kmuto/review/issues/1725
+[#1726]: https://github.com/kmuto/review/issues/1726
+[#1729]: https://github.com/kmuto/review/pull/1729
+[#1730]: https://github.com/kmuto/review/pull/1730
+[#1733]: https://github.com/kmuto/review/issues/1733
+[#1734]: https://github.com/kmuto/review/issues/1734
+[#1740]: https://github.com/kmuto/review/pull/1740
+[#1742]: https://github.com/kmuto/review/pull/1742
+[#1743]: https://github.com/kmuto/review/pull/1743
+[#1744]: https://github.com/kmuto/review/issues/1744
+[#1747]: https://github.com/kmuto/review/pull/1747
+[#1748]: https://github.com/kmuto/review/pull/1748
+[#1753]: https://github.com/kmuto/review/issues/1753
+[#1755]: https://github.com/kmuto/review/issues/1755
+[#1759]: https://github.com/kmuto/review/pull/1759
+[#1760]: https://github.com/kmuto/review/pull/1760
+
+# Version 5.2.0
+## New Features
+* EPUBMaker: added `<section>` based on heading level for CSS formatting, when the `epubmaker/use_section` parameter is set to `true` ([#1685])
+
+## Bug Fixes
+* PDFMaker: fixed a problem that caused a warning in templates for Ruby 2.6 and above ([#1683])
+* EPUBMaker: fixed an issue that caused copied files to be empty in Docker environments ([#1686])
+* added CSS style for correct displaying horizontal characters in vertical typesetting ([#1688])
+* PDFMaker: fixed the pxjahyper option conflict error on latest TeXLive ([#1690])
+* PDFMaker: fixed compile error when image is not found ([#1706])
+
+## Enhancements
+* improve around warn and error handling ([#1674])
+* PDFMaker: introduced `pdfmaker/use_symlink` parameter to speed up the process by using symbolic links instead of actual copies. This may not work on some operating systems such as Windows ([#1696])
+* PDFMaker: don't insert empty page after frontmatter when using review-jlreq with `serial_pagination=true, openany` ([#1711])
+
+## Others
+* fixed related to GitHub Actions ([#1684], [#1691])
+* review-preproc: refactored ([#1697])
+* refactored nested lists handling ([#1698])
+* refactor code with Rubocop 1.12 ([#1689], [#1692], [#1699], [#1700])
+* The `builder_init_file` method of each builder now executes `super` first to use base builder's `builder_init_file` ([#1702])
+* PDFMaker: Stopped implicitly including FileUtils library ([#1704])
+
+## Contributors
+* [@odaki](https://github.com/odaki)
+* [@imamurayusuke](https://github.com/imamurayusuke)
+
+[#1674]: https://github.com/kmuto/review/issues/1674
+[#1683]: https://github.com/kmuto/review/pulls/1683
+[#1684]: https://github.com/kmuto/review/pulls/1684
+[#1685]: https://github.com/kmuto/review/pulls/1685
+[#1686]: https://github.com/kmuto/review/issues/1686
+[#1688]: https://github.com/kmuto/review/pulls/1688
+[#1689]: https://github.com/kmuto/review/pulls/1689
+[#1690]: https://github.com/kmuto/review/pulls/1690
+[#1691]: https://github.com/kmuto/review/pulls/1691
+[#1692]: https://github.com/kmuto/review/pulls/1692
+[#1696]: https://github.com/kmuto/review/issues/1696
+[#1697]: https://github.com/kmuto/review/pulls/1697
+[#1698]: https://github.com/kmuto/review/pulls/1698
+[#1699]: https://github.com/kmuto/review/pulls/1699
+[#1700]: https://github.com/kmuto/review/pulls/1700
+[#1702]: https://github.com/kmuto/review/pulls/1702
+[#1704]: https://github.com/kmuto/review/pulls/1704
+[#1706]: https://github.com/kmuto/review/issues/1706
+[#1711]: https://github.com/kmuto/review/issues/1711
+
 # Version 5.1.1
 ## Bug Fixes
 * Fix the runtime error of `review-preproc` ([#1679])
@@ -1808,4 +1957,3 @@ To support language parameter for syntax highlighting, if you use review-ext.rb
 * add headline level 5 and 6 (paragraph, subparagraph)
 * escape all dash
 * add ``begin{alltt}..\end{alltt}`` into ``\reviewemlist``, ``\reviewlist`` and ``\reviewcmd``
-

data/README.md

@@ -74,13 +74,14 @@ For further information, see [doc/quickstart.md](https://github.com/kmuto/review
 
 ## Resources
 
-|         | URL                                    |
-|---------|----------------------------------------|
-| Home    | http://reviewml.org                    |
-| Project | https://github.com/kmuto/review/       |
-| Gems    | https://rubygems.org/gems/review       |
-| Wiki    | https://github.com/kmuto/review/wiki   |
-| Bugs    | https://github.com/kmuto/review/issues |
+|         | URL                                         |
+|---------|---------------------------------------------|
+| Home    | http://reviewml.org                         |
+| Project | https://github.com/kmuto/review/            |
+| Gems    | https://rubygems.org/gems/review            |
+| Q&A     | https://github.com/kmuto/review/discussions |
+| Wiki    | https://github.com/kmuto/review/wiki        |
+| Bugs    | https://github.com/kmuto/review/issues      |
 
 ### Documents
 
@@ -116,4 +117,4 @@ Exception:
 
 ## Copyright
 
-Copyright (c) 2006-2021 Minero Aoki, Kenshi Muto, Masayoshi Takahashi, Masanori Kado.
+Copyright (c) 2006-2022 Minero Aoki, Kenshi Muto, Masayoshi Takahashi, Masanori Kado.

data/bin/review

@@ -58,7 +58,7 @@ unless File.exist?(command_path)
   usage
 end
 
-if RUBY_PLATFORM =~ /mswin|bccwin|mingw/
+if RUBY_PLATFORM.match?(/mswin|bccwin|mingw/)
   cmd = File.join(RbConfig::CONFIG['bindir'],
                   RbConfig::CONFIG['ruby_install_name'])
   cmd << RbConfig::CONFIG['EXEEXT']

data/bin/review-catalog-converter

@@ -55,21 +55,21 @@ def main
       end
       # chaps and parts
       if File.exist?("#{dir}/CHAPS")
-        if File.exist?("#{dir}/PART")
-          catalog << parse_parts(File.read("#{dir}/PART"),
+        catalog << if File.exist?("#{dir}/PART")
+                     parse_parts(File.read("#{dir}/PART"),
                                  File.read("#{dir}/CHAPS"))
-        else
-          catalog << parse_chaps(File.read("#{dir}/CHAPS"))
-        end
+                   else
+                     parse_chaps(File.read("#{dir}/CHAPS"))
+                   end
       end
       # postdef
       if File.exist?("#{dir}/POSTDEF")
         postdef = File.read("#{dir}/POSTDEF")
-        if ask_yes?('Do you want to convert POSTDEF into APPENDIX? [y/n]')
-          catalog << parse_postdef(postdef, true)
-        else
-          catalog << parse_postdef(postdef)
-        end
+        catalog << if ask_yes?('Do you want to convert POSTDEF into APPENDIX? [y/n]')
+                     parse_postdef(postdef, true)
+                   else
+                     parse_postdef(postdef)
+                   end
       end
     end
   end
@@ -99,11 +99,11 @@ def parse_chaps(str)
 end
 
 def parse_postdef(str, to_appendix = false)
-  if to_appendix
-    header = "APPENDIX:\n"
-  else
-    header = "POSTDEF:\n"
-  end
+  header = if to_appendix
+             "APPENDIX:\n"
+           else
+             "POSTDEF:\n"
+           end
   parse_internal(str, header) + "\n"
 end
 

data/bin/review-check

@@ -18,7 +18,7 @@ include ReVIEW::TextUtils
 
 def sigmain
   Signal.trap(:INT) { exit 1 }
-  if RUBY_PLATFORM !~ /mswin(?!ce)|mingw|cygwin|bccwin/
+  unless RUBY_PLATFORM.match?(/mswin(?!ce)|mingw|cygwin|bccwin/)
     Signal.trap(:PIPE, 'IGNORE')
   end
   main
@@ -95,7 +95,7 @@ end
 def find_line(lines, re)
   # single line?
   lines.each_with_index do |line, idx|
-    if re =~ line
+    if re&.match?(line)
       return line.gsub(re, '<<<\&>>>'), idx
     end
   end
@@ -104,7 +104,7 @@ def find_line(lines, re)
   i = 0
   while i < lines.size - 1
     str = lines[i] + lines[i + 1]
-    return str.gsub(re, '<<<\&>>>'), i if re =~ str
+    return str.gsub(re, '<<<\&>>>'), i if re&.match?(str)
 
     i += 1
   end
@@ -118,7 +118,7 @@ def words_re(rc)
   File.foreach(rc) do |line|
     next if line[0, 1] == '#'
 
-    if / !/ =~ line
+    if / !/.match?(line)
       line, n = *line.split('!', 2)
       nega.push(n.strip)
     end
@@ -141,7 +141,7 @@ def each_paragraph(f)
       yield [$1], f.filename, f.lineno
     when %r<\A//\w.*\{\s*\z>
       while line = f.gets
-        break if %r{//\}} === line
+        break if %r{//\}}.match?(line)
       end
     when /\A=/
       yield [line.slice(/\A=+(?:\[.*?\])?\s+(.*)/, 1).strip], f.lineno
@@ -150,8 +150,8 @@ def each_paragraph(f)
       lineno = f.lineno
       while line = f.gets
         break if line.strip.empty?
-        break if %r{\A(?:=|//[\w\}])} =~ line
-        next if /\A\#@/ =~ line
+        break if %r{\A(?:=|//[\w\}])}.match?(line)
+        next if /\A\#@/.match?(line)
 
         buf.push(line.strip)
       end

data/bin/review-compile

@@ -16,11 +16,13 @@ require 'fileutils'
 require 'optparse'
 require 'yaml'
 
+include ReVIEW::Loggable
+
 DEFAULT_CONFIG_FILENAME = 'config.yml'.freeze
 
 def main
   Signal.trap(:INT) { exit 1 }
-  if RUBY_PLATFORM !~ /mswin(?!ce)|mingw|cygwin|bccwin/
+  unless RUBY_PLATFORM.match?(/mswin(?!ce)|mingw|cygwin|bccwin/)
     Signal.trap(:PIPE, 'IGNORE')
   end
   _main
@@ -32,11 +34,9 @@ def _main
   @logger = ReVIEW.logger
   @mode = :files
   @basedir = nil
-  if File.basename($PROGRAM_NAME).start_with?('review2')
-    @target = File.basename($PROGRAM_NAME, '.rb').sub('review2', '')
-  else
-    @target = nil
-  end
+  @target = if File.basename($PROGRAM_NAME).start_with?('review2')
+              File.basename($PROGRAM_NAME, '.rb').sub('review2', '')
+            end
   @check_only = false
   @output_filename = nil
 
@@ -47,17 +47,17 @@ def _main
   begin
     loader = ReVIEW::YAMLLoader.new
     if @config['yaml']
-      error "#{@config['yaml']} not found." unless File.exist?(@config['yaml'])
+      error! "#{@config['yaml']} not found." unless File.exist?(@config['yaml'])
       begin
         @config.deep_merge!(loader.load_file(@config['yaml']))
-      rescue => e
-        error "yaml error #{e.message}"
+      rescue StandardError => e
+        error! "yaml error #{e.message}"
       end
     elsif File.exist?(DEFAULT_CONFIG_FILENAME)
       begin
         @config.deep_merge!(loader.load_file(DEFAULT_CONFIG_FILENAME))
-      rescue => e
-        error "yaml error #{e.message}"
+      rescue StandardError => e
+        error! "yaml error #{e.message}"
       end
     end
 
@@ -73,12 +73,12 @@ def _main
 
     case @mode
     when :files
-      error('no input') if ARGV.empty?
+      error!('no input') if ARGV.empty?
 
       @basedir = File.dirname(ARGV[0])
       book = ReVIEW::Book::Base.new(@basedir, config: @config)
       ARGV.each do |item|
-        error("file not found: #{item}") unless File.exist?(File.join(book.config['contentdir'], item))
+        error!("file not found: #{item}") unless File.exist?(File.join(book.config['contentdir'], item))
         chap_name = File.basename(item, '.*')
         chap = book.chapter(chap_name)
         compiler = ReVIEW::Compiler.new(load_builder_class(@target, @check_only))
@@ -105,7 +105,7 @@ def _main
       raise "must not happen: #{@mode}"
     end
   rescue ReVIEW::ApplicationError => e
-    error(e.message)
+    error! e.message
   end
 end
 
@@ -164,15 +164,6 @@ def parse_opts
   end
 end
 
-def error(msg)
-  @logger.error msg
-  exit 1
-end
-
-def warn(msg)
-  @logger.warn msg
-end
-
 def load_builder_class(target, strict)
   require "review/#{target}builder"
   ReVIEW.const_get("#{target.upcase}Builder").new(strict)

data/bin/review-index

@@ -15,7 +15,7 @@ require 'review/tocprinter'
 
 begin
   Signal.trap(:INT) { exit 1 }
-  if RUBY_PLATFORM !~ /mswin(?!ce)|mingw|cygwin|bccwin/
+  unless RUBY_PLATFORM.match?(/mswin(?!ce)|mingw|cygwin|bccwin/)
     Signal.trap(:PIPE, 'IGNORE')
   end