Review Request 128730: kdeclarative: fix crash in IconDialog when no QApplication is available.

Marco Martin notmart at gmail.com
Mon Aug 22 08:52:32 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128730/#review98541
-----------------------------------------------------------



unfortunately just making those classes not explode even tough inoperative may be the only solution
but this seem to unconditionally render this inoperative? (never creates the dialog, even when there is a qapp?


src/qmlcontrols/kquickcontrolsaddons/icondialog.cpp (line 38)
<https://git.reviewboard.kde.org/r/128730/#comment66384>

    should create the m_dialog instance in this branch?


- Marco Martin


On Aug. 21, 2016, 5:36 p.m., David Faure wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128730/
> -----------------------------------------------------------
> 
> (Updated Aug. 21, 2016, 5:36 p.m.)
> 
> 
> Review request for KDE Frameworks and Marco Martin.
> 
> 
> Repository: kdeclarative
> 
> 
> Description
> -------
> 
> This happens when QtCreator launches qmlplugindump.
> Testcase: qmlplugindump -nonrelocatable org.kde.plasma.calendar 2.0 /usr/lib64/qml
> 
> There are more similar issues, e.g. in
> plasma-framework/src/declarativeimports/plasmacomponents/qmenu.cpp
> 
> 
> Diffs
> -----
> 
>   src/qmlcontrols/kquickcontrolsaddons/icondialog.cpp 4fc7dcc99c7474b4658150c3622c5685d092e6ad 
> 
> Diff: https://git.reviewboard.kde.org/r/128730/diff/
> 
> 
> Testing
> -------
> 
> as mentionned above
> 
> 
> Thanks,
> 
> David Faure
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160822/73690b2f/attachment.html>


More information about the Kde-frameworks-devel mailing list