aws / aws/aws-sdk-cpp

Fix compiler warnings

Offen
#2,363 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
bug p3
Vorherrschende Sprache
C++
Sterne
2.2k
Forks
1.2k
Ø Merge
4 T. 11 Std.
Gemergte PRs (30 T.)
13

Beschreibung

### Describe the bug

My compiler shows me a bunch of warnings that I'd like to get fixed. This is just a small list:

`aws-cpp-sdk-core/include/aws/core/NoResult.h:29:81: warning: extra ';' after member function definition [-Wextra-semi]`
`/aws-cpp-sdk-core/include/aws/core/AmazonSerializableWebServiceRequest.h:42:17: warning: '~AmazonSerializableWebServiceRequest' overrides a destructor but is not marked 'override' [-Winconsistent-missing-destructor-override]`
`aws-cpp-sdk-core/include/aws/core/http/HttpResponse.h:115:21: warning: 65 enumeration values not explicitly handled in switch: 'REQUEST_NOT_MADE', 'CONTINUE', 'SWITCHING_PROTOCOLS'... [-Wswitch-enum]`
`aws-crt-cpp/crt/aws-c-io/include/aws/io/socket.h:98:18: warning: zero as null pointer constant [-Wzero-as-null-pointer-constant]`
`aws-crt-cpp/include/aws/crt/StringView.h:534:49: warning: implicit conversion changes signedness: 'unsigned long' to 'const Aws::Crt::basic_string_view::difference_type' (aka 'const long') [-Wsign-conversion]`
`aws-crt-cpp/crt/aws-c-common/include/aws/common/math.gcc_builtin.inl:63:28: warning: implicit conversion changes signedness: 'int64_t' (aka 'long long') to 'unsigned long long' [-Wsign-conversion]`
`aws-crt-cpp/crt/aws-c-common/include/aws/common/math.gcc_builtin.inl:63:12: warning: implicit conversion changes signedness: 'int' to 'size_t' (aka 'unsigned long') [-Wsign-conversion]`

### Expected Behavior

I expect the SDK to have 0 warnings.

### Current Behavior

The current behaviour is that there are many warnings.

### Reproduction Steps

I'm compiling a CLion project on MacOS 12.6 with ninja build tool and CMake 3.24.2.

### Possible Solution

_No response_

### Additional Information/Context

_No response_

### AWS CPP SDK version used

1.10.47

### Compiler and Version used

clang 14.0.0

### Operating System and version

MacOS 12.6, CLion 2022.3.2

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Reproduziere die Warnungen mit clang 14, CMake und Ninja und untersuche anschließend die aufgeführten Dateien: NoResult.h, AmazonSerializableWebServiceRequest.h, HttpResponse.h, socket.h, StringView.h und math.gcc_builtin.inl. Bestätige die Korrekturen anhand des MacOS 12.6-Builds; abgeschlossen bedeutet, dass die gemeldeten Compiler-Warnungen beseitigt wurden, ohne neue einzuführen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cmake, cpp
Bereich
build-system, tooling
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
32/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.