[foreign] RFR : Fixes to StdLibTest

Sundararajan Athijegannathan sundararajan.athijegannathan at oracle.com
Mon Sep 24 16:43:23 UTC 2018


Thanks for this patch Jorn!

I filed a bug to track this:  
https://bugs.openjdk.java.net/browse/JDK-8211063

PS. I'm testing your patch on Mac.

Thanks,
-Sundar

On 24/09/18, 9:28 PM, Jorn Vernee wrote:
> Hello,
>
> Please review this patch which does a few fixes to StdLibTest:
>
> Diff: https://gist.github.com/JornVernee/312ecb03d93882efac1f86ccc8ad1f25
>
> This patch fixes 3 things:
>
> 1.) 'puts' returns a non-negative value on success [1], but the test 
> was testing > 0. I changed it to >= 0, since the MSVC implementation 
> returns exactly 0.
>
> 2.) Switched the test_time test from testing localtime(), which is 
> locale dependent to testing gmtime() which always uses the GMT 
> timezone, since the locale dependency was giving trouble in cygwin [2].
>
> 3.) Also for the test_time test, I changed the way the test parameters 
> are being generated, since it was apparently generating invalid values 
> for windows, so the native function was returning null, and the test 
> was failing.
>
> Since I'm not a committer someone else will have to create a commit 
> and push this.
>
> Thanks,
> Jorn
>
> [1] : https://en.cppreference.com/w/c/io/puts
> [2] : 
> http://mail.openjdk.java.net/pipermail/panama-dev/2018-September/002753.html


More information about the panama-dev mailing list