JBotSim icon indicating copy to clipboard operation
JBotSim copied to clipboard

JClock.setTimeUnit doesn't set initial delay of the timer

Open pictavien opened this issue 5 years ago • 1 comments

The method JClock.setTimeUnit(int delay) does not update the initial delay of the timer used by the JClock. I suggest to add timer.setInitialDelay(delay).

pictavien avatar Jan 14 '20 16:01 pictavien

I applied this and pushed it on branch fix/89-JClock-settimeunit-delay .

Note:

  • as far as I can tell, it seems to fix issue #90

  • the rare users which use the maximum 3 seconds delay with the JViewer "Set system speed" feature, if unlucky, might experience a maximum 3 seconds delay before noticing the clock speed update.

    In my opinion, it is a small price to pay compared to issue #90 inconvenience.

remikey avatar Jan 31 '20 09:01 remikey