* [ruby-dev:48536] [ruby-trunk - misc #10233] [Open] iseq.cのget_line_info()でバイナリサーチを使うように修正するか
[not found] <redmine.issue-10233.20140912115059@ruby-lang.org>
@ 2014-09-12 11:50 ` shiracha.rikyu
2014-09-12 12:21 ` [ruby-dev:48537] [ruby-trunk - misc #10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか ko1
` (4 subsequent siblings)
5 siblings, 0 replies; 6+ messages in thread
From: shiracha.rikyu @ 2014-09-12 11:50 UTC (permalink / raw)
To: ruby-dev
Issue #10233 has been reported by Rikyu Shiracha.
----------------------------------------
misc #10233: iseq.cのget_line_info()でバイナリサーチを使うように修正するか
https://bugs.ruby-lang.org/issues/10233
* Author: Rikyu Shiracha
* Status: Open
* Priority: Low
* Assignee:
* Category:
* Target version:
----------------------------------------
以下のPullRequestを投げさせていただきました。
https://github.com/ruby/ruby/pull/712
TODOコメントがあって気になったのでその通りの修正をとりあえず作ってみた形です。
ただむしろlinear searchのままにしておいて、コメントを
「ここは主にエラーハンドリングやデバッグに利用される部分でシンプルに保ちたいのでlinear searchを使っている」
という形に修正するべきかもしれない。とも思って迷っており、やはり辞めるべきかレビューを頂きたいです。
--
https://bugs.ruby-lang.org/
^ permalink raw reply [flat|nested] 6+ messages in thread
* [ruby-dev:48537] [ruby-trunk - misc #10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか
[not found] <redmine.issue-10233.20140912115059@ruby-lang.org>
2014-09-12 11:50 ` [ruby-dev:48536] [ruby-trunk - misc #10233] [Open] iseq.cのget_line_info()でバイナリサーチを使うように修正するか shiracha.rikyu
@ 2014-09-12 12:21 ` ko1
2014-09-12 14:45 ` [ruby-dev:48539] " nobu
` (3 subsequent siblings)
5 siblings, 0 replies; 6+ messages in thread
From: ko1 @ 2014-09-12 12:21 UTC (permalink / raw)
To: ruby-dev
Issue #10233 has been updated by Koichi Sasada.
ありがとうございます。
性能評価みたいなことって出来ますか？
（短いと、実は、線形サーチのほうが速かったりして）
----------------------------------------
misc #10233: iseq.cのget_line_info()でバイナリサーチを使うように修正するか
https://bugs.ruby-lang.org/issues/10233#change-48871
* Author: Rikyu Shiracha
* Status: Open
* Priority: Low
* Assignee:
* Category:
* Target version:
----------------------------------------
以下のPullRequestを投げさせていただきました。
https://github.com/ruby/ruby/pull/712
TODOコメントがあって気になったのでその通りの修正をとりあえず作ってみた形です。
ただむしろlinear searchのままにしておいて、コメントを
「ここは主にエラーハンドリングやデバッグに利用される部分でシンプルに保ちたいのでlinear searchを使っている」
という形に修正するべきかもしれない。とも思って迷っており、やはり辞めるべきかレビューを頂きたいです。
--
https://bugs.ruby-lang.org/
^ permalink raw reply [flat|nested] 6+ messages in thread
* [ruby-dev:48539] [ruby-trunk - misc #10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか
[not found] <redmine.issue-10233.20140912115059@ruby-lang.org>
2014-09-12 11:50 ` [ruby-dev:48536] [ruby-trunk - misc #10233] [Open] iseq.cのget_line_info()でバイナリサーチを使うように修正するか shiracha.rikyu
2014-09-12 12:21 ` [ruby-dev:48537] [ruby-trunk - misc #10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか ko1
@ 2014-09-12 14:45 ` nobu
2014-09-13 6:49 ` [ruby-dev:48540] " shiracha.rikyu
` (2 subsequent siblings)
5 siblings, 0 replies; 6+ messages in thread
From: nobu @ 2014-09-12 14:45 UTC (permalink / raw)
To: ruby-dev
Issue #10233 has been updated by Nobuyoshi Nakada.
とりあえずインデントは揃えてください。
----------------------------------------
misc #10233: iseq.cのget_line_info()でバイナリサーチを使うように修正するか
https://bugs.ruby-lang.org/issues/10233#change-48876
* Author: Rikyu Shiracha
* Status: Open
* Priority: Low
* Assignee:
* Category:
* Target version:
----------------------------------------
以下のPullRequestを投げさせていただきました。
https://github.com/ruby/ruby/pull/712
TODOコメントがあって気になったのでその通りの修正をとりあえず作ってみた形です。
ただむしろlinear searchのままにしておいて、コメントを
「ここは主にエラーハンドリングやデバッグに利用される部分でシンプルに保ちたいのでlinear searchを使っている」
という形に修正するべきかもしれない。とも思って迷っており、やはり辞めるべきかレビューを頂きたいです。
--
https://bugs.ruby-lang.org/
^ permalink raw reply [flat|nested] 6+ messages in thread
* [ruby-dev:48540] [ruby-trunk - misc #10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか
[not found] <redmine.issue-10233.20140912115059@ruby-lang.org>
` (2 preceding siblings ...)
2014-09-12 14:45 ` [ruby-dev:48539] " nobu
@ 2014-09-13 6:49 ` shiracha.rikyu
2019-01-25 13:36 ` [ruby-dev:50771] [Ruby trunk Misc#10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか takashikkbn
2019-01-27 2:42 ` [ruby-dev:50774] " mame
5 siblings, 0 replies; 6+ messages in thread
From: shiracha.rikyu @ 2014-09-13 6:49 UTC (permalink / raw)
To: ruby-dev
Issue #10233 has been updated by Rikyu Shiracha.
File benchmark_short.rb added
File benchmark_long.rb added
File benchmark_log.txt added
Koichi Sasada wrote:
> ありがとうございます。
> 性能評価みたいなことって出来ますか？
> （短いと、実は、線形サーチのほうが速かったりして）
これでよいかちょっとわからないのですが、一応とってみました。
添付のbenchmark_short/long.rbをパッチとパッチを当てる直前のコミットで比較してあります。
結果がbenchmark_log.txtです。
内容としては例外が主な用途だろうということでその場合の動作で見ています。
特に有意差はなさそうな感じですね。。。複雑になるだけかもしれない。
Nobuyoshi Nakada wrote:
> とりあえずインデントは揃えてください。
すいません！ammendコミットをforce pushして直しておきました。
(最初はタブとスペースが混ざったとても変わったフォーマッティングだと思ってそのままの見た目にしたがってしまったんですが、
改めて読むと4spaceでインデント合わせしてるけどtabstopは8で、コード頭のほうのスペースが8になったらタブに置き換えるスタイルだったんですね...
vimでいうと noexpandtab tabstop=8。)
----------------------------------------
misc #10233: iseq.cのget_line_info()でバイナリサーチを使うように修正するか
https://bugs.ruby-lang.org/issues/10233#change-48887
* Author: Rikyu Shiracha
* Status: Open
* Priority: Low
* Assignee:
* Category:
* Target version:
----------------------------------------
以下の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/
^ permalink raw reply [flat|nested] 6+ messages in thread
* [ruby-dev:50771] [Ruby trunk Misc#10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか
[not found] <redmine.issue-10233.20140912115059@ruby-lang.org>
` (3 preceding siblings ...)
2014-09-13 6:49 ` [ruby-dev:48540] " shiracha.rikyu
@ 2019-01-25 13:36 ` takashikkbn
2019-01-27 2:42 ` [ruby-dev:50774] " mame
5 siblings, 0 replies; 6+ messages in thread
From: takashikkbn @ 2019-01-25 13:36 UTC (permalink / raw)
To: ruby-dev
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/
^ permalink raw reply [flat|nested] 6+ messages in thread
* [ruby-dev:50774] [Ruby trunk Misc#10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか
[not found] <redmine.issue-10233.20140912115059@ruby-lang.org>
` (4 preceding siblings ...)
2019-01-25 13:36 ` [ruby-dev:50771] [Ruby trunk Misc#10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか takashikkbn
@ 2019-01-27 2:42 ` mame
5 siblings, 0 replies; 6+ messages in thread
From: mame @ 2019-01-27 2:42 UTC (permalink / raw)
To: ruby-dev
Issue #10233 has been updated by mame (Yusuke Endoh).
このチケットを全く知りませんでした（すみません）が、2.5 で二分探索にしました。さらにその後 2.6 では簡潔ビットベクトルで O(1) にしました。
----------------------------------------
Misc #10233: iseq.cのget_line_info()でバイナリサーチを使うように修正するか
https://bugs.ruby-lang.org/issues/10233#change-76544
* 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/
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-01-27 2:42 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <redmine.issue-10233.20140912115059@ruby-lang.org>
2014-09-12 11:50 ` [ruby-dev:48536] [ruby-trunk - misc #10233] [Open] iseq.cのget_line_info()でバイナリサーチを使うように修正するか shiracha.rikyu
2014-09-12 12:21 ` [ruby-dev:48537] [ruby-trunk - misc #10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか ko1
2014-09-12 14:45 ` [ruby-dev:48539] " nobu
2014-09-13 6:49 ` [ruby-dev:48540] " shiracha.rikyu
2019-01-25 13:36 ` [ruby-dev:50771] [Ruby trunk Misc#10233] iseq.cのget_line_info()でバイナリサーチを使うように修正するか takashikkbn
2019-01-27 2:42 ` [ruby-dev:50774] " mame
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).