]> sigrok.org Git - pulseview.git/commit
Fix #1204 by making sure the lazy event handler is fired
authorSoeren Apel <redacted>
Wed, 25 Jul 2018 11:19:56 +0000 (13:19 +0200)
committerUwe Hermann <redacted>
Tue, 21 Aug 2018 10:02:38 +0000 (12:02 +0200)
commit68e0a53ee49c6a58fe1fadd8b9dc12572cc0dbd0
treee5cf441de95c30e3c5713114dbcd4dfe1ceadfc7
parent7d9dd6f4f619bf53a3b19096f6c4f02bbe1a257b
Fix #1204 by making sure the lazy event handler is fired

The QTimer reference says about QTimer::start():

> Starts or restarts the timer with the timeout specified in interval.
> If the timer is already running, it will be stopped and restarted.
> If singleShot is true, the timer will be activated only once.

This suggests that the timer is reset when start() is called even
when it is in single shot mode. However, this is actually not the case
and start() doesn't appear to be doing anything if it has already
been started. While this works in most cases, it causes issues when
the timer is supposed to be reset and timeout at the specified time
since view items are still in flux.

Adding a stop() before the start() fixes this.
pv/views/trace/view.cpp