Bug #3522: String::size return invalid size on mswin64

• Status: Closed
• Assignee: mame (Yusuke Endoh)
• Target version: 1.9.2
• ruby -v: ruby 1.9.2dev (2010-07-01 revision 28513) [x64-mswin64_90]

[beuniv (shintaro kuwamoto)]

=begin
x64-mswin64_90環境で、15文字以上のUTF-8文字列のサイズが正常に取得できません。

Z:\ruby-trunk\build>ruby -e"puts "1234567890123\u{3042}".size"
14

Z:\ruby-trunk\build>ruby -e"puts "12345678901234\u{3042}".size"
19
=end

[mame (Yusuke Endoh)]

=begin
遠藤です。

2010年7月2日19:05 shintaro kuwamoto [e-mail]:

x64-mswin64_90環境で、15文字以上のUTF-8文字列のサイズが正常に取得できません。

Z:\ruby-trunk\build>ruby -e"puts "1234567890123\u{3042}".size"
14

Z:\ruby-trunk\build>ruby -e"puts "12345678901234\u{3042}".size"
19

ひょっとして以下のパッチで直らないでしょうか。

diff --git a/string.c b/string.c
index 703797f..510c04e 100644
--- a/string.c
+++ b/string.c
@@ -141,7 +141,7 @@ static inline const char *
search_nonascii(const char *p, const char *e)
{
#if SIZEOF_VALUE == 8
-# define NONASCII_MASK 0x8080808080808080LL
+# define NONASCII_MASK 0x8080808080808080ULL
#elif SIZEOF_VALUE == 4

define NONASCII_MASK 0x80808080UL

#endif

--
Yusuke Endoh [e-mail]

=end

[beuniv (shintaro kuwamoto)]

=begin
こんばんわ。

ひょっとして以下のパッチで直らないでしょうか。

diff --git a/string.c b/string.c
index 703797f..510c04e 100644
--- a/string.c
+++ b/string.c
@@ -141,7 +141,7 @@ static inline const char *
search_nonascii(const char *p, const char *e)
{
#if SIZEOF_VALUE == 8
-# define NONASCII_MASK 0x8080808080808080LL
+# define NONASCII_MASK 0x8080808080808080ULL
#elif SIZEOF_VALUE == 4

define NONASCII_MASK 0x80808080UL

#endif

おお。動くようになりました。
ありがとうございます。

Z:\ruby-trunk\build>ruby -e"puts "1234567890123\u{3042}".size"
14

Z:\ruby-trunk\build>ruby -e"puts "12345678901234\u{3042}".size"
15

--
kuwamoto

=end

[mame (Yusuke Endoh)]

=begin
遠藤です。

2010年7月5日23:42 kuwamoto shintaro [e-mail]:

おお。動くようになりました。
ありがとうございます。

おお。確認ありがとうございます。コミットしておきます。

しかし、0x8080808080808080LL を signed long long int にしてしまうのは

VC のバグのような気もする

--
Yusuke Endoh [e-mail]

=end

[mame (Yusuke Endoh)]

• Status changed from Open to Closed
• % Done changed from 0 to 100

=begin
This issue was solved with changeset r28544.
shintaro, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

=end

[nobu (Nobuyoshi Nakada)]

• Status changed from Closed to Assigned
• Assignee set to yugui (Yuki Sonoda)

=begin

=end

[nobu (Nobuyoshi Nakada)]

• Status changed from Assigned to Closed
• Assignee changed from yugui (Yuki Sonoda) to mame (Yusuke Endoh)

=begin

=end