googleapis / googleapis/google-cloud-cpp

Avoid spurious warnings on `*.pb.h` files with MSVC

Abierto
#14,507 0 comentarios 0 reacciones 0 asignados Ver en GitHub
type: cleanup
Lenguaje dominante
C++
Estrellas
659
Forks
462
Merge medio
1 d 2 h
PR fusionados (30 d)
89

Descripción

Files generated by Protobuf have warnings under MSVC. There is little we can do to fix the code to avoid warnings, and disabling the warnings for all the code would mask problems in *our* code.

We can disable warnings in headers with MSVC if the header is included as `#include ` as opposed to `#include "foo/bar.pb.h"`. We have managed to fix this for googleapis protos, and the compute protos, but the test protos in `google-cloud-cpp` are not.

If we move the protos to `protos/...` (as oppposed to `google/cloud/...` or `generator/...`) then we can tweak Bazel and CMake to support include via angle brackets, then the warnings go away, and then we can start using newer versions of Protobuf with MSVC.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Localiza los protos de prueba en las rutas actuales de google/cloud o generator y compara sus reglas de compilación con las de los protos de googleapis y compute ya corregidos. Inspecciona primero la configuración relacionada de Bazel y CMake. Se considera terminado cuando los encabezados generados para las pruebas admitan la inclusión con corchetes angulares y dejen de producir advertencias espurias de MSVC.

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

Evaluación

Stack tecnológico
cmake, cpp
Área
build-system
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.