Upgrade from c++17 to c++20

mit-dci / opencbdc-tx

A transaction processor for a hypothetical, general-purpose, central bank digital currency

Other

891 stars 200 forks source link

Upgrade from c++17 to c++20 #238

Closed AlexRamRam closed 4 months ago

AlexRamRam commented 9 months ago

Fix #237

maurermi commented 9 months ago

concept-ACK, I think this is a good idea -- will test soon

HalosGhost commented 9 months ago

Concept ACK (we had initially avoided 20 till Bitcoin switches, but given we now have dependencies which will require switching, I think that ship has sailed); reminder we need to test basic performance impact locally and in test controller.

HalosGhost commented 4 months ago

tACK. I've checked rebased against the latest trunk, and done some local performance comparisons (no performance degradation is visible; if anything, I see slightly higher sustained throughput for identical configurations).

LGTM. Merging.