googleapis / googleapis/google-cloud-cpp
Avoid spurious warnings on `*.pb.h` files with MSVC
- Langage dominant
- C++
- Étoiles
- 659
- Forks
- 462
- Merge moyen
- 1 j 2 h
- PR mergées (30 j)
- 89
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Repérez les protos de test dans les chemins google/cloud ou generator actuels et comparez leurs règles de build avec celles des protos googleapis et compute déjà corrigés. Inspectez d’abord la configuration Bazel et CMake correspondante. Le travail est terminé lorsque les en-têtes générés par les tests prennent en charge l’inclusion entre chevrons et ne produisent plus d’avertissements parasites de MSVC.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- cmake, cpp
- Domaine
- build-system
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100