ruby-changes:29559
From: nobu <ko1@a...>
Date: Tue, 25 Jun 2013 11:39:48 +0900 (JST)
Subject: [ruby-changes:29559] nobu:r41611 (trunk): bignum.c: redundant expression
nobu 2013-06-25 11:39:37 +0900 (Tue, 25 Jun 2013) New Revision: 41611 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=41611 Log: bignum.c: redundant expression * bignum.c (LSHIFTX): remove redundant never-true expression. Modified files: trunk/ChangeLog trunk/bignum.c Index: ChangeLog =================================================================== --- ChangeLog (revision 41610) +++ ChangeLog (revision 41611) @@ -1,3 +1,7 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1 +Tue Jun 25 11:39:34 2013 Nobuyoshi Nakada <nobu@r...> + + * bignum.c (LSHIFTX): remove redundant never-true expression. + Tue Jun 25 00:55:54 2013 Masaya Tarui <tarui@r...> * gc.c (typedef struct rb_objspace): Change members for monitor objects. Index: bignum.c =================================================================== --- bignum.c (revision 41610) +++ bignum.c (revision 41611) @@ -39,7 +39,7 @@ static VALUE big_three = Qnil; https://github.com/ruby/ruby/blob/trunk/bignum.c#L39 # define HOST_BIGENDIAN_P 0 #endif #define ALIGNOF(type) ((int)offsetof(struct { char f1; type f2; }, f2)) -#define LSHIFTX(d, n) (sizeof(d) * CHAR_BIT <= (n) ? 0 : ((d) << (sizeof(d) * CHAR_BIT <= (n) ? 0 : (n)))) +#define LSHIFTX(d, n) (sizeof(d) * CHAR_BIT <= (n) ? 0 : ((d) << (n))) #define CLEAR_LOWBITS(d, numbits) ((d) & LSHIFTX(~((d)*0), (numbits))) #define FILL_LOWBITS(d, numbits) ((d) | (LSHIFTX(((d)*0+1), (numbits))-1)) -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/