kosaki	2013-02-21 13:41:39 +0900 (Thu, 21 Feb 2013)

  New Revision: 39352

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

  Log:
    * file.c (RB_MAX_GROUPS): moved to
    * internal.h (RB_MAX_GROUPS): here.
    
    * file.c (rb_group_member): use RB_MAX_GROUPS instead of
      RUBY_GROUP_MAX. They are the same.

  Modified files:
    trunk/ChangeLog
    trunk/file.c
    trunk/internal.h
    trunk/process.c

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 39351)
+++ ChangeLog	(revision 39352)
@@ -1,3 +1,11 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1
+Thu Feb 21 13:37:07 2013  KOSAKI Motohiro  <kosaki.motohiro@g...>
+
+	* file.c (RB_MAX_GROUPS): moved to
+	* internal.h (RB_MAX_GROUPS): here.
+
+	* file.c (rb_group_member): use RB_MAX_GROUPS instead of
+	  RUBY_GROUP_MAX. They are the same.
+
 Thu Feb 21 13:15:40 2013  KOSAKI Motohiro  <kosaki.motohiro@g...>
 
 	* file.c (access_internal): removed.
Index: process.c
===================================================================
--- process.c	(revision 39351)
+++ process.c	(revision 39352)
@@ -5432,7 +5432,6 @@ proc_setgid(VALUE obj, VALUE id) https://github.com/ruby/ruby/blob/trunk/process.c#L5432
  * HP-UX			   20
  * Windows			 1015
  */
-#define RB_MAX_GROUPS (65536)
 static int _maxgroups = -1;
 static int
 get_sc_ngroups_max(void)
Index: internal.h
===================================================================
--- internal.h	(revision 39351)
+++ internal.h	(revision 39352)
@@ -191,6 +191,7 @@ VALUE rb_proc_location(VALUE self); https://github.com/ruby/ruby/blob/trunk/internal.h#L191
 st_index_t rb_hash_proc(st_index_t hash, VALUE proc);
 
 /* process.c */
+#define RB_MAX_GROUPS (65536)
 
 struct rb_execarg {
     int use_shell;
Index: file.c
===================================================================
--- file.c	(revision 39351)
+++ file.c	(revision 39352)
@@ -1013,9 +1013,6 @@ rb_file_lstat(VALUE obj) https://github.com/ruby/ruby/blob/trunk/file.c#L1013
 #endif
 }
 
-/* Linux allow 65536 groups and it is maximum value as far as we know. */
-#define RUBY_GROUP_MAX 65536
-
 static int
 rb_group_member(GETGROUPS_T gid)
 {
@@ -1036,7 +1033,7 @@ rb_group_member(GETGROUPS_T gid) https://github.com/ruby/ruby/blob/trunk/file.c#L1033
      * accept more larger value.
      * So we don't trunk NGROUPS anymore.
      */
-    while (groups <= RUBY_GROUP_MAX) {
+    while (groups <= RB_MAX_GROUPS) {
 	gary = ALLOCV_N(GETGROUPS_T, v, groups);
 	anum = getgroups(groups, gary);
 	if (anum != groups)

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