RFR 7118066: Warnings in java.util.concurrent package

[David Holmes]

On 8/12/2011 9:33 PM, Chris Hegarty wrote:
> On 08/12/2011 05:13, David Holmes wrote:
>> ....
>>
>> I also think the SynchronousQueue has no business being used in this
>> test. Though I confess I'm unclear what this test is for - is it for
>> emptyIterator() or for "iterators of empty collections" ?
>
> Thanks David, so the proposed change is to simple remove SynchronousQueue.

Well I think your change is also valid: the test is assuming uses of the 
emptyIterator() that simply are not spec'd to exist.

Maybe we still need something that tests SynchronousQueue to verify its 
iterator is indeed empty.

David
-----

> --- a/test/java/util/Collections/EmptyIterator.java Wed Dec 07 10:17:34
> 2011 +0000
> +++ b/test/java/util/Collections/EmptyIterator.java Thu Dec 08 11:28:16
> 2011 +0000
> @@ -35,8 +35,6 @@ public class EmptyIterator {
> void test(String[] args) throws Throwable {
> testEmptyCollection(Collections.<Object>emptyList());
> testEmptyCollection(Collections.<Object>emptySet());
> - testEmptyCollection(new java.util.concurrent.
> - SynchronousQueue<Object>());
>
> testEmptyMap(Collections.<Object, Object>emptyMap());
>
> Since it is trivial, I'll push it with the warning changes.
>
> -Chris.
>
>>
>> David
>> -----
>>