Hi Pavel; The change and test looks good. Will the test be upstreamed or will Doug be adding a similar test in his upstream?
Mike On May 14 2014, at 08:29 , Pavel Rappo <pavel.ra...@oracle.com> wrote: > Hi everyone, > > could you please review my change for JDK-7153400? > > http://cr.openjdk.java.net/~chegar/7153400/00/webrev/ > http://ccc.us.oracle.com/7153400 > > It's a long expected fix for a minor issue in the ThreadPoolExecutor. This > has been agreed with Doug Lea. The exact same change (except for the test) is > already in jsr166 repo: > http://gee.cs.oswego.edu/cgi-bin/viewcvs.cgi/jsr166/src/main/java/util/concurrent/ThreadPoolExecutor.java?r1=1.151&r2=1.152 > > Thanks > -Pavel