review 3.2.0 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fd7268a71d883dbd67373a3a6c3d4d151b67cb01e6545bf83fc1b26ce9eaee02
-  data.tar.gz: 3214804b54cba60f961ceda0ee83f8f2d50080ba3bbd691cca179057bd8dc3ac
+  metadata.gz: 89a4cecb9a961ec87c1238e52ceadaf474caeeef156ff3d22f2d856ed0959511
+  data.tar.gz: 869b3b59d13a0fa2fc8b5812f0c52ae63f3e1faaad59c3ce0d0230d7baacefc4
 SHA512:
-  metadata.gz: bc7ccb389d4c6d836adf683f53961646e126149a63c29cda14ecbc4b95b44745a856169326ab98e84d5189b8fca4bb08dbf7292089990231a1c22063ccf58557
-  data.tar.gz: 49790ebe6d85e685b1e9fbc0f1bc635a050ca844c8dd34369e79dee8972884c5bb1f2f9cce194ce7c7947d303503b4ba549741ab1f8069a924309b6696934701
+  metadata.gz: 45a497fd65af6dce68031d1ebf4fd04e961462e667c37b1e81082dae503c7d562aa165664c67794731a6997c439066161be3344b58811b6e04ec482b7a07c0ec
+  data.tar.gz: 8944e5336301dc89c65de91656b7084a322b50fdf699ca3e82748206a0c0a5b49a0199e095e25be8258145ec3bde1564de5e7273e31e4272cfc12641f1d0b163

data/.rubocop.yml

@@ -193,6 +193,31 @@ Style/LineEndConcatenation:
 Style/MethodCallWithoutArgsParentheses:
   Enabled: true
 
+Style/MethodCallWithArgsParentheses:
+  IgnoredMethods:
+    - 'require'
+    - 'include'
+    - 'file'
+    - 'task'
+    - 'desc'
+    - 'info'
+    - 'gem'
+    - 'exit'
+    - 'print'
+    - 'puts'
+    - 'write'
+    - 'warn'
+    - 'error'
+    - 'raise'
+    - 'yield'
+    - 'source'
+    - 'assert'
+    - 'assert_equal'
+    - 'sh'
+    - 'mv'
+    - 'rm_rf'
+  Enabled: true
+
 Style/MixinUsage:
   Enabled: false
 
@@ -480,8 +505,5 @@ Naming/VariableNumber:
 
 #### Security
 
-# we use Date object in config.yml so should use YAML.load :-/
 Security/YAMLLoad:
-  Exclude:
-    - 'lib/review/catalog.rb'
-    - 'test/test_helper.rb'
+  Enabled: true

data/.travis.yml

@@ -16,7 +16,7 @@ addons:
 
 before_install:
   - |-
-    gem update --system
+##    gem update --system
     gem install bundler
     gem install epubcheck-ruby
 

data/NEWS.ja.md

