2010/6/19 Bob Schellink
> On 19/06/2010 04:37, bckfnn@apache.org wrote:
>
> Seems SessionMap#entrySet should have been Map#values? Guess we can move that code under
#values
> instead of throwing exception.
Done.
> I think the nested Entry class will give issues inside Velocity since it's not public.
Can you confirm?
No, it is ok. HashMap.Entry is defined the same way. As long as the
object is only used as a Map.Entry, not as a SessionMap.Entry
instance, then all is fine. I have added a test for this.
regards,
Finn
|