https://bugs.kde.org/show_bug.cgi?id=462627

Robert Hairgrove <c...@roberthairgrove.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|FIXED                       |---
             Status|RESOLVED                    |REOPENED
                 CC|                            |c...@roberthairgrove.com

--- Comment #2 from Robert Hairgrove <c...@roberthairgrove.com> ---
Wouldn't it be better to create UMLApp as a local variable, i.e. on the stack,
in the main() function?

Also, it seems that there is this at line 173:
    QPointer<UMLApp> uml;

Doesn't this handle deletion of 'uml' automatically?

--

(In reply to Oliver Kellogg from comment #1)
> Git commit 03b0d2f64495a016dda1998a8f53c706bf4312bb by Oliver Kellogg.
> Committed on 04/12/2022 at 11:49.
> Pushed by okellogg into branch 'master'.
> 
> umbrello/umlwidgets/umlwidget.cpp
> - In function removeAssoc, if changesShape() is true then refrain from
>   calling updateGeometry() if UMLApp::app()->shuttingDown() is true.
>   Reason: Otherwise a signal is propagated which leads to the crash.
> FIXED-IN: 2.37.70 (KDE releases 23.03.70)
> 
> M  +1    -1    umbrello/umlwidgets/umlwidget.cpp
> 
> https://invent.kde.org/sdk/umbrello/commit/
> 03b0d2f64495a016dda1998a8f53c706bf4312bb

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to