* [ruby-dev:47048] [ruby-trunk - Bug #7878] E::Lazy#grep ignores ^C
[not found] <redmine.issue-7878.20130218233640@ruby-lang.org>
@ 2013-02-19 12:20 ` mame (Yusuke Endoh)
2013-02-19 19:12 ` [ruby-dev:47053] " SASADA Koichi
2013-02-20 7:15 ` [ruby-dev:47058] " mame (Yusuke Endoh)
2013-02-23 23:09 ` [ruby-dev:47082] [ruby-trunk - Bug #7878][Closed] " ko1 (Koichi Sasada)
2 siblings, 1 reply; 4+ messages in thread
From: mame (Yusuke Endoh) @ 2013-02-19 12:20 UTC (permalink / raw
To: ruby developers list
Issue #7878 has been updated by mame (Yusuke Endoh).
Assignee changed from mame (Yusuke Endoh) to ko1 (Koichi Sasada)
ko1 (Koichi Sasada) wrote:
> なぜ私が、と思ったんですが、遠藤さんいい読みですね。
RUBY_VM_CHECK_INTS なんだから笹田さんでしょう。
ko1 (Koichi Sasada) wrote:
> r39308 で取り急ぎ対応しました。
>
> mameさん
> backport してもいいでしょうか。
RUBY_VM_CHECK_INTS したらまずいところで RUBY_VM_CHECK_INTS してしまう副作用がないかが大変気になる(具体的に心当たりがあるわけではない)のですが、大丈夫ですかね? 自信があるなら OK です。
--
Yusuke Endoh <mame@tsg•ne.jp>
----------------------------------------
Bug #7878: E::Lazy#grep ignores ^C
https://bugs.ruby-lang.org/issues/7878#change-36610
Author: shyouhei (Shyouhei Urabe)
Status: Assigned
Priority: Normal
Assignee: ko1 (Koichi Sasada)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-08 trunk 39161) [x86_64-linux]
このタイミングでバグかよとお思いでしょうが見つけちまったもんはしょうがない、
[0].lazy.cycle.grep(1).first
が^Cに反応しません。どこかでRUBY_VM_CHECK_INTSしてください。
--
http://bugs.ruby-lang.org/
^ permalink raw reply [flat|nested] 4+ messages in thread
* [ruby-dev:47053] Re: [ruby-trunk - Bug #7878] E::Lazy#grep ignores ^C
2013-02-19 12:20 ` [ruby-dev:47048] [ruby-trunk - Bug #7878] E::Lazy#grep ignores ^C mame (Yusuke Endoh)
@ 2013-02-19 19:12 ` SASADA Koichi
0 siblings, 0 replies; 4+ messages in thread
From: SASADA Koichi @ 2013-02-19 19:12 UTC (permalink / raw
To: ruby developers list; +Cc: mame
(2013/02/19 21:20), mame (Yusuke Endoh) wrote:
>> なぜ私が、と思ったんですが、遠藤さんいい読みですね。
>
> RUBY_VM_CHECK_INTS なんだから笹田さんでしょう。
てっきり、Lazy cycle 内でなんか凄いことをやっているのかと思っておりま
した。enum.c のほうに変更入れるなら私じゃないだろう、と。
> ko1 (Koichi Sasada) wrote:
>> r39308 で取り急ぎ対応しました。
>>
>> mameさん
>> backport してもいいでしょうか。
>
> RUBY_VM_CHECK_INTS したらまずいところで RUBY_VM_CHECK_INTS してしまう副作用がないかが大変気になる(具体的に心当たりがあるわけではない)のですが、大丈夫ですかね? 自信があるなら OK です。
これに関して、実は 1.9.3 に戻したというのが実情です RUBY_VM_CHECK_INTS
を外して綺麗になって速くなるかな、と思ったんですが、ちょっとこの挙動(C
method の繰り返しの中で rb_funcall している)を思いつかなかったので、デ
グレ対応と言えるかと思います。というわけで、自信があります。
本当は、もうちょっと綺麗になりそうだな、という手段を思いついたのです
が、怖いので 1.9.3 相当に戻す、という対応になっております。具体的には、
vm_call0_body() の冒頭、 th->passed_block をクリアした後に CHECK_INTS す
ればよい、はず。ただし、影響が読めないので、2.1 送り。
よろしくお願いします。
--
// SASADA Koichi at atdot dot net
^ permalink raw reply [flat|nested] 4+ messages in thread
* [ruby-dev:47058] [ruby-trunk - Bug #7878] E::Lazy#grep ignores ^C
[not found] <redmine.issue-7878.20130218233640@ruby-lang.org>
2013-02-19 12:20 ` [ruby-dev:47048] [ruby-trunk - Bug #7878] E::Lazy#grep ignores ^C mame (Yusuke Endoh)
@ 2013-02-20 7:15 ` mame (Yusuke Endoh)
2013-02-23 23:09 ` [ruby-dev:47082] [ruby-trunk - Bug #7878][Closed] " ko1 (Koichi Sasada)
2 siblings, 0 replies; 4+ messages in thread
From: mame (Yusuke Endoh) @ 2013-02-20 7:15 UTC (permalink / raw
To: ruby developers list
Issue #7878 has been updated by mame (Yusuke Endoh).
ko1 (Koichi Sasada) wrote:
> これに関して、実は 1.9.3 に戻したというのが実情です RUBY_VM_CHECK_INTS
> を外して綺麗になって速くなるかな、と思ったんですが、ちょっとこの挙動(C
> method の繰り返しの中で rb_funcall している)を思いつかなかったので、デ
> グレ対応と言えるかと思います。というわけで、自信があります。
なるほど、とても安心できる説明ありがとうございます。
バックポートお願いします。
--
Yusuke Endoh <mame@tsg•ne.jp>
----------------------------------------
Bug #7878: E::Lazy#grep ignores ^C
https://bugs.ruby-lang.org/issues/7878#change-36657
Author: shyouhei (Shyouhei Urabe)
Status: Assigned
Priority: Normal
Assignee: ko1 (Koichi Sasada)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-08 trunk 39161) [x86_64-linux]
このタイミングでバグかよとお思いでしょうが見つけちまったもんはしょうがない、
[0].lazy.cycle.grep(1).first
が^Cに反応しません。どこかでRUBY_VM_CHECK_INTSしてください。
--
http://bugs.ruby-lang.org/
^ permalink raw reply [flat|nested] 4+ messages in thread
* [ruby-dev:47082] [ruby-trunk - Bug #7878][Closed] E::Lazy#grep ignores ^C
[not found] <redmine.issue-7878.20130218233640@ruby-lang.org>
2013-02-19 12:20 ` [ruby-dev:47048] [ruby-trunk - Bug #7878] E::Lazy#grep ignores ^C mame (Yusuke Endoh)
2013-02-20 7:15 ` [ruby-dev:47058] " mame (Yusuke Endoh)
@ 2013-02-23 23:09 ` ko1 (Koichi Sasada)
2 siblings, 0 replies; 4+ messages in thread
From: ko1 (Koichi Sasada) @ 2013-02-23 23:09 UTC (permalink / raw
To: ruby developers list
Issue #7878 has been updated by ko1 (Koichi Sasada).
Status changed from Assigned to Closed
merged at r39332.
----------------------------------------
Bug #7878: E::Lazy#grep ignores ^C
https://bugs.ruby-lang.org/issues/7878#change-36848
Author: shyouhei (Shyouhei Urabe)
Status: Closed
Priority: Normal
Assignee: ko1 (Koichi Sasada)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2013-02-08 trunk 39161) [x86_64-linux]
このタイミングでバグかよとお思いでしょうが見つけちまったもんはしょうがない、
[0].lazy.cycle.grep(1).first
が^Cに反応しません。どこかでRUBY_VM_CHECK_INTSしてください。
--
http://bugs.ruby-lang.org/
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-02-23 23:34 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <redmine.issue-7878.20130218233640@ruby-lang.org>
2013-02-19 12:20 ` [ruby-dev:47048] [ruby-trunk - Bug #7878] E::Lazy#grep ignores ^C mame (Yusuke Endoh)
2013-02-19 19:12 ` [ruby-dev:47053] " SASADA Koichi
2013-02-20 7:15 ` [ruby-dev:47058] " mame (Yusuke Endoh)
2013-02-23 23:09 ` [ruby-dev:47082] [ruby-trunk - Bug #7878][Closed] " ko1 (Koichi Sasada)
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).