Code Review Request: 7045655: An empty InMemoryCookieStore should not return true for removeAll

Kurchi Hazra kurchi.subhra.hazra at oracle.com
Wed Mar 14 15:36:33 PDT 2012


The CookieStore.removeAll() is supposed to return true according to its 
spec, only if the CookieStore changes as a
result of the call.

InMemoryCookieStore:removeAll() was returning true by default, even if 
the CookieStore object was already empty,
and no changes were being done by the call. This fix is to simply return 
false in case the CookieStore is empty.

Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7045655
Webrev: http://cr.openjdk.java.net/~khazra/7045655/webrev.01/


Thanks,
Kurchi




More information about the net-dev mailing list