[PATCHv2] Refactor tst-strtod-round.c for type-generic-ness

Paul E. Murphy murphyp@linux.vnet.ibm.com
Wed May 25 19:30:00 GMT 2016


On 05/25/2016 12:21 PM, Joseph Myers wrote:
> On Wed, 25 May 2016, Paul E. Murphy wrote:
> 
>> +/* Define the long double choose (CHOOSE_ld) macro
>> +   to select the appropriate generated long double
>> +   value from the generated test data.  */
>>  #if LDBL_MANT_DIG == 53 && LDBL_MAX_EXP == 1024
>> -# define TEST(s, fexact, fd, fn, fz, fu, dexact, dd, dn, dz, du,	\
>> -	      ld53exact, ld53d, ld53n, ld53z, ld53u,			\
>> -	      ld64iexact, ld64id, ld64in, ld64iz, ld64iu,		\
>> -	      ld64mexact, ld64md, ld64mn, ld64mz, ld64mu,		\
>> -	      ld106exact, ld106d, ld106n, ld106z, ld106u,		\
>> -	      ld113exact, ld113d, ld113n, ld113z, ld113u)		\
>> -  {									\
>> -    s,									\
>> -    { fexact, dexact, ld53exact },					\
>> -    { fd, dd, ld53d },							\
>> -    { fn, dn, ld53n },							\
>> -    { fz, dz, ld53z },							\
>> -    { fu, du, ld53u }							\
>> -  }
>> +/* This is for the long double == double format.  */
>> +#define CHOOSE_ld(f,d,...) d
> 
> Preprocessor directives inside preprocessor conditionals should have 
> appropriate indentation after the '#', so "# define CHOOSE_ld" for all the 
> CHOOSE_ld definitions.
> 
> OK with that fixed.

Spaces added, committed as 1ced34c.



More information about the Libc-alpha mailing list