[ruby-dev:50771] [Ruby trunk Misc#10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか
From:
takashikkbn@...
Date:
2019-01-25 13:36:51 UTC
List:
ruby-dev #50771
Issue #10233 has been updated by k0kubun (Takashi Kokubun). Status changed from Open to Closed たまたまGitHubで見かけて、現状のtrunkには適用できない状態になっていたので、closeします。 ---------------------------------------- Misc #10233: iseq.cのget_line_info()でバイナリサーチを使うように修正するか https://bugs.ruby-lang.org/issues/10233#change-76511 * Author: shiracha (Rikyu Shiracha) * Status: Closed * Priority: Normal * Assignee: ---------------------------------------- 以下のPullRequestを投げさせていただきました。 https://github.com/ruby/ruby/pull/712 TODOコメントがあって気になったのでその通りの修正をとりあえず作ってみた形です。 ただむしろlinear searchのままにしておいて、コメントを 「ここは主にエラーハンドリングやデバッグに利用される部分でシンプルに保ちたいのでlinear searchを使っている」 という形に修正するべきかもしれない。とも思って迷っており、やはり辞めるべきかレビューを頂きたいです。 ---Files-------------------------------- benchmark_short.rb (1.5 KB) benchmark_long.rb (7.65 KB) benchmark_log.txt (1.69 KB) -- https://bugs.ruby-lang.org/