cleanup(storage): remove disable_deprecation_warnings.inc headers once deprecated checksum options are removed
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 55/100
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- C++
- Sterne
- 659
- Forks
- 471
- Ø Merge
- 1 T. 10 Std.
- Gemergte PRs (30 T.)
- 100
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus googleapis/google-cloud-cpp
-
cleanup(storage): remove deprecated OpenTelemetry rpc.message.type and rpc.message.id attributesOffenapi: storage type: cleanup
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
googleapis/google-cloud-cpp#16544 ·
Maintainer antworten meist innerhalb von 1 Tag
-
api: storage priority: p2 type: bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
googleapis/google-cloud-cpp#16529 ·
Maintainer antworten meist innerhalb von 1 Tag
-
type: feature request
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 73/100
googleapis/google-cloud-cpp#16193 ·
Maintainer antworten meist innerhalb von 1 Tag
-
api: storage type: cleanup
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100
googleapis/google-cloud-cpp#16404 ·
Maintainer antworten meist innerhalb von 1 Tag
-
cpp: generator type: cleanup
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100
googleapis/google-cloud-cpp#16394 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in googleapis/google-cloud-cpp
Ähnliche Issues
-
Status: Awaiting triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
espressif/arduino-esp32#12984 ·
Maintainer antworten meist innerhalb von 1 Tag
-
torch_ops/logprob.cu does not compile with the serving container's nvcc (13.3.73); check_torch_ops.py cannot run as shippedEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
ashhart/TensorFold#535 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag