cleanup(storage): remove disable_deprecation_warnings.inc headers once deprecated checksum options are removed
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 55/100
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- C++
- Estrellas
- 659
- Forks
- 471
- Merge medio
- 1 d 10 h
- PR fusionados (30 d)
- 100
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de googleapis/google-cloud-cpp
-
cleanup(storage): remove deprecated OpenTelemetry rpc.message.type and rpc.message.id attributesAbiertoapi: storage type: cleanup
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
googleapis/google-cloud-cpp#16544 ·
Los mantenedores suelen responder en 1 día
-
api: storage priority: p2 type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
googleapis/google-cloud-cpp#16529 ·
Los mantenedores suelen responder en 1 día
-
type: feature request
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
googleapis/google-cloud-cpp#16193 ·
Los mantenedores suelen responder en 1 día
-
api: storage type: cleanup
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
googleapis/google-cloud-cpp#16404 ·
Los mantenedores suelen responder en 1 día
-
cpp: generator type: cleanup
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
googleapis/google-cloud-cpp#16394 ·
Los mantenedores suelen responder en 1 día
Todos los issues de googleapis/google-cloud-cpp
Issues similares
-
Component: Python API
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Vector35/binaryninja-api#8649 ·
Los mantenedores suelen responder en 3 días
-
ai_p2 comp-parquet-reader-v3
Dificultad 2/5 Medio día Aptitud para principiantes 66/100
ClickHouse/ClickHouse#124986 ·
Los mantenedores suelen responder en 1 día
-
bug product: very_good_flutter_plugin
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
VeryGoodOpenSource/very_good_templates#654 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
AcademySoftwareFoundation/OpenImageIO#5550 ·
Los mantenedores suelen responder en 2 días