Cleaning up cmake and enabled more warnings.

cisco / libsrtp

Library for SRTP (Secure Realtime Transport Protocol)

Other

1.19k stars 470 forks source link

Cleaning up cmake and enabled more warnings. #666

Closed palerikm closed 6 months ago

palerikm commented 6 months ago

Warning! There are still lots of compiler warnings turned off. A good code cleanup is needed to turn those on (cmake/Warnings.cmake) -Wformat=2 # warn on security issues around functions that format output -Wcast-align # warn for potential performance problem casts -Wconversion # warn on type conversions that may lose data -Wsign-conversion # warn on sign conversions

pabuhler commented 6 months ago

LGTM, if only the conflict was fixed :)

palerikm commented 6 months ago

Sorry about that. Conflicts resolved.