[前][次][番号順一覧][スレッド一覧]

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/

[前][次][番号順一覧][スレッド一覧]