[前][次][番号順一覧][スレッド一覧]

ruby-changes:48866

From: shugo <ko1@a...>
Date: Sun, 3 Dec 2017 09:39:32 +0900 (JST)
Subject: [ruby-changes:48866] shugo:r60984 (trunk): The superclass of a refinement should have BasicObject as its ancestor.

shugo	2017-12-03 09:39:26 +0900 (Sun, 03 Dec 2017)

  New Revision: 60984

  https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=60984

  Log:
    The superclass of a refinement should have BasicObject as its ancestor.
    
    Otherwise, VM_ASSERT(callable_method_entry_p(cme)) in
    prepare_callable_method_entry() fails if VM_CHECK_MODE is 2.

  Modified files:
    trunk/eval.c
Index: eval.c
===================================================================
--- eval.c	(revision 60983)
+++ eval.c	(revision 60984)
@@ -1273,6 +1273,18 @@ hidden_identity_hash_new(void) https://github.com/ruby/ruby/blob/trunk/eval.c#L1273
     return hash;
 }
 
+static VALUE
+refinement_superclass(VALUE superclass)
+{
+    if (RB_TYPE_P(superclass, T_MODULE)) {
+	/* FIXME: Should ancestors of superclass be used here? */
+	return rb_include_class_new(superclass, rb_cBasicObject);
+    }
+    else {
+	return superclass;
+    }
+}
+
 /*!
  * \private
  * \todo can be static?
@@ -1304,10 +1316,7 @@ rb_using_refinement(rb_cref_t *cref, VAL https://github.com/ruby/ruby/blob/trunk/eval.c#L1316
 	}
     }
     FL_SET(module, RMODULE_IS_OVERLAID);
-    if (RB_TYPE_P(superclass, T_MODULE)) {
-	superclass = rb_include_class_new(superclass,
-					  RCLASS_SUPER(superclass));
-    }
+    superclass = refinement_superclass(superclass);
     c = iclass = rb_include_class_new(module, superclass);
     RCLASS_REFINED_CLASS(c) = klass;
 
@@ -1402,10 +1411,7 @@ add_activated_refinement(VALUE activated https://github.com/ruby/ruby/blob/trunk/eval.c#L1411
 	}
     }
     FL_SET(refinement, RMODULE_IS_OVERLAID);
-    if (RB_TYPE_P(superclass, T_MODULE)) {
-	superclass = rb_include_class_new(superclass,
-					  RCLASS_SUPER(superclass));
-    }
+    superclass = refinement_superclass(superclass);
     c = iclass = rb_include_class_new(refinement, superclass);
     RCLASS_REFINED_CLASS(c) = klass;
     refinement = RCLASS_SUPER(refinement);
@@ -1460,13 +1466,9 @@ rb_mod_refine(VALUE module, VALUE klass) https://github.com/ruby/ruby/blob/trunk/eval.c#L1466
     }
     refinement = rb_hash_lookup(refinements, klass);
     if (NIL_P(refinement)) {
+	VALUE superclass = refinement_superclass(klass);
 	refinement = rb_module_new();
-	if (RB_TYPE_P(klass, T_MODULE)) {
-	    rb_include_module(refinement, klass);
-	}
-	else {
-	    RCLASS_SET_SUPER(refinement, klass);
-	}
+	RCLASS_SET_SUPER(refinement, superclass);
 	FL_SET(refinement, RMODULE_IS_REFINEMENT);
 	CONST_ID(id_refined_class, "__refined_class__");
 	rb_ivar_set(refinement, id_refined_class, klass);

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]