ClickHouse / ClickHouse/clickhouse-cpp

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

Abierto
#370 0 comentarios 0 reacciones 0 asignados Ver en GitHub
3.0 enhancement
Lenguaje dominante
C
Estrellas
382
Forks
209
Merge medio
2 d 19 h
PR fusionados (30 d)
14

Descripción

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

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Comienza en ut/utils_meta.h, alrededor de la línea 37, y busca en el repositorio todos los usos de my_result_of_t. Actualiza el código de compatibilidad para usar la funcionalidad estándar de C++20 y, después, verifica que la compilación y las pruebas de C++20 del proyecto se completen correctamente.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
cpp
Área
tooling
Tipo de issue
Refactorización
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
55/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.