[jboss-dev] SoftValueHashMap behaves diff then WeakValueHashMap

[Dimitris Andreadis]

BTW, this is done and trunk moved to this latest 2.2.9.GA

Dimitris Andreadis wrote:
> Ok, let me see what other tasks remain for 2.2.9 and I'll release.
> 
> Ales Justin wrote:
>> I'm done with changes.
>> Can you please cut new release, since I need it in VFS.
>> And all other SoftValueHashMap users will appreciate it as well. ;-)
>>
>> Dimitris Andreadis wrote:
>>> Were is this needed/used?
>>>
>>> Ales Justin wrote:
>>>> jboss-commons-core issue:
>>>>
>>>> SoftValueHashMap::values doesn't return expected value type:
>>>>
>>>> e.g.
>>>>    public void testBasicOperations() throws Exception
>>>>    {
>>>>       Map map = createMap();
>>>>       map.put("date1", new Date());
>>>>
>>>>       Iterable<String> keys = map.keySet();
>>>>       assertIterable(keys, String.class);
>>>>       Iterable<Date> values = map.values();
>>>>       assertIterable(values, Date.class);
>>>>    }
>>>>
>>>>    protected <T> void assertIterable(Iterable<T> iter, Class<T> clazz)
>>>>    {
>>>>       assertTrue(iter.iterator().hasNext());
>>>>       T next = iter.iterator().next();
>>>>       assertTrue("Next " + next + " is not instance of " + 
>>>> clazz.getName(), clazz.isInstance(next));
>>>>       assertNotNull(next);
>>>>    }
>>>>
>>>> ---
>>>>
>>>> junit.framework.AssertionFailedError: Next 
>>>> org.jboss.util.collection.SoftValueHashMap$SoftValueRef at 157f0dc is 
>>>> not instance of java.util.Date
>>>>     at 
>>>> org.jboss.test.util.test.collection.AbstractMapUnitTest.assertIterable(AbstractMapUnitTest.java:33) 
>>>>
>>>>     at 
>>>> org.jboss.test.util.test.collection.AbstractMapUnitTest.testBasicOperations(AbstractMapUnitTest.java:26) 
>>>>
>>>>
>>>> ---
>>>>
>>>> Where WeakValueHashMap works OK.
>>>>
>>>> I'll do the fix, Dimitris|Scott can you do a new release then?
>>>> _______________________________________________
>>>> jboss-development mailing list
>>>> jboss-development at lists.jboss.org
>>>> https://lists.jboss.org/mailman/listinfo/jboss-development
>>> _______________________________________________
>>> jboss-development mailing list
>>> jboss-development at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/jboss-development
>>>
> _______________________________________________
> jboss-development mailing list
> jboss-development at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-development