> On 2 Mar 2016, at 21:37, Martin Buchholz <[email protected]> wrote: > > Webrevs updated, incorporating changes to tests in my previous message.
Looks ok, but i went through rather quickly. java/util/concurrent/ScheduledThreadPoolExecutor/DelayOverflow.java — - pool.schedule(keepPoolBusy, 0, TimeUnit.SECONDS); + pool.schedule(keepPoolBusy, 0, DAYS); It probably does not matter that you changed the units here? Paul.
