aws / aws/aws-sdk-cpp

Fix compiler warnings

Aperta
#2,363 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub
bug p3
Lingua principale
C++
Stelle
2.2k
Fork
1.2k
Merge medio
4g 11h
PR unite (30g)
13

Descrizione

### 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

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Riproduci gli avvisi con clang 14, CMake e Ninja, quindi esamina i file elencati: NoResult.h, AmazonSerializableWebServiceRequest.h, HttpResponse.h, socket.h, StringView.h e math.gcc_builtin.inl. Conferma le correzioni con la build di MacOS 12.6; il lavoro è completato quando gli avvisi del compilatore segnalati sono stati eliminati senza introdurne di nuovi.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cmake, cpp
Ambito
build-system, tooling
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
32/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.