From e0020d36f20be650b0c26f4bdbc943585151468b Mon Sep 17 00:00:00 2001 From: David Capello Date: Thu, 13 Aug 2015 20:45:44 -0300 Subject: [PATCH] Fix bug undoing/redoing pixel movements with several mask movements If we call updateDocumentMask() in this point, we'll leave a sequence of cmds: SetMask(old mask, new mask) -> ClearMask(new mask) And we need the opposite: ClearMask(old mask) -> SetMask(old mask, new mask) This bug was introduced in 35229e99a6196d71b90ef2762abb32b0a164f594 --- src/app/ui/editor/pixels_movement.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/app/ui/editor/pixels_movement.cpp b/src/app/ui/editor/pixels_movement.cpp index aa47ff4f5..ad0daa689 100644 --- a/src/app/ui/editor/pixels_movement.cpp +++ b/src/app/ui/editor/pixels_movement.cpp @@ -88,9 +88,7 @@ PixelsMovement::PixelsMovement( // and its owner could destroy our new "extra cel". ASSERT(!m_document->getExtraCel()); redrawExtraImage(); - redrawCurrentMask(); - updateDocumentMask(); } PixelsMovement::~PixelsMovement()