Value types should
- be movable (ideally with both move-constructor and move-assignment operator)
- use explicit constructors from other types
- use noexcept as appropriate (for all implicitly shared classes: moving, copying, swapping, comparing)
- not have any protected or virtual members (subclassing a value type is not allowed)
- use hidden friends for operators
Types can either have none of the special member functions defined, or all of them.
| For Gerrit Dashboard: QTBUG-105206 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 423819,2 | Remove unused placemacro header | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 423820,2 | Clean up test cases for value types | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 423821,2 | Modernize Maps value types | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 423822,7 | Modernize Places value types | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 423891,2 | Remove unused placemacro header | lts-6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
| 423892,2 | Clean up test cases for value types | lts-6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
| 424185,2 | Modernize Maps value types | lts-6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
| 424231,3 | Modernize Places value types | lts-6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
| 424301,4 | Clean up mix of polymorph and value semantics | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 424302,4 | Modernize and clean up QGeoRoute(Leg) | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 424342,3 | Cleanup private map types | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 427574,3 | Clean up mix of polymorph and value semantics | lts-6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
| 427575,3 | Modernize and clean up QGeoRoute(Leg) | lts-6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
| 429503,3 | Cleanup private map types | lts-6.2 | qt/qtlocation | Status: MERGED | +2 | 0 |
| 431869,10 | Merge QGeoRouteLeg into QGeoRoute | dev | qt/qtlocation | Status: MERGED | +2 | 0 |