ClickHouse / ClickHouse/clickhouse-cpp
Remove `my_result_of_t` in favor of `std::invoke_result_t`
Ouverte
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