Re: [Tinycc-devel] Windows test suite, why 24_math_library is removed?

[Christian Jullien]

As no one complained, I changed 28_strings.c to use ISO C functions and 
reintroduced 24 and 28 tests in Makefile for Windows.

ð  http://repo.or.cz/tinycc.git/commit/8986bc8af473080bed0efa01cb569f3e25f179a9

 

Christian

 

From: Tinycc-devel [mailto:tinycc-devel-bounces+eligis=orange...@nongnu.org] On 
Behalf Of avih
Sent: dimanche 9 octobre 2016 16:35
To: tinycc-devel@nongnu.org
Subject: Re: [Tinycc-devel] Windows test suite, why 24_math_library is removed?

 

round and several other floating point functions from math.h were not working 
on windows for some years, and around November 2015 I fixed most of those 
issues. However, I wasn't aware that some tests were disabled on windows due to 
the missing functions. Good catch (I'd leave it to you to re-enable them).

 

On Sunday, October 9, 2016 5:20 PM, Christian Jullien <eli...@orange.fr> wrote:

 

Hi again,

 

I see from test/test2/Makefile that 24_math_library is removed for Windows 
because of lack of round().

 

ifdef CONFIG_WIN32

SKIP += 24_math_library.test # don't have round()

 

With mob compiled with a recent MinGW gcc compiler.

This test is fully working. Can you reconsider to add it as part of standard 
tests suite.

 

I also note that 28_strings is removed because Windows lacks index/rindex 
(which is true).

Now, if you remove  which was a BSDish include now removed from 
POSIX and replace

   printf("%s\n", index(a, 'o'));

   printf("%s\n", rindex(a, 'l'));

   printf("%d\n", rindex(a, 'x') == NULL);

 

By iso C equivalent functions:

 

   printf("%s\n", strchr(a, 'o'));

   printf("%s\n", strrchr(a, 'l'));

   printf("%d\n", strrchr(a, 'x') == NULL);

 

It works on Windows and should work on any system.


___
Tinycc-devel mailing list
Tinycc-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/tinycc-devel



___
Tinycc-devel mailing list
Tinycc-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/tinycc-devel

Re: [Tinycc-devel] Windows test suite, why 24_math_library is removed?

[avih]

round and several other floating point functions from math.h were not working 
on windows for some years, and around November 2015 I fixed most of those 
issues. However, I wasn't aware that some tests were disabled on windows due to 
the missing functions. Good catch (I'd leave it to you to re-enable them).
 

On Sunday, October 9, 2016 5:20 PM, Christian Jullien  
wrote:
 

 Hi 
again,  I see from test/test2/Makefile that 24_math_library is removed for 
Windows because of lack of round().  ifdef CONFIG_WIN32 SKIP += 
24_math_library.test # don't have round()  With mob compiled with a recent 
MinGW gcc compiler.This test is fully working. Can you reconsider to add it as 
part of standard tests suite.  I also note that 28_strings is removed because 
Windows lacks index/rindex (which is true).Now, if you remove  which 
was a BSDish include now removed from POSIX and replace   printf("%s\n", 
index(a, 'o'));   printf("%s\n", rindex(a, 'l'));   printf("%d\n", rindex(a, 
'x') == NULL);  By iso C equivalent functions:     printf("%s\n", strchr(a, 
'o'));   printf("%s\n", strrchr(a, 'l'));   printf("%d\n", strrchr(a, 'x') == 
NULL);  It works on Windows and should work on any system.
___
Tinycc-devel mailing list
Tinycc-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/tinycc-devel


   ___
Tinycc-devel mailing list
Tinycc-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/tinycc-devel

[Tinycc-devel] Windows test suite, why 24_math_library is removed?

[Christian Jullien]

Hi again,

 

I see from test/test2/Makefile that 24_math_library is removed for Windows
because of lack of round().

 

ifdef CONFIG_WIN32

SKIP += 24_math_library.test # don't have round()

 

With mob compiled with a recent MinGW gcc compiler.

This test is fully working. Can you reconsider to add it as part of standard
tests suite.

 

I also note that 28_strings is removed because Windows lacks index/rindex
(which is true).

Now, if you remove  which was a BSDish include now removed from
POSIX and replace

   printf("%s\n", index(a, 'o'));

   printf("%s\n", rindex(a, 'l'));

   printf("%d\n", rindex(a, 'x') == NULL);

 

By iso C equivalent functions:

 

   printf("%s\n", strchr(a, 'o'));

   printf("%s\n", strrchr(a, 'l'));

   printf("%d\n", strrchr(a, 'x') == NULL);

 

It works on Windows and should work on any system.

___
Tinycc-devel mailing list
Tinycc-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/tinycc-devel