diff options
author | Dirk Hohndel <dirk@hohndel.org> | 2015-12-01 15:22:38 -0800 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2015-12-01 15:32:16 -0800 |
commit | c0ac9532427dee713c8ba96a1ceb10ee78b2bfc8 (patch) | |
tree | 1a66c8f01201f0e2c755c91c634465990cac66a6 /subsurface-core/git-access.c | |
parent | 9dd26a00e86eb0c7e5f0f9650f021b24e73d67df (diff) | |
download | subsurface-c0ac9532427dee713c8ba96a1ceb10ee78b2bfc8.tar.gz |
QML UI: call plot dive when we set the dive
The asynchronous nature of the profile bites us here. plotDive() signals
that it changes model data and expects the rest of the data structures to
respond to that. Very neat and it seems to work perfectly well on the
desktop, but on Android calling render() right after plotDive() resulted
in paint() functions being called before all the elements had been
calculated as a result of the signals being emitted in the model change.
That's why so often the profile was missing parts.
Now admittedly this makes me nervous. Do we now know that all calculations
have finished by the time render() gets called? Not really. It just seems
that in my testing we tend to get lucky and things work out. But that does
not feel like a sane architecture to me.
Messing around with the animation speed is silly as we render the profile
into a pixmap, so let's turn this off globally.
Also, the scaling of the pixmap is still completely bogus.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'subsurface-core/git-access.c')
0 files changed, 0 insertions, 0 deletions