ClickHouse / ClickHouse/clickhouse-cpp
Remove `my_result_of_t` in favor of `std::invoke_result_t`
Offen
3.0
enhancement
- Vorherrschende Sprache
- C
- Sterne
- 382
- Forks
- 209
- Ø Merge
- 1 T. 22 Std.
- Gemergte PRs (30 T.)
- 13
Beschreibung
Kludged to support both C++ 17 and C++ 20, but now since we've completely switched to C++20 in 3.0 branch, it is safe to remove it.
https://github.com/ClickHouse/clickhouse-cpp/blob/ca8a430765302803b40d68589e84b98a4612ac66/ut/utils_meta.h#L37
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Beginne in ut/utils_meta.h ungefähr bei Zeile 37 und durchsuche das Repository nach allen Verwendungen von my_result_of_t. Aktualisiere den Kompatibilitätscode so, dass er die C++20-Standardfunktion verwendet, und überprüfe anschließend, dass der C++20-Build und die Tests des Projekts erfolgreich sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- tooling
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 55/100