Put back `-Wsign-conversion` for gcc in CompilerWarnings.cmake
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 85/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- cmake, cpp
- Área
- build-system
Línea de trabajo
Comienza en CompilerWarnings.cmake e inspecciona cómo se seleccionan las flags de advertencia para gcc y C++. Reproduce el ejemplo proporcionado de C++23 con gcc usando -Wconversion y -Wsign-conversion. Se considera terminado cuando las compilaciones de C++ con gcc vuelvan a funcionar con -Wsign-conversion habilitado, manteniendo el comportamiento indicado para C y Clang.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Issue #76 / PR #149 removed the -Wsign-conversion flag in CompilerWarnings.cmake. This may be OK for Clang and gcc using C, but it is not OK for gcc using C++.
The gcc documentation says:
In C, this option is enabled also by -Wconversion.
So NOT for C++.
See this demo for an example: https://godbolt.org/z/8YMbWr3Wx
Using gcc 15.2, compiling the code with -O3 -std=c++23 -Werror -Wconversion will work, but using -O3 -std=c++23 -Werror -Wconversion -Wsign-conversion will not.
- Lenguaje dominante
- CMake
- Estrellas
- 1.8k
- Forks
- 205
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cpp-best-practices/cmake_template
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
-
Enable sanitizers globallyAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
cpp-best-practices/cmake_template#140 · 2 comentarios ·
-
From-scratch clone/build fails on `src/ftxui_sample/CMakeFiles/intro.dir/main.cpp.obj`Posiblemente ocupada @leftibot la tomó hace 195 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
cpp-best-practices/cmake_template#139 · 2 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
cpp-best-practices/cmake_template#91 · 2 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
cpp-best-practices/cmake_template#79 · 4 comentarios · 1 reacción ·
Todos los issues de cpp-best-practices/cmake_template
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
immortalwrt/immortalwrt#2522 ·
-
Font inlining cache is not created or reusedPosiblemente ocupada @nagahara-naoki la tomó hoy. Abiertoarea: @angular/build
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
angular/angular-cli#34719 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
osaurus-ai/osaurus#3422 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
foundation/foundation-emails#2127 ·