[patch] Selection stats in statusbar
Pavel Sanda
sanda at lyx.org
Wed Jul 13 22:13:34 UTC 2022
On Wed, Jul 13, 2022 at 08:54:28AM +0200, Daniel wrote:
> Hi,
>
> There is also a patch for instant counts at
> https://www.lyx.org/trac/ticket/12422. It works with delayed calculations. I
> have no idea how this does performance wise, but it might be worth checking
> out.
Yes, that's lazy man approach, using timer instead of parallel thread.
The advantage is that you avoid race-conditions (though I think you miss stop()
inside closeEvent()?).
The drawback is that's it's difficult to understand the interaction between
the two timers now. I stared on the code for couple minutes and it was
not clear to me what is the idea behind your stop/start changes.
JMarc's warning about ineeficiency of counting remain unaddressed, though
it migh not be visible because of 300ms update sampling frequency.
Pavel
More information about the lyx-devel
mailing list