@@ -1,6 +1,102 @@
+# Version 4.0.0
+## 新機能
+* IDGXML ファイルをまとめて生成する、review-idgxmlmaker を導入しました ([#1337])
+* review-textmaker は、imgmath パラメータが有効になっている場合に、数式を画像化するようになりました ([#1338])
+* review-init に `-w` オプションを指定することで、Web ブラウザ上で TeX のレイアウトができるウィザードモードを用意しました。なお、この機能は実験的であり、将来別のものに置き換える可能性もあります ([#1403])
+* 実験的実装として、複数行から段落を結合する際に、前後の文字の種類に基づいて空白文字の挿入を行う機能を追加しました。この機能を利用するには、unicode-eaw gem をインストールした上で、config.yml に `join_lines_by_lang: true` を追加してください [#1362]
+
+## 非互換の変更
+* 通常の利用では使われることがないので、review-init の実行時に空の layouts フォルダを作成するのをやめました ([#1340])
+* PDFMaker: `@<code>`、`@<tt>`、`@<tti>`、`@<ttb>` で空白文字が消えてしまう問題を修正しました。および利便性のために、文字列が版面からあふれるときに途中で改行するようにもしました ([#1348])
+* `//texequation`、`//embed`、`//graph` はもともとインライン命令を許容しないので、内容のエスケープもしないようにしました。また、末尾に余計な空行が加わるのも防ぐようにしました ([#1371], [#1374])
+* PDFMaker: コラム内での使用を考えて、表の配置のデフォルトを htp (指定位置→ページ上→独立ページの順に試行) から H (絶対に指定位置) にしました (review-style.sty の `\floatplacement{table}` の値) [#1385]
+* PDFMaker: コードリスト内では和文欧文間の空きを 1/4 文字ではなく 0 にするようにしました ([#1401])
+* config.yml の目次を制御する toc パラメータの値は、これまで null (false、目次は作らない) でしたが、一般的な利用方法を鑑みて、デフォルトを true (目次を作る) に切り替えました ([#1405])
+
+## バグ修正
+* review-jlreq がタイプミスのために一部の jlreq.cls バージョンで正しく動作しないのを修正しました ([#1350])
+* re ファイルが改行コード CR で記述されたときに不正な結果になるのを修正しました ([#1341])
+* PDFMaker: review-jlreq において `//cmd` のブロックがページをまたいだときに文字色が黒になって見えなくなってしまうのを修正しました ([#1363])
+* PDFMaker: `@<column>` で「コラム」ラベルが重複して出力されるのを修正しました ([#1367])
+* PDFMaker: gentombow.sty と jsbook.cls は review-jsbook の場合のみコピーするようにしました ([#1381])
+* PDFMaker: LuaLaTeX で review-jlreq を使ったときに壊れた PDFDocumentInformation ができる問題を修正しました ([#1392])
+* PDFMaker: review-jlreq で偶数ページに隠しノンブルが入らなかったのを修正しました ([#1395])
+
+## 機能強化
+* IDGXML ビルダで `@<em>` および `@<strong>` をサポートしました ([#1353])
+* PDFMaker: コードブロックの各行の処理を `code_line`, `code_line_num` のメソッドに切り出しました ([#1368])
+* PDFMaker: デフォルトのコンパイルオプションに `-halt-on-error` を追加しました。TeX のコンパイルエラーが発生したときに即終了することで問題が把握しやすくなります ([#1378])
+* PDFMaker: コラム内に脚注 (`@<fn>`) があるときの挙動がコラムの実装手段によって異なり、番号がずれるなどの問題を起こすことがあるため、脚注の文章 (`//footnote`) はコラムの後に置くことを推奨します。コラム内に脚注文章が存在する場合は警告するようにしました ([#1379])
+* YAML ファイルのエラーチェックを強化しました ([#1386])
+* Logger での表示時に標準の progname を使うようにしました ([#1388])
+* PDFMaker: 電子版の作成時に、表紙のページ番号を偶数とし、名前を「cover」にするようにしました ([#1402])
+* PDFMaker: `generate_pdf` メソッドのリファクタリングを行いました ([#1404])
+* プロジェクトの新規作成時に登録除外ファイル一覧の .gitignore ファイルを置くようにしました ([#1407])
+
+## ドキュメント
+* sample-book の README.md を更新しました ([#1354])
+* review-jsbook の README.md に jsbook.cls のオプションの説明を追加しました ([#1365])
+
+## その他
+* メソッド引数のコーディングルールを統一しました ([#1360])
+* `Catalog#{chaps,parts,predef,postdef,appendix}` は String ではなく Array を返すようにしました ([#1372])
+* YAML ファイルの読み込みに `safe_load` を使うようにしました ([#1375])
+* `table` メソッドをリファクタリングし、ビルダ個々の処理を簡略化しました ([#1356])
+* `XXX_header` と `XXX_body` まわりをリファクタリングしました ([#1359])
+* `Builder#highlight?` メソッドを HTMLBuilder 以外でも利用できるようにしました ([#1373])
+* mkchap* と mkpart* まわりをリファクタリングしました ([#1383])
+* Travis CI で rubygems を更新しないようにしました ([#1389])
+* Index まわりをリファクタリングしました ([#1390])
+* samples フォルダのサンプルドキュメントに review-jlreq のための設定を追加しました ([#1391])
+* 用語リストは `:` の前にスペースを入れることを強く推奨するようにしました。スペースがない場合、警告されます ([#1398])
+
+## コントリビューターのみなさん
+* [@m-shibata](https://github.com/m-shibata)
+* [@masarakki](https://github.com/masarakki)
+
+[#1337]: https://github.com/kmuto/review/issues/1337
+[#1338]: https://github.com/kmuto/review/issues/1338
+[#1340]: https://github.com/kmuto/review/issues/1340
+[#1341]: https://github.com/kmuto/review/issues/1341
+[#1348]: https://github.com/kmuto/review/issues/1348
+[#1350]: https://github.com/kmuto/review/issues/1350
+[#1353]: https://github.com/kmuto/review/pull/1353
+[#1354]: https://github.com/kmuto/review/pull/1354
+[#1356]: https://github.com/kmuto/review/pull/1356
+[#1359]: https://github.com/kmuto/review/pull/1359
+[#1360]: https://github.com/kmuto/review/pull/1360
+[#1362]: https://github.com/kmuto/review/pull/1362
+[#1363]: https://github.com/kmuto/review/issues/1363
+[#1365]: https://github.com/kmuto/review/pull/1365
+[#1367]: https://github.com/kmuto/review/issues/1367
+[#1368]: https://github.com/kmuto/review/issues/1368
+[#1371]: https://github.com/kmuto/review/pull/1371
+[#1372]: https://github.com/kmuto/review/pull/1372
+[#1373]: https://github.com/kmuto/review/pull/1373
+[#1374]: https://github.com/kmuto/review/pull/1374
+[#1375]: https://github.com/kmuto/review/pull/1375
+[#1378]: https://github.com/kmuto/review/pull/1378
+[#1379]: https://github.com/kmuto/review/issues/1379
+[#1381]: https://github.com/kmuto/review/issues/1381
+[#1383]: https://github.com/kmuto/review/issues/1383
+[#1385]: https://github.com/kmuto/review/issues/1385
+[#1386]: https://github.com/kmuto/review/pull/1386
+[#1388]: https://github.com/kmuto/review/pull/1388
+[#1389]: https://github.com/kmuto/review/pull/1389
+[#1390]: https://github.com/kmuto/review/pull/1390
+[#1391]: https://github.com/kmuto/review/pull/1391
+[#1392]: https://github.com/kmuto/review/issues/1392
+[#1395]: https://github.com/kmuto/review/issues/1395
+[#1398]: https://github.com/kmuto/review/issues/1398
+[#1401]: https://github.com/kmuto/review/pull/1401
+[#1402]: https://github.com/kmuto/review/pull/1402
+[#1403]: https://github.com/kmuto/review/pull/1403
+[#1404]: https://github.com/kmuto/review/pull/1404
+[#1405]: https://github.com/kmuto/review/pull/1405
+[#1407]: https://github.com/kmuto/review/pull/1407
+
 # Version 3.2.0
 
-## 新機能
 ## 非互換の変更
 * PDFMaker: `//image` 命令などで画像を配置するときに `\includegraphics` ではなく、それを抽象化した `\reviewincludegraphics` を使うようにしました ([#1318])
 

data/NEWS.md

@@ -1,6 +1,102 @@
+# Version 4.0.0
+## New Features
+* introduce review-idgxmlmaker which generates IDGXML files at once ([#1337])
+* review-textmaker converts the math in the document to image files when `imgmath` parameter has `true` ([#1338])
+* introduce wizard mode to layout of LaTeX on Web browser. Add `-w` option to review-init. This feature is experimental and may be replaced in the future ([#1403])
+* experimental feature: introduce the feature to insert whitespace based on character when combining lines into a paragraph. To enable this, install unicode-eaw gem and add `join_lines_by_lang: true` into config.yml [#1362]
+
+## Breaking Changes
+* review-init no longer creates empty `layouts` folder ([#1340])
+* PDFMaker: fix a problem that white space characters disappeared in `@<code>`, `@<tt>`, `@<tti>`, and `@<ttb>`. Also the string is automatically wrapped ([#1348])
+* `//texequation`、`//embed` and `//graph` that don't allow inline op no longer escape inline op in strings. And don't put extra line break ([#1371], [#1374])
+* PDFMaker: change the default table placement from `htp` to `H` for use in columns (`\floatplacement{table}` value in review-style.sty) [#1385]
+* PDFMaker: the space between Japanese/Western characters in the code lists is changed to 0 from 1/4 character ([#1401])
+* change the default value of `toc` parameter from null (false, don't create a table of contents) to true (create a table of contents) ([#1405])
+
+## Bug Fixes
+* fix a typo in review-jlreq ([#1350])
+* fix incorrect result when `re` file uses CR for line-feed code ([#1341])
+* PDFMaker: fix foreground color of `//cmd` with review-jlreq after page breaking ([#1363])
+* PDFMaker: fix duplicate 'column' label for `@<column>` ([#1367])
+* PDFMaker: copy gentombow.sty and jsbook.cls only for review-jsbook ([#1381])
+* PDFMaker: fix invalid PDFDocumentInformation on review-jlreq with LuaLaTeX ([#1392])
+* PDFMaker: fix missing hiddenfolio information at even pages on review-jlreq ([#1395])
+
+## Enhancements
+* support `@<em>` and `@<strong>` in IDGXMLBuilder ([#1353])
+* PDFMaker: extract `code_line` and `code_line_num` from code blocks for ease handling each line ([#1368])
+* PDFMaker: add new compile option `-halt-on-error` to make it easier to find the problem when an error occurs ([#1378])
+* PDFMaker: when there is a footnote text (`//footnote`) in the column block, it may cuse problems such as numbering. So warn it if there is. ([#1379])
+* Logger: progname should be add in logger, not in message arguments ([#1388])
+* improve error checking for yaml files ([#1386])
+* PDFMaker: the cover page becomes even number (p.0) and is named "cover" ([#1402])
+* PDFMaker: refactor `generate_pdf` method ([#1404])
+* create `.gitignore` for new project ([#1407])
+
+## Docs
+* update sample-book/README.md ([#1354])
+* add descriptions about options of jsbook.cls to review-jsbook/README.md ([#1365])
+
+## Others
+* unify styles of a method with arguments ([#1360])
+* `Catalog#{chaps,parts,predef,postdef,appendix}` should return Array, not String ([#1372])
+* use `safe_load` for loading YAML ([#1375])
+* refactor `table` method to simplify each builder ([#1356])
+* refactor `XXX_header` and `XXX_body` ([#1359])
+* enable `Builder#highlight?` method on each builder ([#1373])
+* refactor mkdchap* and mkpart* ([#1383])
+* don't update rubygems in Travis CI ([#1389])
+* refactor around Index ([#1390])
+* add configration for review-jlreq to sample documents ([#1391])
+* definition list should start with spaces ([#1398])
+
+## Contributors
+* [@m-shibata](https://github.com/m-shibata)
+* [@masarakki](https://github.com/masarakki)
+
+[#1337]: https://github.com/kmuto/review/issues/1337
+[#1338]: https://github.com/kmuto/review/issues/1338
+[#1340]: https://github.com/kmuto/review/issues/1340
+[#1341]: https://github.com/kmuto/review/issues/1341
+[#1348]: https://github.com/kmuto/review/issues/1348
+[#1350]: https://github.com/kmuto/review/issues/1350
+[#1353]: https://github.com/kmuto/review/pull/1353
+[#1354]: https://github.com/kmuto/review/pull/1354
+[#1356]: https://github.com/kmuto/review/pull/1356
+[#1359]: https://github.com/kmuto/review/pull/1359
+[#1360]: https://github.com/kmuto/review/pull/1360
+[#1362]: https://github.com/kmuto/review/pull/1362
+[#1363]: https://github.com/kmuto/review/issues/1363
+[#1365]: https://github.com/kmuto/review/pull/1365
+[#1367]: https://github.com/kmuto/review/issues/1367
+[#1368]: https://github.com/kmuto/review/issues/1368
+[#1371]: https://github.com/kmuto/review/pull/1371
+[#1372]: https://github.com/kmuto/review/pull/1372
+[#1373]: https://github.com/kmuto/review/pull/1373
+[#1374]: https://github.com/kmuto/review/pull/1374
+[#1375]: https://github.com/kmuto/review/pull/1375
+[#1378]: https://github.com/kmuto/review/pull/1378
+[#1379]: https://github.com/kmuto/review/issues/1379
+[#1381]: https://github.com/kmuto/review/issues/1381
+[#1383]: https://github.com/kmuto/review/issues/1383
+[#1385]: https://github.com/kmuto/review/issues/1385
+[#1386]: https://github.com/kmuto/review/pull/1386
+[#1388]: https://github.com/kmuto/review/pull/1388
+[#1389]: https://github.com/kmuto/review/pull/1389
+[#1390]: https://github.com/kmuto/review/pull/1390
+[#1391]: https://github.com/kmuto/review/pull/1391
+[#1392]: https://github.com/kmuto/review/issues/1392
+[#1395]: https://github.com/kmuto/review/issues/1395
+[#1398]: https://github.com/kmuto/review/issues/1398
+[#1401]: https://github.com/kmuto/review/pull/1401
+[#1402]: https://github.com/kmuto/review/pull/1402
+[#1403]: https://github.com/kmuto/review/pull/1403
+[#1404]: https://github.com/kmuto/review/pull/1404
+[#1405]: https://github.com/kmuto/review/pull/1405
+[#1407]: https://github.com/kmuto/review/pull/1407
+
 # Version 3.2.0
 
-## New Features
 ## Breaking Changes
 * PDFMaker: changed to use the abstract name `\reviewincludegraphics` instead of `\includegraphics` for image placements (such as `//image`) ([#1318])
 

data/bin/review-catalog-converter

@@ -140,7 +140,7 @@ end
 def replace_old_style_part(dir)
   catalog = ReVIEW::Catalog.new(File.open("#{dir}/catalog.yml"))
   Array.new(catalog.parts.split("\n")).each_with_index do |part, part_idx|
-    next if part.end_with? '.re'
+    next if part.end_with?('.re')
 
     allow_overwrite = true
     part_name = sprintf('part%d.re', part_idx + 1)

data/bin/review-check

@@ -50,7 +50,7 @@ def main
   end
   opts.on('--text', 'Check text.') do
     modes ||= []
-    modes.push :text
+    modes.push(:text)
   end
   opts.on('--help', 'print this message and quit.') do
     puts opts.help
@@ -72,7 +72,7 @@ def main
   modes.each do |mode|
     case mode
     when :text
-      check_text files
+      check_text(files)
     else
       raise 'must not happen'
     end
@@ -122,9 +122,9 @@ def words_re(rc)
     next if line[0, 1] == '#'
     if / !/ =~ line
       line, n = *line.split('!', 2)
-      nega.push n.strip
+      nega.push(n.strip)
     end
-    words.push line.strip
+    words.push(line.strip)
   end
   return Regexp.compile(words.join('|')),
          nega.empty? ? nil : Regexp.compile(nega.join('|'))
@@ -157,7 +157,7 @@ def each_paragraph(f)
         break if line.strip.empty?
         break if %r{\A(?:=|//[\w\}])} =~ line
         next if /\A\#@/ =~ line
-        buf.push line.strip
+        buf.push(line.strip)
       end
       yield buf, lineno
       @review_utils_word_ok = nil

data/bin/review-checkdep

@@ -28,7 +28,7 @@ def main
         @unprovided[kw] = location
       end
     when /\A\#@provide\((.*)\)/
-      provide $1
+      provide($1)
     else
       line.scan(/@<kw>\{(.*?)[,\}]/) { provide $1 }
     end

data/bin/review-compile

@@ -88,7 +88,7 @@ def _main
         compiler = ReVIEW::Compiler.new(load_strategy_class(@target, @check_only))
         result = compiler.compile(chap)
         if @output_filename
-          write @output_filename, result
+          write(@output_filename, result)
         else
           puts result unless @check_only
         end
@@ -99,12 +99,12 @@ def _main
       compiler = ReVIEW::Compiler.new(load_strategy_class(@target, @check_only))
       book.chapters.each do |chap|
         str = compiler.compile(chap)
-        write "#{chap.name}#{compiler.strategy.extname}", str unless @check_only
+        write("#{chap.name}#{compiler.strategy.extname}", str) unless @check_only
       end
       # PART
       book.parts_in_file.each do |part|
         str = compiler.compile(part)
-        write "#{part.name}#{compiler.strategy.extname}", str unless @check_only
+        write("#{part.name}#{compiler.strategy.extname}", str) unless @check_only
       end
     else
       raise "must not happen: #{@mode}"
@@ -170,12 +170,12 @@ def parse_opts
 end
 
 def error(msg)
-  @logger.error "#{File.basename($PROGRAM_NAME, '.*')}: #{msg}"
+  @logger.error msg
   exit 1
 end
 
 def warn(msg)
-  @logger.warn "#{File.basename($PROGRAM_NAME, '.*')}: #{msg}"
+  @logger.warn msg
 end
 
 def load_strategy_class(target, strict)

data/bin/review-idgxmlmaker

@@ -0,0 +1,16 @@
+#!/usr/bin/env ruby
+# Copyright (c) 2019 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/idgxmlmaker'
+
+ReVIEW::IDGXMLMaker.execute(*ARGV)

data/bin/review-index

@@ -45,13 +45,13 @@ def _main
   opts.on('-a', '--all', 'print all chapters.') { source = book }
   opts.on('-p', '--part N', 'list only part N.') do |n|
     source = book.part(Integer(n)) or
-      error_exit "part #{n} does not exist in this book"
+      error_exit("part #{n} does not exist in this book")
   end
   opts.on('-c', '--chapter C', 'list only chapter C.') do |c|
     begin
       source = ReVIEW::Book::Part.new(nil, 1, [book.chapter(c)])
     rescue
-      error_exit "chapter #{c} does not exist in this book"
+      error_exit("chapter #{c} does not exist in this book")
     end
   end
   opts.on('-l', '--level N', 'list upto N level (1..4, default=4)') do |n|
@@ -79,7 +79,7 @@ def _main
   book.load_config(yamlfile) if yamlfile
 
   if source
-    error_exit '-a/-s option and file arguments are exclusive' unless ARGV.empty?
+    error_exit('-a/-s option and file arguments are exclusive') unless ARGV.empty?
   else
     puts opts.help
     exit 0
@@ -88,18 +88,18 @@ def _main
   begin
     printer = printer_class.new(upper, param)
     if source.is_a?(ReVIEW::Book::Part)
-      printer.print_part source
+      printer.print_part(source)
     else
-      printer.print_book source
+      printer.print_book(source)
     end
   rescue ReVIEW::Error, Errno::ENOENT => e
     raise if $DEBUG
-    error_exit e.message
+    error_exit(e.message)
   end
 end
 
 def error_exit(msg)
-  @logger.error "#{File.basename($PROGRAM_NAME)}: #{msg}"
+  @logger.error msg
   exit 1
 end
 

data/bin/review-preproc

@@ -39,7 +39,7 @@ def main
     if ENV['REVIEW_SAFE_MODE'].to_i & 2 > 0
       @logger.warn 'review-preproc-ext.rb is prohibited in safe mode. ignored.'
     else
-      Kernel.load File.expand_path('review-preproc-ext.rb')
+      Kernel.load(File.expand_path('review-preproc-ext.rb'))
     end
   end
 
@@ -71,21 +71,21 @@ def main
     current_file = path
     case mode
     when :output
-      File.open(path) { |f| pp.process f, $stdout }
+      File.open(path) { |f| pp.process(f, $stdout) }
     when :replace
-      File.write "#{path}.pptmp", preproc(pp, path)
-      File.rename "#{path}.pptmp", path
+      File.write("#{path}.pptmp", preproc(pp, path))
+      File.rename("#{path}.pptmp", path)
     when :diff, :check
       tmp = '/tmp/review.pptmp'
       begin
-        File.write tmp, preproc(pp, path)
+        File.write(tmp, preproc(pp, path))
         if mode == :check
-          system "diff -qu #{path} #{tmp} >/dev/null || echo #{path}"
+          system("diff -qu #{path} #{tmp} >/dev/null || echo #{path}")
         else
-          system "diff -u #{path} #{tmp}"
+          system("diff -u #{path} #{tmp}")
         end
       ensure
-        FileUtils.rm_f tmp
+        FileUtils.rm_f(tmp)
       end
     else
       raise "must not happen: #{mode}"
@@ -99,7 +99,7 @@ end
 
 def preproc(pp, path)
   buf = StringIO.new
-  File.open(path) { |f| pp.process f, buf }
+  File.open(path) { |f| pp.process(f, buf) }
   buf.string
 end