ReVIEW.loggerが冗長にすぎる

[kmuto]

ログ系はできるだけloggerに統一ということで取り込んだ機能ですが、デフォルトのは冗長すぎる気がします。
デフォルト設定を変えることはできるでしょうか（そもそもこれの切り替えはどうやるのを想定していたんでしたっけ…）。

[takahashim]

review/lib/review/logger.rb

Line 7 in d893e14

super(STDERR, formatter: ->(severity, _datetime, _progname, msg) { "#{severity}: #{msg}\n" })

この辺でReVIEW::Logger.new()したときには「ログレベル: メッセージ」だけにしてるつもりだったんですが、なんか変な使い方になってるんですかね…

コードの挙動はまだ確認できてないです。

[kdmsnr]

手元で動かした限り #962 みたいには出ませんでした。

:
[1] [2] (./book.aux) )
Output written on book.dvi (4 pages, 2868 bytes).
Transcript written on book.log.
INFO: dvipdfmx -d 5 book.dvi
book.dvi -> book.pdf
:

[kmuto]

おや… Linuxだからかなぁ。

[kmuto]

macでもダメだな、

formatter: ->(severity, _datetime, _progname, msg) { "#{severity}: #{msg}\n" }

が効かないみたい。試しにlogger.formatter = proc... で直接指定したら短くなりました。

[kdmsnr]

formatter は Ruby 2.4 からの引数ですかね。
http://ruby-doc.org/stdlib-2.3.0/libdoc/logger/rdoc/Logger.html#method-c-new
http://ruby-doc.org/stdlib-2.4.0/libdoc/logger/rdoc/Logger.html#method-c-new

[kmuto]

おぉ、なるほど。ありがとうございます。
今までずっと「なんか長いなー」と思っていたのはバグだったのか…

[takahashim]

なんと、そういうオチだったんですね…。