8000 Enumerator::Lazy optimized using Procs chaining by gregolsen · Pull Request #150 · ruby/ruby · GitHub
[go: up one dir, main page]
Skip to content

Enumerator::Lazy optimized using Procs chaining #150

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 17 commits into from
Closed

Enumerator::Lazy optimized using Procs chaining #150

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
013a3c4
Enumerator::Lazy changed to use direct procs chaining for #map and #s…
gregolsen Apr 8, 2012
394ab24
lazy_init_iterator cleanup
gregolsen Apr 17, 2012
463760c
using generator as an object for procs chaining when unoptimized lazy…
gregolsen Apr 17, 2012
dda7f1c
move_next madness moved to process_element
gregolsen Apr 17, 2012
b35d33a
lazy_take optimized
gregolsen Apr 28, 2012
4c70750
lazy_drop optimized
gregolsen Apr 28, 2012
a08728f
marking all generators created by unoptimized lazy methods as hybrid
gregolsen Apr 30, 2012
eb11c11
lazy #take_while optimized
gregolsen May 10, 2012
540f7bd
lazy #reject & #drop_while optimized
gregolsen May 10, 2012
8c05fcc
#grep optimized
gregolsen May 15, 2012
6a6284f
instantiating proc_entry properly with marking for GC
gregolsen May 16, 2012
b8af5d6
adding test for hybrid chaining
gregolsen Jun 6, 2012
e40e8ed
fixing multiple force call problem for #take
gregolsen Jun 7, 2012
b2437fc
tiny cleanup
gregolsen Jun 7, 2012
434ff9d
#inspect fixed to support procs chainged lazy enumerators
gregolsen Jul 14, 2012
68cc5b2
cycle_chain test fixed
gregolsen Jul 31, 2012
30f1146
rb_funcall changed to rb_proc_call_with_block for speed up
gregolsen Jul 31, 2012
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
rb_funcall changed to rb_proc_call_with_block for speed up
  • Loading branch information
@gregolsen
gregolsen committed Jul 31, 2012
commit 30f11469511f6ae56e980d92b24c77b53996f079
18 changes: 6 additions & 12 deletions enumerator.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1160,12 +1160,10 @@ process_element(VALUE procs_array, VALUE yielder, int argc, VALUE* argv)
if (RTEST(move_next)) {
switch ((enum proc_entry_type) entry->type) {
case T_PROC_MAP:
result = rb_funcall(entry->proc, rb_intern("call"),
1, result);
result = rb_proc_call_with_block(entry->proc, 1, &result, Qnil);
break;
case T_PROC_SELECT:
move_next = rb_funcall(entry->proc, rb_intern("call"),
1, result);
move_next = rb_proc_call_with_block(entry->proc, 1, &result, Qnil);
break;
case T_PROC_TAKE:
memo = RNODE(entry->memo);
Expand All @@ -1184,28 +1182,24 @@ process_element(VALUE procs_array, VALUE yielder, int argc, VALUE* argv)
}
break;
case T_PROC_TAKE_WHILE:
move_next = rb_funcall(entry->proc, rb_intern("call"),
1, result);
move_next = rb_proc_call_with_block(entry->proc, 1, &result, Qnil);
if (!RTEST(move_next)) result = Qundef;
break;
case T_PROC_DROP_WHILE:
memo = RNODE(entry->memo);
if (!memo->u3.state) {
move_next = !RTEST(rb_funcall(entry->proc,
rb_intern("call"), 1, result));
move_next = !RTEST(rb_proc_call_with_block(entry->proc, 1, &result, Qnil));
if (move_next) memo->u3.state = TRUE;
}
break;
case T_PROC_REJECT:
move_next = !RTEST(rb_funcall(entry->proc, rb_intern("call"),
1, result));
move_next = !RTEST(rb_proc_call_with_block(entry->proc, 1, &result, Qnil));
break;
case T_PROC_GREP:
move_next = rb_funcall(entry->memo, id_eqq, 1, result);

if (RTEST(move_next) && entry->proc) {
result = rb_funcall(entry->proc,
rb_intern("call"), 1, result);
result = rb_proc_call_with_block(entry->proc, 1, &result, Qnil);
}
break;
}
Expand Down
0