From 98685cc70fd4ea86bccc81983283c29eb7b2afd7 Mon Sep 17 00:00:00 2001 From: mrkn Date: Sun, 1 Aug 2010 13:20:22 +0000 Subject: [PATCH] * ext/bigdecimal/bigdecimal.c (BigDecimal_hash): use rb_memhash and take care of negative finite numbers properly. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@28825 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 5 +++++ ext/bigdecimal/bigdecimal.c | 9 +++------ 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index 1d78571868..133690733a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Sun Aug 1 22:16:30 2010 Kenta Murata + + * ext/bigdecimal/bigdecimal.c (BigDecimal_hash): use rb_memhash and + take care of negative finite numbers properly. + Sun Aug 1 20:57:22 2010 Tanaka Akira * ext/pathname/pathname.c (path_realpath): Pathname#realpath translated diff --git a/ext/bigdecimal/bigdecimal.c b/ext/bigdecimal/bigdecimal.c index 146145ce41..239fe05a76 100644 --- a/ext/bigdecimal/bigdecimal.c +++ b/ext/bigdecimal/bigdecimal.c @@ -228,12 +228,9 @@ BigDecimal_hash(VALUE self) GUARD_OBJ(p,GetVpValue(self,1)); hash = (U_LONG)p->sign; /* hash!=2: the case for 0(1),NaN(0) or +-Infinity(3) is sign itself */ - if(hash==2) { - for(i = 0; i < p->Prec;i++) { - hash = 31 * hash + p->frac[i]; - hash ^= p->frac[i]; - } - hash += p->exponent; + if(hash == 2 || hash == -2) { + hash ^= rb_memhash(p->frac, sizeof(U_LONG)*p->Prec); + hash += p->exponent; } return INT2FIX(hash); }