From: Daniel Berger Date: 2007-06-23T00:00:19+09:00 Subject: Unnecessary varargs for rb_hash_select --- hash.orig Fri Jun 22 08:42:30 2007 +++ hash.c Fri Jun 22 08:43:12 2007 @@ -853,16 +853,11 @@ */ VALUE -rb_hash_select(argc, argv, hash) - int argc; - VALUE *argv; +rb_hash_select(hash) VALUE hash; { VALUE result; - if (argc > 0) { - rb_raise(rb_eArgError, "wrong number of arguments (%d for 0)", argc); - } result = rb_ary_new(); rb_hash_foreach(hash, select_i, result); return result; @@ -2499,7 +2494,7 @@ rb_define_method(rb_cHash,"shift", rb_hash_shift, 0); rb_define_method(rb_cHash,"delete", rb_hash_delete, 1); rb_define_method(rb_cHash,"delete_if", rb_hash_delete_if, 0); - rb_define_method(rb_cHash,"select", rb_hash_select, -1); + rb_define_method(rb_cHash,"select", rb_hash_select, 0); rb_define_method(rb_cHash,"reject", rb_hash_reject, 0); rb_define_method(rb_cHash,"reject!", rb_hash_reject_bang, 0); rb_define_method(rb_cHash,"clear", rb_hash_clear, 0);