PolkitActionsKCM: Use a simple pointer for m_actionWidget.
Review Request #113360 - Created Oct. 20, 2013 and submitted
QPointer<> crashes for me (Arch), the crash is non-debuggable (only in Release mode) and I do not see a way how can m_actionWidget suddenly disappear.
- Ensured that it fixes the crash for me - Made some test modifications and ensured that it does not crash in other places
To be completely honest, the QPointer crash doesn't make sense at all and sounds more like an actual Qt bug more than a PolkitKCM bug. I don't think that would be a good solution in the big picture. That said, it's also true that the QPointer there is pretty random. Looks like the whole code needs quite a revamp. So I'm giving a Ship it to the patch because: * It fixes a leak * The QPointer was, apparently, mere overhead, as there are no situations where a null check happens or is needed at all. So I even wonder why it was there in the first place... Would be nice to know, anyways, whether you experienced that crash with Qt4 or Qt5, and if it is reproducible anywhere else. There's still no apparent justification for this to be honest. Thanks for the patch anyway.