Commit e5d4026c authored by Ghislain MARY's avatar Ghislain MARY

Stricter compilation options + new compilation error fixes.

parent 0d3874b5
......@@ -87,7 +87,7 @@ if(NOT MSVC)
list(APPEND STRICT_OPTIONS_CPP "-Qunused-arguments")
endif()
if(ENABLE_STRICT)
list(APPEND STRICT_OPTIONS_CPP "-Werror")
list(APPEND STRICT_OPTIONS_CPP "-Werror" "-Wextra" "-Wno-unused-parameter")
endif()
endif()
if(STRICT_OPTIONS_CPP)
......
......@@ -59,7 +59,7 @@ case $CC in
esac
if test $GCC = yes && test $wall_werror = yes; then
CFLAGS="$CFLAGS -Werror "
CFLAGS="$CFLAGS -Werror -Wextra -Wno-unused-parameter "
fi
# Create the following files from their .in counterparts
......
......@@ -689,7 +689,7 @@ void bzrtp_setSupportedCryptoTypes(bzrtpContext_t *zrtpContext, uint8_t algoType
* @return 0 on success, errorcode otherwise
*/
int bzrtp_setPeerHelloHash(bzrtpContext_t *zrtpContext, uint32_t selfSSRC, uint8_t *peerHelloHashHexString, size_t peerHelloHashHexStringLength) {
int i=0;
size_t i=0;
uint8_t *hexHashString = NULL;
size_t hexHashStringLength = peerHelloHashHexStringLength;
/* get channel context */
......
......@@ -34,11 +34,11 @@
test_t crypto_utils_tests[] = {
{ "zrtpKDF", test_zrtpKDF },
{ "CRC32", test_CRC32 },
{ "algo agreement", test_algoAgreement },
{ "context algo setter and getter", test_algoSetterGetter },
{ "adding mandatory crypto algorithms if needed", test_addMandatoryCryptoTypesIfNeeded }
TEST_NO_TAG("zrtpKDF", test_zrtpKDF),
TEST_NO_TAG("CRC32", test_CRC32),
TEST_NO_TAG("algo agreement", test_algoAgreement),
TEST_NO_TAG("context algo setter and getter", test_algoSetterGetter),
TEST_NO_TAG("adding mandatory crypto algorithms if needed", test_addMandatoryCryptoTypesIfNeeded)
};
test_suite_t crypto_utils_test_suite = {
......@@ -52,11 +52,11 @@ test_suite_t crypto_utils_test_suite = {
};
test_t packet_parser_tests[] = {
{ "Parse", test_parser },
{ "Parse hvi check fail", test_parser_hvi },
{ "Parse Exchange", test_parserComplete },
{ "State machine", test_stateMachine },
{ "ZRTP-hash", test_zrtphash }
TEST_NO_TAG("Parse", test_parser),
TEST_NO_TAG("Parse hvi check fail", test_parser_hvi),
TEST_NO_TAG("Parse Exchange", test_parserComplete),
TEST_NO_TAG("State machine", test_stateMachine),
TEST_NO_TAG("ZRTP-hash", test_zrtphash)
};
test_suite_t packet_parser_test_suite = {
......
......@@ -41,7 +41,7 @@ void bzrtp_message(const char *fmt, ...) {
void printHex(char *title, uint8_t *data, uint32_t length) {
if (verbose) {
int i;
uint32_t i;
printf ("%s : ", title);
for (i=0; i<length; i++) {
printf ("0x%02x, ", data[i]);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment