bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: large integer truncation in regex module


From: Gianluigi Tiesi
Subject: Re: large integer truncation in regex module
Date: Sun, 25 Mar 2012 13:41:09 +0200
User-agent: Mozilla/5.0 (Windows NT 5.2; WOW64; rv:11.0) Gecko/20120312 Thunderbird/11.0

On 25/03/2012 6.58, Paul Eggert wrote:
On 03/24/2012 09:28 PM, Gianluigi Tiesi wrote:
While compiling regex module one android I've discovered a problem
that on other 32bit compiler is only a warning

...
it's correct, but I think an ifdef may be used instead since
BITSET_WORD_BITS is a define


Generally speaking we prefer 'if (xxx)' to '#if xxx' where either will
do, because the former is easier to read and reason about. If the only
problem with 'if (xxx)' is a bogus warning by some random compiler then
it's probably better to leave it alone (and get the compiler fixed....).

I don't think assigning a 64bit integer constant in a 32bit variable type is a bogus warning, and some software using gnulib like parted enable Werror by default

--
Gianluigi Tiesi <address@hidden>
EDP Project Leader
Netfarm S.r.l. - http://www.netfarm.it/
Free Software: http://oss.netfarm.it/

Q: Because it reverses the logical flow of conversation.
A: Why is putting a reply at the top of the message frowned upon?



reply via email to

[Prev in Thread] Current Thread [Next in Thread]