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

Review request for KDE Frameworks and Thiago Macieira.


Repository: kdelibs4support


Description
-------

We're calling newInstance from the parent but the child may have not gotten to 
the point where the application object has been created so the call may fail.

I don't know why this did not manifest before using Thiago's Qt dbus threading 
patches but seems the code was racy anyway, should not hurt to have this for 
Qt's without those patches.


Diffs
-----

  src/kdeui/kuniqueapplication.cpp 713c6f4 

Diff: https://git.reviewboard.kde.org/r/124856/diff/


Testing
-------

konsole starts fine with this + Thiago's Qt dbus threading patches


Thanks,

Albert Astals Cid

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to