Summary: | Krita freezes when trying to edit markers for a vector layer | ||
---|---|---|---|
Product: | [Applications] krita | Reporter: | Dmitry Kazakov <dimula73> |
Component: | Tools/Vector | Assignee: | Dmitry Kazakov <dimula73> |
Status: | RESOLVED FIXED | ||
Severity: | crash | Keywords: | regression, release_blocker |
Priority: | NOR | ||
Version: | 4.3.0-beta2 | ||
Target Milestone: | --- | ||
Platform: | Microsoft Windows | ||
OS: | Other | ||
Latest Commit: | https://invent.kde.org/graphics/krita/commit/eca114cde93e20d19e79b6570e7c11a5a083bacc | Version Fixed In: |
Description
Dmitry Kazakov
2020-06-04 21:20:11 UTC
That is a regression against 4.2.9 Git commit 28cb6b1afcc05434665c0c23fa94068de95716fb by Dmitry Kazakov. Committed on 05/06/2020 at 07:56. Pushed by dkazakov into branch 'krita/4.3'. Fix double updates when changing shape's marker M +5 -4 libs/flake/commands/KoPathShapeMarkerCommand.cpp https://invent.kde.org/graphics/krita/commit/28cb6b1afcc05434665c0c23fa94068de95716fb Git commit dce24a2887ed398204a20e79772829ad6acf4cf2 by Dmitry Kazakov. Committed on 05/06/2020 at 07:56. Pushed by dkazakov into branch 'krita/4.3'. Fix freeze when changing shape's marker KoMarker::maxInset() should not multiply the inset by the line width, because it has already been multiplied in KoMarker::boundingRect(). M +0 -4 libs/flake/KoMarker.cpp https://invent.kde.org/graphics/krita/commit/dce24a2887ed398204a20e79772829ad6acf4cf2 Git commit adc30236e4fe15e14c98358330eecac1af600b0f by Dmitry Kazakov. Committed on 05/06/2020 at 07:56. Pushed by dkazakov into branch 'krita/4.3'. Don't update too much area around path shapes outlineSweepWidth is 'width' of the line, not an inset. Therefore, we should grow the bounding box by half of it, not by the full value. M +2 -1 libs/flake/KoPathShape.cpp https://invent.kde.org/graphics/krita/commit/adc30236e4fe15e14c98358330eecac1af600b0f Git commit 8c93348a311df72b8bb2381c0d98723768b07527 by Dmitry Kazakov. Committed on 05/06/2020 at 07:56. Pushed by dkazakov into branch 'krita/4.3'. Notify KoShapeManager's r-tree about changing the shape's bounds Otherwise, the tree will skip updating the area under the marker until the shape is changed in a proper way. M +3 -0 libs/flake/KoPathShape.cpp https://invent.kde.org/graphics/krita/commit/8c93348a311df72b8bb2381c0d98723768b07527 Git commit e2ce6347d630c1833babe1cfe1c27663b74af555 by Dmitry Kazakov. Committed on 05/06/2020 at 07:56. Pushed by dkazakov into branch 'master'. Fix double updates when changing shape's marker M +5 -4 libs/flake/commands/KoPathShapeMarkerCommand.cpp https://invent.kde.org/graphics/krita/commit/e2ce6347d630c1833babe1cfe1c27663b74af555 Git commit e361f9fe0f4c3a38347d380b85f8cc6d39fd7bf6 by Dmitry Kazakov. Committed on 05/06/2020 at 07:56. Pushed by dkazakov into branch 'master'. Notify KoShapeManager's r-tree about changing the shape's bounds Otherwise, the tree will skip updating the area under the marker until the shape is changed in a proper way. M +3 -0 libs/flake/KoPathShape.cpp https://invent.kde.org/graphics/krita/commit/e361f9fe0f4c3a38347d380b85f8cc6d39fd7bf6 Git commit 042ad182551efe29d64869e86edeab422fc15809 by Dmitry Kazakov. Committed on 05/06/2020 at 07:57. Pushed by dkazakov into branch 'master'. Don't update too much area around path shapes outlineSweepWidth is 'width' of the line, not an inset. Therefore, we should grow the bounding box by half of it, not by the full value. M +2 -1 libs/flake/KoPathShape.cpp https://invent.kde.org/graphics/krita/commit/042ad182551efe29d64869e86edeab422fc15809 Git commit ae5d76636c42880dc074a80d036663f00cb11b9e by Dmitry Kazakov. Committed on 05/06/2020 at 07:57. Pushed by dkazakov into branch 'master'. Fix freeze when changing shape's marker KoMarker::maxInset() should not multiply the inset by the line width, because it has already been multiplied in KoMarker::boundingRect(). M +0 -4 libs/flake/KoMarker.cpp https://invent.kde.org/graphics/krita/commit/ae5d76636c42880dc074a80d036663f00cb11b9e Git commit 6fd1a2e68308fe35cba60d5e7b39872b18a7045b by Dmitry Kazakov. Committed on 05/06/2020 at 07:58. Pushed by rempt into branch 'krita/4.3.0'. Fix double updates when changing shape's marker M +5 -4 libs/flake/commands/KoPathShapeMarkerCommand.cpp https://invent.kde.org/graphics/krita/commit/6fd1a2e68308fe35cba60d5e7b39872b18a7045b Git commit eca114cde93e20d19e79b6570e7c11a5a083bacc by Dmitry Kazakov. Committed on 05/06/2020 at 07:58. Pushed by rempt into branch 'krita/4.3.0'. Fix freeze when changing shape's marker KoMarker::maxInset() should not multiply the inset by the line width, because it has already been multiplied in KoMarker::boundingRect(). M +0 -4 libs/flake/KoMarker.cpp https://invent.kde.org/graphics/krita/commit/eca114cde93e20d19e79b6570e7c11a5a083bacc Git commit 61b683a836db0e7c6c605cf405d5dd0838077955 by Dmitry Kazakov. Committed on 05/06/2020 at 07:58. Pushed by rempt into branch 'krita/4.3.0'. Notify KoShapeManager's r-tree about changing the shape's bounds Otherwise, the tree will skip updating the area under the marker until the shape is changed in a proper way. M +3 -0 libs/flake/KoPathShape.cpp https://invent.kde.org/graphics/krita/commit/61b683a836db0e7c6c605cf405d5dd0838077955 Git commit 5287dba3880cfa0ce3086ceff9b203d4daa23abf by Dmitry Kazakov. Committed on 05/06/2020 at 07:58. Pushed by rempt into branch 'krita/4.3.0'. Don't update too much area around path shapes outlineSweepWidth is 'width' of the line, not an inset. Therefore, we should grow the bounding box by half of it, not by the full value. M +2 -1 libs/flake/KoPathShape.cpp https://invent.kde.org/graphics/krita/commit/5287dba3880cfa0ce3086ceff9b203d4daa23abf |