ruby-dev

むとうです。

On Sat, 21 Jun 2003 10:07:48 +0900
nobu.nakada@nifty.ne.jp wrote:

> なかだです。
> 
> At Sat, 21 Jun 2003 09:45:49 +0900,
> Masao Mutoh wrote:
> > > > known problemです。sprintf(3)を使わずに(localeに影響されずに)
> > > > 浮動小数点数を文字列化する手段がなかなか手に入らないので。
> > > > 逆は自前のstrtod(3)を用意することで対応したのですが。
> > > 
> > > ふと、思いついたのですが、(思いついただけですが)
> > > localeconv(3) を使って decimal_point を得て、sprintf の結果を
> > > decimal_point から "." に書き換えてしまうとか。
> > 
> > なるほど。せっかくですので上記を実装してみました。添付します。
> 
> locale.hとlocaleconv()はconfigureでチェックすべきでしょうね。

すみません...。改めまして。

でも、私、configure.inの書き方がよくわかっていないので
これで良いのかはちょっと自信ないです。

それでは。
-- 
.:% Masao Mutoh<mutoh@highway.ne.jp>

Attachments (1)

marshal.patch2 (2.11 KB, text/x-diff)

Index: configure.in
===================================================================
RCS file: /src/ruby/configure.in,v
retrieving revision 1.175
diff -u -r1.175 configure.in
--- configure.in	9 Jun 2003 10:02:36 -0000	1.175
+++ configure.in	21 Jun 2003 01:42:57 -0000
@@ -353,7 +353,7 @@
 AC_CHECK_HEADERS(stdlib.h string.h unistd.h limits.h sys/file.h sys/ioctl.h\
 		 fcntl.h sys/fcntl.h sys/select.h sys/time.h sys/times.h sys/param.h\
 		 syscall.h pwd.h grp.h a.out.h utime.h memory.h direct.h sys/resource.h \
-		 sys/mkdev.h sys/utime.h float.h)
+		 sys/mkdev.h sys/utime.h float.h locale.h)
 
 dnl Checks for typedefs, structures, and compiler characteristics.
 AC_TYPE_UID_T
@@ -378,7 +378,7 @@
 	      setrgid setegid setregid setresgid pause lchown lchmod\
 	      getpgrp setpgrp getpgid setpgid getgroups setgroups getpriority getrlimit\
 	      dlopen sigprocmask sigaction _setjmp setsid telldir seekdir fchmod\
-	      mktime timegm cosh sinh tanh)
+	      mktime timegm cosh sinh tanh localeconv)
 AC_STRUCT_TIMEZONE
 AC_CACHE_CHECK(for struct tm.tm_gmtoff, rb_cv_member_struct_tm_tm_gmtoff,
   [AC_TRY_COMPILE([#include <time.h>],
Index: marshal.c
===================================================================
RCS file: /src/ruby/marshal.c,v
retrieving revision 1.91
diff -u -r1.91 marshal.c
--- marshal.c	6 Jun 2003 09:24:59 -0000	1.91
+++ marshal.c	21 Jun 2003 01:42:57 -0000
@@ -15,6 +15,10 @@
 #include "st.h"
 #include "util.h"
 
+#ifdef HAVE_LOCALE_H
+#include <locale.h>
+#endif
+
 #include <math.h>
 #ifdef HAVE_FLOAT_H
 #include <float.h>
@@ -292,10 +296,24 @@
     }
     else {
 	int len;
-
+#ifdef HAVE_LOCALECONV 
+        int i;
+        struct lconv *lc;
+#endif
 	/* xxx: should not use system's sprintf(3) */
 	sprintf(buf, "%.*g", FLOAT_DIG, d);
 	len = strlen(buf);
+#ifdef HAVE_LOCALECONV 
+        lc = localeconv();
+        if (lc->decimal_point[0] != '.'){
+          for (i=0; i<len; i++) {
+            if (buf[i] == lc->decimal_point[0]){
+              buf[i] = '.';
+              break;
+            }
+          }
+        }
+#endif
 	w_bytes(buf, len + save_mantissa(d, buf + len), arg);
 	return;
     }

Thread

Prev Next

In This Thread

Prev Next

[#20315] Re: [Oniguruma] intersection of char class — kkosako@...

[#20316] Re: local variable and local variable in block behave differently — nobu.nakada@...

[#20317] Re: [Oniguruma] possessive quantifier — kkosako@...

[#20318] make test failed — nobu.nakada@...

[#20320] Apollo.exeでsingleton_method_addedが効かない — Kazuhiro Yoshida <moriq@...>

[#20326] Re: Forward: RCR: Enhancing matrix.rb — keiju@... (石塚圭樹)

[#20328] cannot commit to cvs — "U.Nakamura" <usa@...>

[#20330] Re: [Oniguruma] intersection of char class — kkosako@...

[#20335] 1.8 gsub — "H.Suzuki" <hsuzux@...>

[#20339] rb_fix2int->rb_fix2uint? — akira yamada <akira@...>

[#20341] Re: irb warning(create Proc object without a block) (PR#492) — nobu.nakada@...

[#20343] Re: [Oniguruma] intersection of char class — kkosako@...

[#20347] 1.8.0 preview3 schedule — matz@... (Yukihiro Matsumoto)

[#20348] lib/optparse/date.rb — Tadayoshi Funaba <tadf@...5.so-net.ne.jp>

[#20350] How to create Proc from C extension — nobu.nakada@...

[#20351] socket.writeでBUG — "so" <dan@...2.so-net.ne.jp>

[#20361] [PATCH] lambda + def — Minero Aoki <aamine@...>

[#20362] [Oniguruma] quoting substring — kkosako@...

[#20366] forwadable.rb on ftp.ruby-lang.org — keiju@... (Keiju ISHITSUKA)

[#20370] [PATCH] 1.8 range.c: [][0..-1] => [] — "NAKAMURA, Hiroshi" <nakahiro@...>

[#20374] case sensitivity of the names of environment variables — "U.Nakamura" <usa@...>

[#20379] locale and Marshal — Masao Mutoh <mutoh@...>

[#20390] Re: [Oniguruma] quoting substring — kkosako@...

[#20392] [BigDecimal] proposal to change specification — "Tadashi Saito" <shiba@...2.accsnet.ne.jp>

[#20393] [Oniguruma] Version 1.9.0 — kkosako@...

[#20412] Re: In 1.8.0 nil.to_s is not the same as "" — nobu.nakada@...

[#20416] ruby 1.8.0 preview3 — matz@... (Yukihiro Matsumoto)

[#20434] [PATCH] config.guess returns invalid value on Linux/Alpha — Minero Aoki <aamine@...>

[#20441] [New Block] 悪いのは誰？ — Shin-ichiro HARA <sinara@...>

[#20445] Re: *.soの autoload(PR#504) — nobu.nakada@...

[#20447] [BigDecimal] renaming proposal — "Tadashi Saito" <shiba@...2.accsnet.ne.jp>

[#20462] 原因不明のSEGV — "yamamoto madoka" <dan@...2.so-net.ne.jp>

[#20466] Ruby OpenSSL and MS_CALLBACK cpp(1) macro. — Takahiro Kambe <taca@...>

[ruby-dev:20403] Re: locale and Marshal

Attachments (1)

Thread

In This Thread

[#20362] [Oniguruma]　quoting substring — kkosako@...

[#20390] Re: [Oniguruma]　quoting substring — kkosako@...