nobu	2008-08-31 13:42:42 +0900 (Sun, 31 Aug 2008)

  New Revision: 18974

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

  Log:
    * ext/ripper/eventids2.c (ripper_init_eventids2): use rb_intern_const.
    
    * ext/ripper/tools/generate.rb (generate_eventids1): ditto.
    
    * ext/ripper/tools/generate.rb (generate_eventids2_table): ditto.

  Modified files:
    trunk/ChangeLog
    trunk/ext/ripper/eventids2.c
    trunk/ext/ripper/tools/generate.rb

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 18973)
+++ ChangeLog	(revision 18974)
@@ -1,3 +1,11 @@
+Sun Aug 31 13:42:39 2008  Nobuyoshi Nakada  <nobu@r...>
+
+	* ext/ripper/eventids2.c (ripper_init_eventids2): use rb_intern_const.
+
+	* ext/ripper/tools/generate.rb (generate_eventids1): ditto.
+
+	* ext/ripper/tools/generate.rb (generate_eventids2_table): ditto.
+
 Sun Aug 31 12:56:00 2008  Tanaka Akira  <akr@f...>
 
 	* ext/dl/mkcallback.rb (rb_dl_init_callbacks): fix indent.
Index: ext/ripper/tools/generate.rb
===================================================================
--- ext/ripper/tools/generate.rb	(revision 18973)
+++ ext/ripper/tools/generate.rb	(revision 18974)
@@ -79,13 +79,13 @@
   buf << %Q[    VALUE h;\n]
   buf << %Q[    ID id;\n]
   ids.each do |id, arity|
-    buf << %Q[    ripper_id_#{id} = rb_intern("on_#{id}");\n]
+    buf << %Q[    ripper_id_#{id} = rb_intern_const("on_#{id}");\n]
   end
   buf << %Q[\n]
   buf << %Q[    h = rb_hash_new();\n]
   buf << %Q[    rb_define_const(self, "PARSER_EVENT_TABLE", h);\n]
   ids.each do |id, arity|
-    buf << %Q[    id = rb_intern("#{id}");\n]
+    buf << %Q[    id = rb_intern_const("#{id}");\n]
     buf << %Q[    rb_hash_aset(h, ID2SYM(id), INT2NUM(#{arity}));\n]
   end
   buf << %Q[}\n]
@@ -101,7 +101,7 @@
   buf << %Q[    ID id;\n]
   buf << %Q[    rb_define_const(self, "SCANNER_EVENT_TABLE", h);\n]
   ids.each do |id|
-    buf << %Q[    id = rb_intern("#{id}");\n]
+    buf << %Q[    id = rb_intern_const("#{id}");\n]
     buf << %Q[    rb_hash_aset(h, ID2SYM(id), INT2NUM(1));\n]
   end
   buf << %Q[}\n]
Index: ext/ripper/eventids2.c
===================================================================
--- ext/ripper/eventids2.c	(revision 18973)
+++ ext/ripper/eventids2.c	(revision 18974)
@@ -61,53 +61,53 @@
 static void
 ripper_init_eventids2(VALUE self)
 {
-    ripper_id_backref = rb_intern("on_backref");
-    ripper_id_backtick = rb_intern("on_backtick");
-    ripper_id_comma = rb_intern("on_comma");
-    ripper_id_const = rb_intern("on_const");
-    ripper_id_cvar = rb_intern("on_cvar");
-    ripper_id_embexpr_beg = rb_intern("on_embexpr_beg");
-    ripper_id_embexpr_end = rb_intern("on_embexpr_end");
-    ripper_id_embvar = rb_intern("on_embvar");
-    ripper_id_float = rb_intern("on_float");
-    ripper_id_gvar = rb_intern("on_gvar");
-    ripper_id_ident = rb_intern("on_ident");
-    ripper_id_int = rb_intern("on_int");
-    ripper_id_ivar = rb_intern("on_ivar");
-    ripper_id_kw = rb_intern("on_kw");
-    ripper_id_lbrace = rb_intern("on_lbrace");
-    ripper_id_lbracket = rb_intern("on_lbracket");
-    ripper_id_lparen = rb_intern("on_lparen");
-    ripper_id_nl = rb_intern("on_nl");
-    ripper_id_op = rb_intern("on_op");
-    ripper_id_period = rb_intern("on_period");
-    ripper_id_rbrace = rb_intern("on_rbrace");
-    ripper_id_rbracket = rb_intern("on_rbracket");
-    ripper_id_rparen = rb_intern("on_rparen");
-    ripper_id_semicolon = rb_intern("on_semicolon");
-    ripper_id_symbeg = rb_intern("on_symbeg");
-    ripper_id_tstring_beg = rb_intern("on_tstring_beg");
-    ripper_id_tstring_content = rb_intern("on_tstring_content");
-    ripper_id_tstring_end = rb_intern("on_tstring_end");
-    ripper_id_words_beg = rb_intern("on_words_beg");
-    ripper_id_qwords_beg = rb_intern("on_qwords_beg");
-    ripper_id_words_sep = rb_intern("on_words_sep");
-    ripper_id_regexp_beg = rb_intern("on_regexp_beg");
-    ripper_id_regexp_end = rb_intern("on_regexp_end");
-    ripper_id_label = rb_intern("on_label");
-    ripper_id_tlambda = rb_intern("on_tlambda");
-    ripper_id_tlambeg = rb_intern("on_tlambeg");
+    ripper_id_backref = rb_intern_const("on_backref");
+    ripper_id_backtick = rb_intern_const("on_backtick");
+    ripper_id_comma = rb_intern_const("on_comma");
+    ripper_id_const = rb_intern_const("on_const");
+    ripper_id_cvar = rb_intern_const("on_cvar");
+    ripper_id_embexpr_beg = rb_intern_const("on_embexpr_beg");
+    ripper_id_embexpr_end = rb_intern_const("on_embexpr_end");
+    ripper_id_embvar = rb_intern_const("on_embvar");
+    ripper_id_float = rb_intern_const("on_float");
+    ripper_id_gvar = rb_intern_const("on_gvar");
+    ripper_id_ident = rb_intern_const("on_ident");
+    ripper_id_int = rb_intern_const("on_int");
+    ripper_id_ivar = rb_intern_const("on_ivar");
+    ripper_id_kw = rb_intern_const("on_kw");
+    ripper_id_lbrace = rb_intern_const("on_lbrace");
+    ripper_id_lbracket = rb_intern_const("on_lbracket");
+    ripper_id_lparen = rb_intern_const("on_lparen");
+    ripper_id_nl = rb_intern_const("on_nl");
+    ripper_id_op = rb_intern_const("on_op");
+    ripper_id_period = rb_intern_const("on_period");
+    ripper_id_rbrace = rb_intern_const("on_rbrace");
+    ripper_id_rbracket = rb_intern_const("on_rbracket");
+    ripper_id_rparen = rb_intern_const("on_rparen");
+    ripper_id_semicolon = rb_intern_const("on_semicolon");
+    ripper_id_symbeg = rb_intern_const("on_symbeg");
+    ripper_id_tstring_beg = rb_intern_const("on_tstring_beg");
+    ripper_id_tstring_content = rb_intern_const("on_tstring_content");
+    ripper_id_tstring_end = rb_intern_const("on_tstring_end");
+    ripper_id_words_beg = rb_intern_const("on_words_beg");
+    ripper_id_qwords_beg = rb_intern_const("on_qwords_beg");
+    ripper_id_words_sep = rb_intern_const("on_words_sep");
+    ripper_id_regexp_beg = rb_intern_const("on_regexp_beg");
+    ripper_id_regexp_end = rb_intern_const("on_regexp_end");
+    ripper_id_label = rb_intern_const("on_label");
+    ripper_id_tlambda = rb_intern_const("on_tlambda");
+    ripper_id_tlambeg = rb_intern_const("on_tlambeg");
 
-    ripper_id_ignored_nl = rb_intern("on_ignored_nl");
-    ripper_id_comment = rb_intern("on_comment");
-    ripper_id_embdoc_beg = rb_intern("on_embdoc_beg");
-    ripper_id_embdoc = rb_intern("on_embdoc");
-    ripper_id_embdoc_end = rb_intern("on_embdoc_end");
-    ripper_id_sp = rb_intern("on_sp");
-    ripper_id_heredoc_beg = rb_intern("on_heredoc_beg");
-    ripper_id_heredoc_end = rb_intern("on_heredoc_end");
-    ripper_id___end__ = rb_intern("on___end__");
-    ripper_id_CHAR = rb_intern("on_CHAR");
+    ripper_id_ignored_nl = rb_intern_const("on_ignored_nl");
+    ripper_id_comment = rb_intern_const("on_comment");
+    ripper_id_embdoc_beg = rb_intern_const("on_embdoc_beg");
+    ripper_id_embdoc = rb_intern_const("on_embdoc");
+    ripper_id_embdoc_end = rb_intern_const("on_embdoc_end");
+    ripper_id_sp = rb_intern_const("on_sp");
+    ripper_id_heredoc_beg = rb_intern_const("on_heredoc_beg");
+    ripper_id_heredoc_end = rb_intern_const("on_heredoc_end");
+    ripper_id___end__ = rb_intern_const("on___end__");
+    ripper_id_CHAR = rb_intern_const("on_CHAR");
 
     ripper_init_eventids2_table(self);
 }

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