aboutsummaryrefslogtreecommitdiffstats
path: root/proc.c
diff options
context:
space:
mode:
authornobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2015-06-19 05:53:41 +0000
committernobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2015-06-19 05:53:41 +0000
commite77f9fcb13a157c2478a0e78cdb0b9696e8ded8c (patch)
tree146acf56d92ac965489f640dcadefb33c6f05124 /proc.c
parent19d2532a7920ab6da8b5f7977d4439fa0a198340 (diff)
downloadruby-e77f9fcb13a157c2478a0e78cdb0b9696e8ded8c.tar.gz
proc.c: ArgumentError if no block
* proc.c (rb_mod_define_method): now requires a block direct to this method call. [ruby-core:69655] [Bug #11283] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@50971 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'proc.c')
-rw-r--r--proc.c18
1 files changed, 18 insertions, 0 deletions
diff --git a/proc.c b/proc.c
index e333048af4..adb9f674a2 100644
--- a/proc.c
+++ b/proc.c
@@ -14,6 +14,10 @@
#include "gc.h"
#include "iseq.h"
+/* Proc.new with no block will raise an exception in the future
+ * versions */
+#define PROC_NEW_REQUIRES_BLOCK 0
+
const rb_cref_t *rb_vm_cref_in_context(VALUE self, VALUE cbase);
struct METHOD {
@@ -575,6 +579,7 @@ proc_new(VALUE klass, int8_t is_lambda)
rb_block_t *block;
if (!(block = rb_vm_control_frame_block_ptr(cfp))) {
+#if !PROC_NEW_REQUIRES_BLOCK
cfp = RUBY_VM_PREVIOUS_CONTROL_FRAME(cfp);
if ((block = rb_vm_control_frame_block_ptr(cfp)) != 0) {
@@ -582,6 +587,9 @@ proc_new(VALUE klass, int8_t is_lambda)
rb_warn(proc_without_block);
}
}
+#else
+ if (0)
+#endif
else {
rb_raise(rb_eArgError, proc_without_block);
}
@@ -1671,7 +1679,17 @@ rb_mod_define_method(int argc, VALUE *argv, VALUE mod)
name = argv[0];
id = rb_check_id(&name);
if (argc == 1) {
+#if PROC_NEW_REQUIRES_BLOCK
body = rb_block_lambda();
+#else
+ rb_thread_t *th = GET_THREAD();
+ rb_block_t *block = rb_vm_control_frame_block_ptr(th->cfp);
+ if (!block) rb_raise(rb_eArgError, proc_without_block);
+ body = block->proc;
+ if (!body) {
+ body = rb_vm_make_proc_lambda(th, block, rb_cProc, TRUE);
+ }
+#endif
}
else {
body = argv[1];