[PATCH][libm] Replace global constants with preprocessor defines

Andreas Jaeger aj@suse.com
Fri Dec 28 03:52:00 GMT 2012


On 12/27/2012 12:49 PM, Siddhesh Poyarekar wrote:
> On Thu, Dec 27, 2012 at 10:01:38AM +0100, Andreas Jaeger wrote:
>> Previously these had some comments (but were in hex, so this was
>> really needed). For some of them it is obvious but I suggest to stay
>> with the comments.
>>
>> Let's say explictily "1/2" for HALF and 2^24 for RADIX etc.
>>
>> Similar above for TWO5 etc. Either add a comment before that the
>> following are TWOX with 2^X or add it for each comment...
>
> Thanks, here's v2 of the patch.

Thanks, this is fine,

Andreas
-- 
  Andreas Jaeger aj@{suse.com,opensuse.org} Twitter/Identica: jaegerandi
   SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
    GF: Jeff Hawn,Jennifer Guild,Felix Imendörffer,HRB16746 (AG Nürnberg)
     GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126



More information about the Libc-alpha mailing list