Diffusion Krita f336923b8796

Fix assert and a data-loss in Transform Tool

Authored by dkazakov on Oct 30 2019, 9:58 AM.

Description

Fix assert and a data-loss in Transform Tool

The patch rewrites the logic of transform stroke completion. Previously,
the decision whether to cancel transformation or to recover continued
state was done by the GUI thread. It cased troubles, because the user
could press esc/enter keys too quickly, even before the stroke was
actually initialized (and before sigTransactionGenerated() was received).
It caused confsion, resulting in the loss of the data of the continued
state.

Now GUI thread doesn't worry about the continued state. All the decisions
are done by the stroke itself, so no races should happen.

BUG:412561

Details

Committed
dkazakovNov 5 2019, 6:41 PM
Parents
R37:179649cf5a83: Fix regression (deadlock) when loading .kra with non-default color prfofile
Branches
Unknown
Tags
Unknown