You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Nathan Beyer (JIRA)" <ji...@apache.org> on 2009/03/20 04:41:50 UTC

[jira] Commented: (HARMONY-6122) [classlib][luni] java.util.Collections.swap(List list, int i, int j) should throw IndexOutOfBoundsException when i equals to j and is also out of boundary

    [ https://issues.apache.org/jira/browse/HARMONY-6122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12683739#action_12683739 ] 

Nathan Beyer commented on HARMONY-6122:
---------------------------------------

The change isn't obvious to me, can you help be understand it? Please describe what's being tested to determine if the code can return without any change or exception.

> [classlib][luni] java.util.Collections.swap(List<?> list, int i, int j) should throw IndexOutOfBoundsException when i equals to j and is also out of boundary
> -------------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-6122
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6122
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M8
>            Reporter: Kevin Zhou
>             Fix For: 5.0M9
>
>         Attachments: HARMONY-6122.diff
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Given a test case [1], RI passes while HY fails.
> The java spec specifies that java.util.Collections.wap(List<?> list, int i, int j) should throw IndexOutOfBoundsException when either i or j is out of range.
> [1] Test Case:
> public void test_Collections_swap_IndexOutOfBoundsException() {
> try { 
> Collections.swap(new ArrayList(), 3, 3); 
> fail("should throw IndexOutOfBoundsException");
>  } catch (IndexOutOfBoundsException e) {
>  // Expected 
> }
> }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.