今日は大量の gem がある時などの require のパフォーマンス改善の変更がありました。
大量の gem が存在するため $LOADED_FEATURES が大きくなると require が遅くなっていくのを低減するため $LOADED_FEATURE にファイルの拡張子の違いによる優先度を反映したソートをしておくことで不要な繰り返しの探索を避けられるようにしています。同じ feature name だと拡張子が .rb のほうが優先ですが、$LOADED_FEATURES は load した順序そのままなので先に .so などの拡張ライブラリのほうが先にヒットすると再度検索する必要があって遅くなっていたようです。 [ruby-core:92686] [Bug #15856]
version.h の日付更新
今日は rb_enc_sprintf() などで文字列オブジェクトを埋め込んだ時のエスケープの挙動の変更などがありました。
文字列を sprintf などで比表示文字をエスケープして表示する時に rb_str_inspect() を利用していたのを rb_str_escape() を利用するように変更しています。クオート文字までエスケープされてしまうのを避けるためみたいです。
version.h の日付更新
今日は finalizer の処理内での例外を警告出力するようにする変更や irb の字句解析の不具合修正などがありました。
gc.c の finalize_list() の中で引数 zombie のオブジェクトの struct RBasic::flags を 0 にクリアしていたのをやめています。この後で呼んでいる obj_free_object_id() の中でもクリアされているので冗長だからみたいです。
同じく gc.c の finalize_list() で heap_page_add_freeobj() の引数に渡すためのオブジェクトの slot が所属している page を GET_HEAP_PAGE() で取得していたのを既に変数に格納済みのものを使いまわすようにリファクタリングしています。
irb の字句解析器 IRB::RubyLex で :1 のように数字が後続する Symbol リテラルを許容するようにしているようです。が、数字だけの Symbol リテラルってありでしたっけ? 試してみたら SyntaxError になったけど。
と、思ったけど https://github.com/ruby/irb/issues/256 をみると irb で parse が終わらないために評価が始まらなくてそもそも SyntaxError にすらできないという不具合の修正みたいです。
version.h の日付更新
finalizer の定義時に $SAFE の safe level を保存しておいて実行時に適用するための機構がまだ残っていたのを消しているようです。
ObjectSpace.define_finalizer の実装で同じ finalizer を登録しないようにするチェックで rb_funcall() を利用していたのを直接 rb_equal() を呼ぶようにしています。
gc.c の run_finalizer() で rb_execution_context_t::errinfo の操作に rb_set_errinfo() と rb_errinfo() という API を利用していたのを、ここで扱う ec を変数として持っているので直接構造体のメンバーにアクセスするように変更しています。毎回 GET_EC() するのが無駄だからかな?
finalizer の処理内で例外が発生した時に警告メッセージを出力するようにしています。 [ruby-dev:51050] [Feature #17798]
fc4dd45d0142221880d1b2c9b54dee0597be2b78 で finalizer 内の例外メッセージを警告出力するようにしましたが $VERBOSE が nil 以外に設定されていた時(-w や -W1 以上のコマンドラインオプションが渡されてた時)のみ出力するようにしています。また GC のテストで警告が出る場合があるものがあったので EnvUtil.suppress_warning で抑制するようにしています。 [ruby-dev:51050] [Feature #17798]
NEWS のリンク用の footer の記述を整形しています。
fc4dd45d0142221880d1b2c9b54dee0597be2b78 の finalizer での例外の警告出力について NEWS に追記しています。 [ruby-dev:51050] [Feature #17798]
今日はヒアドキュメント内での式展開の不具合修正などがありました。
ヒアドキュメント内の式の埋め込みが内容によってうまく展開されないのを修正しています。式展開内に式展開を含む文字列リテラルがあった時など。 [ruby-core:104597] [Bug #18038]
version.h の日付更新
gc.c の gc_verify_internal_consistency_() (typo ではない) で objspace_each_objects() を使って全オブジェクトについて呼び出してたのを heap_pages_sorted について直にループして解放済みの slot についても verify_internal_consistency_i() を呼び出すようにしています。コミットログによると tomb heap を対象にするためみたいです。
util.c の ruby_scan_oct() と ruby_scan_hex() の scan したサイズを返す時に int にキャストしていたのを引数の型にあわせて size_t にキャストするように修正しています。
hash.c の rb_hash_stlike_update() で ar_update() や st_update() に渡す key が元々 st_data_t なのに st_data_t へのキャストが書かれたままになってたので不要なキャストを削っています。
range.c の sym_each_i() で rb_yield() が使われていたのをほぼ同じ内容の each_i() に委譲するようにしています。
rb_proc_call_with_block() もほぼ同じ内容の rb_proc_call_with_block_kw() に RB_NO_KEYWORDS を最後の引数を渡して委譲するようにリファクタリングしています。
doc/regexp.rdoc のメタキャラクタ "|" についての説明を 3つ以上の正規表現を連結できるというふうに変更しています。まあ 2つの連結が連続してると解釈してもいいので、正直どっちでもいいような気もしますが。
process.c から未使用になってる mutex/cond 関係の関数群の前方宣言を削除しています。
sprintf.c の quad_t 型の定義についての preprocessor 分岐を整理して不要な分岐を消しています。
今日は主に bundled gems の typeprof と rbs の更新がありました。
object.c の rb_check_to_integer() でオブジェクトのタイプのチェックに RB_INTEGER_TYPE_P() を使うようにリファクタリングしています。
rubyspec の IO#close_on_exec= の戻り値をチェックしているテストを削除しています。 [ruby-core:102998] [Feature #17745]
gems/bundled_gems の typeprof のバージョンをコミットハッシュ指定にしています。 rbs の 1.3 の対応のために typeprof にも変更が必要になったみたいです。
version.h の日付更新
util.c の ruby_scan_hex() という関数で strchr(3) を使った実装を ruby_digit36_to_number_table という配列テーブルを利用した実装に変更しています。
tool/update-bundled_gems.rb で gems/bundled_gems のコミットハッシュを書く記法の時にはそれを取り除くようにしています。
gems/bundled_gems の typeprof と rbs をそれぞれ 0.15.0 と 1.3.1 に更新しています。
今日は irb の irb_info コマンドの Windows での端末の言語設定が英語じゃない時の対応などがありました。
036f26a4e24d7e6e8de30aa7e3cedb19f7da88b0 の switch 文の fall through コメントを UNREACHABLE マクロを置くのに変更しています。
version.h の日付更新
irb の builtin command irb_info で Windows 版の時に chcp の出力を表示する時に、出力メッセージが英語じゃない場合にそなえて切り出すための正規表現のマッチ条件を緩めています。
今日は標準添付ライブラリ racc の更新などがありました。
Exception#full_message のキーワード引数 highlight に true/false/nil 以外が渡された時の例外発生を rb_bool_expected() という関数を利用するようにして、また rb_bool_expected() のメッセージも少し変更しています。
doc/method_documentation.rdoc を doc/documentation_guide.rdoc というファイル名に変更したうえで内容も書き換えてるっぽいです。 https://github.com/ruby/ruby/pull/4600
version.h の日付更新
did_you_mean のテストで assert_match の第1引数が正規表現リテラルなので文法の警告抑制のためメソッド呼び出しの明示的なかっこを追加しています。
拡張ライブラリ openssl のテストで未使用のローカル変数への代入が警告が出てたので削除しています。
拡張ライブラリ racc の lib/racc/pre-setup という既に使われなくなっているスクリプトを削除しています。
標準添付ライブラリ racc の lib/racc/rdoc/grammar.en.rdoc というドキュメントのファイルを削除しています。また gemspec ファイルの spec.files に doc/en/grammar2.en.rdoc というファイルパスの指定を追加しています。重複してドキュメントファイルが含まれてたみたいですね。
同じく標準添付ライブラリ racc の gemspec ファイルの spec.files から sample/ の中の *.y ファイルや Rakefile や JRuby 向けの .java のファイルを削除しています。
同じく標準添付ライブラリ racc の gemspec ファイルの spec.files から 7217f60bb344d2e97fb6bf3ae6cf29c7f5ca7dbf で削除した lib/racc/pre-setup も削除しています。
common.mk の update-bundled_gems のコマンドで $(IFCONFIG) の変数展開をダブルクオートで囲んでいたのをやめています。unneeded とありますがこれって囲んでても問題はないですよね?
error.c の exc_full_message() で switch 文の例外発生させる rb_bool_expected() のあとに break がないため警告が出てたので /* fall through */ というコメントを追加しています。しかしこれは rb_bool_expected() から戻ってこないことがわかっているからなので break を入れるか UNREACHABLE() を書くのが良いのではないかと。