Page MenuHomePhorge

Use C++20 default comparisons
ClosedPublic

Authored by tusooa on Oct 2 2025, 7:26 PM.
Tags
None
Referenced Files
F13613961: D238.1765232820.diff
Sun, Dec 7, 2:27 PM
F13611844: D238.1765228555.diff
Sun, Dec 7, 1:15 PM
F13611420: D238.1765227568.diff
Sun, Dec 7, 12:59 PM
F13610112: D238.1765224838.diff
Sun, Dec 7, 12:13 PM
F13603036: D238.1765211006.diff
Sun, Dec 7, 8:23 AM
F13585610: D238.1765183935.diff
Sun, Dec 7, 12:52 AM
F13570280: D238.1765166915.diff
Sat, Dec 6, 8:08 PM
Subscribers

Details

Summary

This requires C++20 and uses the default comparisons where possible so we will no longer need to manually change the comparison operators when we add fields.

Type: add

Test Plan

Build kazv ( https://iron.lily-is.land/D240 ) against this. Verify everything works.

Diff Detail

Repository
rL libkazv
Branch
tusooa/cpp20
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 452
Build 887: GitLab CI for libkazv
Build 886: arc lint + arc unit

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Oct 2 2025, 7:40 PM
Harbormaster failed remote builds in B452: Diff 709!

Fix discovery-test compilation

Harbormaster returned this revision to the author for changes because remote builds failed.Oct 3 2025, 8:28 AM
Harbormaster failed remote builds in B453: Diff 710!

kazv cannot be compiled with this diff.

In D238#4678, @nannanko wrote:

kazv cannot be compiled with this diff.

kazv can be compiled with c++20 in https://iron.lily-is.land/D240 together with this diff.

This revision is now accepted and ready to land.Oct 11 2025, 7:40 PM
This revision was automatically updated to reflect the committed changes.