View Single Post
Old 11-14-2019, 11:27 AM   #114
KevinH
Sigil Developer
KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.
 
Posts: 8,930
Karma: 6361444
Join Date: Nov 2009
Device: many
I have now pushed another fix for this problem which really comes down to a guarded QPointer being updated too quickly causing a collision with the pointer being manually updated when one of its earlier values is being updated to 0 due to the underlying widget it once pointed to being deleted (via a deleteLater).

There was really no reason to make this a QPointer as we manually update it in our own TabManager delete routines, and it is never shared outside the TabManager except via a signal and we generate said signal so we control it. That the signal is sent and processed **before** the underlying tab widget is ever destructed.

With this change, I can no longer get too fast tabs to fail even when using ctrl-W on 50 images.

If this change does not cause other problems, we should be able to put this to rest.

Please grab a new version and give it a try.
KevinH is offline   Reply With Quote