ClickHouse / ClickHouse/clickhouse-cpp

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

Ouverte
#370 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
3.0 enhancement
Langage dominant
C
Étoiles
382
Forks
209
Merge moyen
2 j 19 h
PR mergées (30 j)
14

Description

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

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez dans ut/utils_meta.h, vers la ligne 37, et recherchez dans le dépôt toutes les utilisations de my_result_of_t. Mettez à jour le code de compatibilité pour utiliser la fonctionnalité standard de C++20, puis vérifiez que le build et les tests C++20 du projet passent.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
tooling
Type d'issue
Refactorisation
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
55/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.