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

ruby-changes:26481

From: ko1 <ko1@a...>
Date: Fri, 21 Dec 2012 20:44:20 +0900 (JST)
Subject: [ruby-changes:26481] ko1:r38532 (trunk): * iseq.c (Init_ISeq): remove definition of the following methods:

ko1	2012-12-21 20:44:09 +0900 (Fri, 21 Dec 2012)

  New Revision: 38532

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=38532

  Log:
    * iseq.c (Init_ISeq): remove definition of the following methods:
      ISeq#line_trace_all and ISeq#line_trace_specify because they are
      half baked.
      C APIs are remained as experimental. These functions will be
      renamed, removed their parameters may be changed.
      You can use these methods by C exts. Please give us your comments.

  Modified files:
    trunk/ChangeLog
    trunk/iseq.c

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 38531)
+++ ChangeLog	(revision 38532)
@@ -1,3 +1,12 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1
+Fri Dec 21 20:38:28 2012  Koichi Sasada  <ko1@a...>
+
+	* iseq.c (Init_ISeq): remove definition of the following methods:
+	  ISeq#line_trace_all and ISeq#line_trace_specify because they are
+	  half baked.
+	  C APIs are remained as experimental. These functions will be
+	  renamed, removed their parameters may be changed.
+	  You can use these methods by C exts. Please give us your comments.
+
 Fri Dec 21 20:21:04 2012  Koichi Sasada  <ko1@a...>
 
 	* vm_trace.c (tracepoint_new): add code to support specified thread.
Index: iseq.c
===================================================================
--- iseq.c	(revision 38531)
+++ iseq.c	(revision 38532)
@@ -2076,9 +2076,15 @@ Init_ISeq(void) https://github.com/ruby/ruby/blob/trunk/iseq.c#L2076
     rb_define_method(rb_cISeq, "base_label", iseq_base_label, 0);
     rb_define_method(rb_cISeq, "first_lineno", iseq_first_lineno, 0);
 
-    /* experimental */
+#if 0
+    /* Now, it is experimental. No discussions, no tests. */
+    /* They can be used from C level. Please give us feedback. */
     rb_define_method(rb_cISeq, "line_trace_all", rb_iseq_line_trace_all, 0);
     rb_define_method(rb_cISeq, "line_trace_specify", rb_iseq_line_trace_specify, 2);
+#else
+    (void)rb_iseq_line_trace_all;
+    (void)rb_iseq_line_trace_specify;
+#endif
 
 #if 0 /* TBD */
     rb_define_private_method(rb_cISeq, "marshal_dump", iseq_marshal_dump, 0);

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

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