Remove usage of QWeakPointer for QObject for AbstractThumbnailItem::m_parent
ClosedPublic

Authored by gladhorn on Sat, Aug 10, 10:16 AM.

Details

Summary

This usage of QWeakPointer has been deprecated since Qt 5.0, since it
leads to really confusing API - usually you must never dereference a
QWeakPointer directly, but always go through QSharedPointer, except in
this one case, where it's permissible.

Use QPointer instead, which is clean.

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
gladhorn created this revision.Sat, Aug 10, 10:16 AM
Restricted Application added a project: KWin. · View Herald TranscriptSat, Aug 10, 10:16 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
gladhorn requested review of this revision.Sat, Aug 10, 10:16 AM
zzag accepted this revision.Sat, Aug 10, 10:37 AM
This revision is now accepted and ready to land.Sat, Aug 10, 10:37 AM
romangg requested changes to this revision.Sat, Aug 10, 10:37 AM
This revision now requires changes to proceed.Sat, Aug 10, 10:37 AM

Sorry, was just to write a commet on the other patch about the names of the commits and missing (?) part 1. See https://phabricator.kde.org/D23070#509746

gladhorn updated this revision to Diff 63474.Sat, Aug 10, 10:53 AM
gladhorn retitled this revision from Remove usage of QWeakPointer for QObject part 3 to Remove usage of QWeakPointer for QObject for AbstractThumbnailItem::m_parent.

Improve commit message

gladhorn updated this revision to Diff 63475.Sat, Aug 10, 11:30 AM
gladhorn edited the summary of this revision. (Show Details)

improve commit message

romangg accepted this revision.Sat, Aug 10, 12:33 PM
This revision is now accepted and ready to land.Sat, Aug 10, 12:33 PM
This revision was automatically updated to reflect the committed changes.