-
Task
-
Resolution: Done
-
P2: Important
-
None
-
None
-
13
-
Qt6_Foundation_Sprint 38
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.
| 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 |