ClickHouse / ClickHouse/clickhouse-cpp

Remove `my_result_of_t` in favor of `std::invoke_result_t`

Aperta
#370 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
3.0 enhancement
Lingua principale
C
Stelle
382
Fork
209
Merge medio
1g 22h
PR unite (30g)
13

Descrizione

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

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Start at ut/utils_meta.h around line 37 and search the repository for all my_result_of_t usages. Update the compatibility code to use the C++20 standard facility, then verify that the project’s C++20 build and tests pass.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
tooling
Tipo di issue
Refactoring
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
55/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.