ruby-changes:6333
From: kazu <ko1@a...>
Date: Thu, 3 Jul 2008 19:46:17 +0900 (JST)
Subject: [ruby-changes:6333] Ruby:r17849 (ruby_1_8): * ruby.c: Mac OS X needs origargc times of '\0' in origargv.
kazu 2008-07-03 19:46:06 +0900 (Thu, 03 Jul 2008)
New Revision: 17849
Modified files:
branches/ruby_1_8/ChangeLog
branches/ruby_1_8/ruby.c
Log:
* ruby.c: Mac OS X needs origargc times of '\0' in origargv. [ruby-dev:35308]
http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=17849
Index: ruby_1_8/ChangeLog
===================================================================
--- ruby_1_8/ChangeLog (revision 17848)
+++ ruby_1_8/ChangeLog (revision 17849)
@@ -1,3 +1,8 @@
+Thu Jul 3 19:35:45 2008 Kazuhiro NISHIYAMA <zn@m...>
+
+ * ruby.c: Mac OS X needs origargc times of '\0' in
+ origargv. [ruby-dev:35308]
+
Thu Jul 3 19:23:20 2008 Masaki Suketa <masaki.suketa@n...>
* ext/win32ole/win32ole.c: avoid creating Ruby object during
Index: ruby_1_8/ruby.c
===================================================================
--- ruby_1_8/ruby.c (revision 17848)
+++ ruby_1_8/ruby.c (revision 17849)
@@ -1039,6 +1039,39 @@
#define set_arg0space() ((void)0)
#endif
+static int
+get_arglen(int argc, char **argv)
+{
+ char *s = argv[0];
+ int i;
+
+ if (!argc) return 0;
+ s += strlen(s);
+ /* See if all the arguments are contiguous in memory */
+ for (i = 1; i < argc; i++) {
+ if (argv[i] == s + 1) {
+ s++;
+ s += strlen(s); /* this one is ok too */
+ }
+ else {
+ break;
+ }
+ }
+#if defined(USE_ENVSPACE_FOR_ARG0)
+ if (environ && (s == environ[0])) {
+ s += strlen(s);
+ for (i = 1; environ[i]; i++) {
+ if (environ[i] == s + 1) {
+ s++;
+ s += strlen(s); /* this one is ok too */
+ }
+ }
+ ruby_setenv("", NULL); /* duplicate environ vars */
+ }
+#endif
+ return s - argv[0];
+}
+
static void
set_arg0(val, id)
VALUE val;
@@ -1047,8 +1080,9 @@
VALUE progname;
char *s;
long i;
+ int j;
#if !defined(PSTAT_SETCMD) && !defined(HAVE_SETPROCTITLE)
- static int len;
+ static int len = 0;
#endif
if (origargv == 0) rb_raise(rb_eRuntimeError, "$0 not initialized");
@@ -1075,27 +1109,7 @@
progname = rb_tainted_str_new(s, i);
#else
if (len == 0) {
- char *s = origargv[0];
- int i;
-
- s += strlen(s);
- /* See if all the arguments are contiguous in memory */
- for (i = 1; i < origargc; i++) {
- if (origargv[i] == s + 1) {
- s++;
- s += strlen(s); /* this one is ok too */
- }
- else {
- break;
- }
- }
-#if defined(USE_ENVSPACE_FOR_ARG0)
- if (s + 1 == envspace.begin) {
- s = envspace.end;
- ruby_setenv("", NULL); /* duplicate environ vars */
- }
-#endif
- len = s - origargv[0];
+ len = get_arglen(origargc, origargv);
}
if (i >= len) {
@@ -1105,8 +1119,10 @@
s = origargv[0] + i;
*s = '\0';
if (++i < len) memset(s + 1, ' ', len - i);
- for (i = 1; i < origargc; i++)
- origargv[i] = s;
+ for (i = len-1, j = origargc-1; j > 0 && i >= 0; --i, --j) {
+ origargv[j] = origargv[0] + i;
+ *origargv[j] = '\0';
+ }
progname = rb_tainted_str_new2(origargv[0]);
#endif
rb_progname = rb_obj_freeze(progname);
--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/