Details
-
Task
-
Resolution: Done
-
P2: Important
-
None
-
None
-
13
-
Qt6_Foundation_Sprint 38
Description
During the API review we've found out that we're missing some important things:
- convert all comparison operators to hidden friends
- convert all streaming operators to hidden friends
- use constexpr and noexcept when possible
- consider adding qHash overloads for value types
- consider adding move constructors and swap() to value types
- consider using Q_DECLARE_SHARED instead of Q_DECLARE_TYPEINFO()
- add virtual setBackendProperty()/backendProperty() to QGeoAreaMonitorSource
Some of the changes are not BC-breaking, so they can be postponed until later releases. But we should at least check if they make sense.
Attachments
For Gerrit Dashboard: QTBUG-95163 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
359926,3 | QtPositioning: make comparison operators hidden friends | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
360185,6 | QtPositioning: convert streaming operators to hidden friends | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
360334,2 | QGeoAreaMonitorSource: add virtual methods to toggle custom properties | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
360422,3 | QGeoAddress: support move operations | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
360483,2 | QGeoAreaMonitorSource: add virtual methods to toggle custom properties | 6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
360485,2 | QtPositioning: make comparison operators hidden friends | 6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
360611,2 | QtPositioning: convert streaming operators to hidden friends | 6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
360612,2 | QGeoAddress: support move operations | 6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
361126,2 | QGeoCoordinate: support move operations | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
361157,2 | QGeoLocation: support move operations | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
361187,1 | QGeoShape hierarchy: support move operations | dev | qt/qtlocation | Status: ABANDONED | 0 | 0 |
361260,2 | QGeoCoordinate: support move operations | 6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
361261,2 | QGeoLocation: support move operations | 6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
361959,3 | Add qHash overloads to QGeoShape hierarchy | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
361972,3 | QGeoAddress: add qHash overload | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
361973,4 | QGeoLocation: add qHash overload | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
362110,2 | Add qHash overloads to QGeoShape hierarchy | 6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
362111,2 | QGeoAddress: add qHash overload | 6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
362159,2 | QGeoLocation: add qHash overload | 6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |