cleanup(storage): remove disable_deprecation_warnings.inc headers once deprecated checksum options are removed
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 55/100
Piste de recherche
Start by checking issue #16404 to confirm the deprecated checksum and hashing symbols have been removed; this cleanup depends on that work. Then inspect the listed Cloud Storage headers and implementation files for disable_deprecation_warnings.inc and, in signed_url_requests.h, diagnostics_pop.inc. Remove the suppression includes once no deprecated symbols remain, and verify the relevant Bazel builds no longer need them.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Summary
Several Cloud Storage headers and implementation files include google/cloud/internal/disable_deprecation_warnings.inc (and google/cloud/internal/diagnostics_pop.inc in headers) to suppress -Wdeprecated-declarations compiler warnings while maintaining backward compatibility for deprecated checksum/hashing options.
Once the deprecated checksum/hashing symbols (MD5HashValue, Crc32cChecksumValue, UseCrc32cValueOption, UseMD5ValueOption, EnableCrc32cValidationOption, EnableMD5ValidationOption, DisableMD5Hash, DisableCrc32cChecksum) are removed from the codebase (see #16404), remove these .inc diagnostic suppression headers.
Background
- PR #16311 formally deprecated
MD5HashValue,Crc32cChecksumValue,UseCrc32cValueOption, andUseMD5ValueOptionin favor ofPrecomputedChecksumsOption(andEnableCrc32cValidationOption/EnableMD5ValidationOption/DisableCrc32cChecksum/DisableMD5Hashin favor ofUploadChecksumValidationOptionandDownloadChecksumValidationOption). - To prevent
-Wdeprecated-declarationswarnings in Bazel builds while continuing to honor these deprecated options internally,disable_deprecation_warnings.inc(anddiagnostics_pop.inc) was added to internal headers and implementation files.
Affected Files
Files updated to suppress deprecation diagnostics for internal backward-compatibility plumbing:
google/cloud/storage/internal/signed_url_requests.h(disable_deprecation_warnings.incanddiagnostics_pop.inc)google/cloud/storage/parallel_upload.ccgoogle/cloud/storage/internal/async/writer_connection_impl.ccgoogle/cloud/storage/internal/async/connection_impl.ccgoogle/cloud/storage/internal/async/default_options.cc
Additional google/cloud/storage/ files containing disable_deprecation_warnings.inc for deprecated checksum options (see also #16404):
google/cloud/storage/hashing_options.hgoogle/cloud/storage/parallel_upload.hgoogle/cloud/storage/client.ccgoogle/cloud/storage/internal/async/checksum_helpers.hgoogle/cloud/storage/internal/checksum_helpers.hgoogle/cloud/storage/internal/connection_impl.ccgoogle/cloud/storage/internal/grpc/stub.ccgoogle/cloud/storage/internal/hash_function.hgoogle/cloud/storage/internal/hash_function.ccgoogle/cloud/storage/internal/hash_validator.ccgoogle/cloud/storage/internal/object_requests.hgoogle/cloud/storage/internal/object_requests.ccgoogle/cloud/storage/internal/rest/stub.cc
- Langage dominant
- C++
- Étoiles
- 659
- Forks
- 475
- Merge moyen
- 1 j 12 h
- PR mergées (30 j)
- 77
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de googleapis/google-cloud-cpp
-
cleanup(storage): remove deprecated OpenTelemetry rpc.message.type and rpc.message.id attributesOuverteapi: storage type: cleanup
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
googleapis/google-cloud-cpp#16544 ·
Les mainteneurs répondent en général sous 1 jour
-
api: storage priority: p2 type: bug
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
googleapis/google-cloud-cpp#16529 ·
Les mainteneurs répondent en général sous 1 jour
-
type: feature request
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
googleapis/google-cloud-cpp#16193 ·
Les mainteneurs répondent en général sous 1 jour
-
storage: MSan use-after-dtor flake at exit in tests calling EnableGrpcMetricsImplPeut-être pris @kalragauri l’a pris il y a 2 jours. Ouverteapi: storage type: bug
googleapis/google-cloud-cpp#16587 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
googleapis/google-cloud-cpp#16575 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de googleapis/google-cloud-cpp
Issues similaires
-
[Bug] [Go] Ack returns nil when the server answers with a non-OK statusPeut-être pris @leizhiyuan l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
apache/rocketmq-clients#1465 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
KhronosGroup/Vulkan-Tutorial#534 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
Les mainteneurs répondent en général sous 2 jours
-
bug IO
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
MrNeRF/LichtFeld-Studio#3824 ·
Les mainteneurs répondent en général sous 1 jour
-
pnnx: manual tanh GELU fusion loses the approximation modePeut-être pris @ktz03 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Tencent/ncnn#7175 · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour