You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Vladimir Ivanov (JIRA)" <ji...@apache.org> on 2007/01/15 08:09:27 UTC
[jira] Commented: (HARMONY-917) [classlib][drlvm] no
IllegalArgumentException for java.lang.Thread.join(long,int) when
parameters are incorrect
[ https://issues.apache.org/jira/browse/HARMONY-917?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12464668 ]
Vladimir Ivanov commented on HARMONY-917:
-----------------------------------------
verified, thanks
> [classlib][drlvm] no IllegalArgumentException for java.lang.Thread.join(long,int) when parameters are incorrect
> ---------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-917
> URL: https://issues.apache.org/jira/browse/HARMONY-917
> Project: Harmony
> Issue Type: Bug
> Components: Classlib
> Reporter: Vladimir Ivanov
> Assigned To: Alexey Petrenko
> Attachments: harmony-917.patch, ThreadRTest.patch
>
>
> The Harmony method java.lang.Thread.join(long,int) does not throw IllegalArgumentException for incorrect parameters while spec requires it. The j2se spec says: throws IllegalArgumentException - if the value of millis is negative the value of nanos is not in the range 0-999999.
> ==================== test.java =========================
> public class test {
> public static void main(String[] args) {
> Thread th = new Thread();
> long mls = 688204075024689866L;
> int nn = -10000;
> try {
> th.join(mls, nn);
> System.out.println("Test failed");
> } catch (IllegalArgumentException e) {
> System.out.println("1. Test passed: "+ e);
> } catch (Throwable e) {
> System.out.println("Test failed: unexpected error");
> e.printStackTrace();
> }
> mls = -1000000000000L;
> nn = 90000;
> try {
> th.join(mls, nn);
> System.out.println("Test failed");
> } catch (IllegalArgumentException e) {
> System.out.println("2. Test passed: "+ e);
> } catch (Throwable e) {
> System.out.println("Test failed: unexpected error");
> e.printStackTrace();
> }
> mls = 10000000000000L;
> nn = 1000001;
> try {
> th.join(mls, nn);
> System.out.println("Test failed");
> } catch (IllegalArgumentException e) {
> System.out.println("3. Test passed: "+ e);
> } catch (Throwable e) {
> System.out.println("Test failed: unexpected error");
> e.printStackTrace();
> }
> }
> }
> ====================================================
> Output:
> C:\tmp\tmp17>C:\jrockit-jdk1.5.0-windows-ia32\bin\java.exe -cp . -showversion test
> java version "1.5.0"
> Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0-b64)
> BEA WebLogic JRockit(R) (build dra-38972-20041208-2001-win-ia32, R25.0.0-75, GC: System optimized over throughput (initial strategy singleparpar))
> 1. Test passed: java.lang.IllegalArgumentException: nanosecond timeout value out of range
> 2. Test passed: java.lang.IllegalArgumentException: timeout value is negative
> 3. Test passed: java.lang.IllegalArgumentException: nanosecond timeout value out of range
> C:\tmp\tmp17>C:\harmony\trunk_0427\deploy\jdk\jre\bin\java.exe -cp . -showversion test
> java version 1.5 (subset)
> (c) Copyright 1991, 2006 The Apache Software Foundation or its licensors, as applicable.
> 1. Test passed: java.lang.IllegalArgumentException
> 2. Test passed: java.lang.IllegalArgumentException
> 3. Test passed: java.lang.IllegalArgumentException
> C:\tmp\tmp17>C:\harmony\drlvm\trunk\build\win_ia32_msvc_debug\deploy\jre\bin\ij -cp . -showversion test
> Apache Harmony DRLVM HEAD-4807 (2006-05-02), Windows/ia32/msvc 1310, debug
> Test failed
> Test failed
> Test failed
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira