ruby-changes:28874
From: mame <ko1@a...>
Date: Sat, 25 May 2013 18:55:28 +0900 (JST)
Subject: [ruby-changes:28874] mame:r40926 (trunk): * proc.c (rb_method_entry_min_max_arity): fix missing break in switch.
mame 2013-05-25 18:55:17 +0900 (Sat, 25 May 2013) New Revision: 40926 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=40926 Log: * proc.c (rb_method_entry_min_max_arity): fix missing break in switch. This was introduced in r38236, which is not intentional apparently. This has caused no actual harm because VM_METHOD_TYPE_OPTIMIZED is not used except for OPTIMIZED_METHOD_TYPE_SEND, but may do in future. Coverity Scan found this inadequacy. Modified files: trunk/ChangeLog trunk/proc.c Index: ChangeLog =================================================================== --- ChangeLog (revision 40925) +++ ChangeLog (revision 40926) @@ -1,3 +1,11 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1 +Sat May 25 18:46:23 2013 Yusuke Endoh <mame@t...> + + * proc.c (rb_method_entry_min_max_arity): fix missing break in switch. + This was introduced in r38236, which is not intentional apparently. + This has caused no actual harm because VM_METHOD_TYPE_OPTIMIZED is + not used except for OPTIMIZED_METHOD_TYPE_SEND, but may do in + future. Coverity Scan found this inadequacy. + Sat May 25 18:08:06 2013 Yusuke Endoh <mame@t...> * dir.c (bracket): fix copy-paste error. When the first and last Index: proc.c =================================================================== --- proc.c (revision 40925) +++ proc.c (revision 40926) @@ -1791,6 +1791,7 @@ rb_method_entry_min_max_arity(const rb_m https://github.com/ruby/ruby/blob/trunk/proc.c#L1791 default: break; } + break; } case VM_METHOD_TYPE_REFINED: *max = UNLIMITED_ARGUMENTS; -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/