[jira] Commented: (HARMONY-6236) [classlib][luni] ArrayList breaks when given concurrent collections as input

["Jesse Wilson (JIRA)" <ji...@apache.org>]

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

Jesse Wilson commented on HARMONY-6236:
---------------------------------------

Thanks Tim.

I'll put it on my todo list to run Google Collections' absurdly exhaustive test suite against Harmony's collections; perhaps that will expose similar problems.
  http://code.google.com/p/google-collections/source/browse/trunk/testfw/com/google/common/collect/testing/SetTestSuiteBuilder.java

> [classlib][luni] ArrayList breaks when given concurrent collections as input
> ----------------------------------------------------------------------------
>
>                 Key: HARMONY-6236
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6236
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M10
>         Environment: SVN Revision: 784898
>            Reporter: Jesse Wilson
>            Assignee: Tim Ellison
>             Fix For: 5.0M11
>
>         Attachments: ArrayList_on_concurrent_collections.patch
>
>   Original Estimate: 0.5h
>  Remaining Estimate: 0.5h
>
> new ArrayList() and ArrayList.addAll(Collection) assume the passed-in collection will not change between calls. 
> In the attached test case, a special collection simulates a background thread calling "remove" after the ArrayList has called size(), but before it has called "toArray()". This causes the collection to return a null-padded array, which leads to a corrupt ArrayList. The attached patch fixes the problem.

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