On 23/07/2014 14:53, Petr Pchelko wrote:
Hello, Alan.

I'm skimmed over the updated webrev, it mostly looks good except for getFlavorMap where it doesn't set map, I assume you meant to do this:

if (map == null)
    flavorMap = map = supplier.get();
Thank you! Updated the fix: http://cr.openjdk.java.net/~pchelko/9/8037485/webrev.03 <http://cr.openjdk.java.net/%7Epchelko/9/8037485/webrev.03/>
This part looks good to me now.

-Alan.

Reply via email to