Alan's suggestion of using the singular is a small improvement. I'm committing
* <p>A timer task is <em>not</em> reusable. Once a task has been scheduled * for execution on a {@code Timer} or cancelled, subsequent attempts to * schedule it for execution will throw {@code IllegalStateException}. On Thu, Sep 18, 2014 at 11:54 AM, Alan Bateman <alan.bate...@oracle.com> wrote: > On 18/09/2014 18:52, Martin Buchholz wrote: > >> Hi Chris, >> >> I'd like you to do a code review. >> >> http://cr.openjdk.java.net/~martin/webrevs/openjdk9/ >> TimerTask-clarification/ >> https://bugs.openjdk.java.net/browse/JDK-8058550 >> > This looks okay. If I were wording this then I think I might word in the > singular, as in "Once a timer task has been scheduled ... then subsequent > attempts to schedule it ...". > > -Alan. >