cleanup(storage): remove disable_deprecation_warnings.inc headers once deprecated checksum options are removed
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 55/100
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- C++
- Stelle
- 659
- Fork
- 471
- Merge medio
- 1g 10h
- PR unite (30g)
- 100
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di googleapis/google-cloud-cpp
-
cleanup(storage): remove deprecated OpenTelemetry rpc.message.type and rpc.message.id attributesApertaapi: storage type: cleanup
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
googleapis/google-cloud-cpp#16544 ·
I maintainer di solito rispondono entro 1 giorno
-
api: storage priority: p2 type: bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
googleapis/google-cloud-cpp#16529 ·
I maintainer di solito rispondono entro 1 giorno
-
type: feature request
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
googleapis/google-cloud-cpp#16193 ·
I maintainer di solito rispondono entro 1 giorno
-
api: storage type: cleanup
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
googleapis/google-cloud-cpp#16404 ·
I maintainer di solito rispondono entro 1 giorno
-
cpp: generator type: cleanup
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
googleapis/google-cloud-cpp#16394 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di googleapis/google-cloud-cpp
Issue simili
-
c++ good first issue M2: audio plumbing no hardware needed tests
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
GageLawton/noise-cancellation#23 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
objectionary/eo-graphs#85 ·
-
bug derived types format I/O medium priority semantics
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Kindly pin CUDA 12.X as required toolkit for stable release v0.3.3 compilation in CUDA systemsAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 68/100
facebookresearch/habitat-sim#2645 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno