knu	2008-05-26 01:06:00 +0900 (Mon, 26 May 2008)

  New Revision: 16591

  Modified files:
    trunk/ChangeLog
    trunk/hash.c

  Log:
    * hash.c (env_each_key, env_each_value, env_reject_bang)
      (rb_env_clear, env_replace): Omit duplicated secure level check.


  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/trunk/hash.c?r1=16591&r2=16590&diff_format=u
  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/trunk/ChangeLog?r1=16591&r2=16590&diff_format=u

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 16590)
+++ ChangeLog	(revision 16591)
@@ -1,3 +1,8 @@
+Mon May 26 00:52:52 2008  Akinori MUSHA  <knu@i...>
+
+	* hash.c (env_each_key, env_each_value, env_reject_bang)
+	  (rb_env_clear, env_replace): Omit duplicated secure level check.
+
 Mon May 26 00:37:16 2008  Akinori MUSHA  <knu@i...>
 
 	* hash.c (env_each_value): Do not call env_values() twice.
Index: hash.c
===================================================================
--- hash.c	(revision 16590)
+++ hash.c	(revision 16591)
@@ -2086,8 +2086,7 @@
     long i;
 
     RETURN_ENUMERATOR(ehash, 0, 0);
-    rb_secure(4);
-    keys = env_keys();
+    keys = env_keys();	/* rb_secure(4); */
     for (i=0; i<RARRAY_LEN(keys); i++) {
 	rb_yield(RARRAY_PTR(keys)[i]);
     }
@@ -2121,8 +2120,7 @@
     long i;
 
     RETURN_ENUMERATOR(ehash, 0, 0);
-    rb_secure(4);
-    values = env_values();
+    values = env_values();	/* rb_secure(4); */
     for (i=0; i<RARRAY_LEN(values); i++) {
 	rb_yield(RARRAY_PTR(values)[i]);
     }
@@ -2165,8 +2163,7 @@
     int del = 0;
 
     RETURN_ENUMERATOR(ehash, 0, 0);
-    rb_secure(4);
-    keys = env_keys();
+    keys = env_keys();	/* rb_secure(4); */
     for (i=0; i<RARRAY_LEN(keys); i++) {
 	VALUE val = rb_f_getenv(Qnil, RARRAY_PTR(keys)[i]);
 	if (!NIL_P(val)) {
@@ -2235,8 +2232,7 @@
     volatile VALUE keys;
     long i;
 
-    rb_secure(4);
-    keys = env_keys();
+    keys = env_keys();	/* rb_secure(4); */
     for (i=0; i<RARRAY_LEN(keys); i++) {
 	VALUE val = rb_f_getenv(Qnil, RARRAY_PTR(keys)[i]);
 	if (!NIL_P(val)) {
@@ -2518,8 +2514,7 @@
     volatile VALUE keys;
     long i;
 
-    rb_secure(4);
-    keys = env_keys();
+    keys = env_keys();	/* rb_secure(4); */
     if (env == hash) return env;
     hash = to_hash(hash);
     rb_hash_foreach(hash, env_replace_i, keys);

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