From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id 7737E248B4 for ; Wed, 4 Apr 2018 04:59:59 -0400 (EDT) Received: from turing.freelists.org ([127.0.0.1]) by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id q9wWa1v12Xcu for ; Wed, 4 Apr 2018 04:59:59 -0400 (EDT) Received: from smtp55.i.mail.ru (smtp55.i.mail.ru [217.69.128.35]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id E850622A81 for ; Wed, 4 Apr 2018 04:59:58 -0400 (EDT) Received: by smtp55.i.mail.ru with esmtpa (envelope-from ) id 1f3eGq-0005ZA-D7 for tarantool-patches@freelists.org; Wed, 04 Apr 2018 11:59:57 +0300 From: "Gleb" (Redacted sender "gleb-skiba" for DMARC) Subject: [tarantool-patches] [PATCH] Fix warnings Date: Wed, 4 Apr 2018 11:59:44 +0300 Message-Id: <1522832384-14155-1-git-send-email-gleb-skiba@mail.ru> Sender: tarantool-patches-bounce@freelists.org Errors-to: tarantool-patches-bounce@freelists.org Reply-To: tarantool-patches@freelists.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: tarantool-patches List-subscribe: List-owner: List-post: List-archive: To: tarantool-patches@freelists.org Ensure -Werror -Wall set for the whole src/. Fix warnings which have been find with -Werror and -Wall. Add new building target RelWithDebInfoWError. Change building target on RelWithDebInfoWError in Ci. Fixes #3238 --- -Issue from https://github.com/tarantool/tarantool/issues/3238. -Source from https://github.com/tarantool/tarantool/tree/gh-3238-check-warnings. .travis.mk | 4 ++-- cmake/compiler.cmake | 9 ++++++++- src/box/sql/alter.c | 1 + src/box/sql/pragma.c | 4 ++-- src/box/vy_read_iterator.c | 2 +- src/box/xrow.c | 10 +++++++--- src/httpc.c | 2 +- src/lua/init.c | 3 ++- src/say.c | 10 ++++++---- 9 files changed, 30 insertions(+), 15 deletions(-) diff --git a/.travis.mk b/.travis.mk index 95713e1..2d24256 100644 --- a/.travis.mk +++ b/.travis.mk @@ -39,7 +39,7 @@ deps_ubuntu: lcov ruby tcl test_ubuntu: deps_ubuntu - cmake . -DCMAKE_BUILD_TYPE=RelWithDebInfo + cmake . -DCMAKE_BUILD_TYPE=RelWithDebInfoWError make -j8 cd test && /usr/bin/python test-run.py -j -1 @@ -49,7 +49,7 @@ deps_osx: pip install -r test-run/requirements.txt --user test_osx: deps_osx - cmake . -DCMAKE_BUILD_TYPE=RelWithDebInfo + cmake . -DCMAKE_BUILD_TYPE=RelWithDebInfoWError # Increase the maximum number of open file descriptors on macOS sudo sysctl -w kern.maxfiles=20480 || : sudo sysctl -w kern.maxfilesperproc=20480 || : diff --git a/cmake/compiler.cmake b/cmake/compiler.cmake index 05d33ab..6ec5f93 100644 --- a/cmake/compiler.cmake +++ b/cmake/compiler.cmake @@ -251,12 +251,19 @@ macro(enable_tnt_compile_flags) add_definitions("-D__STDC_LIMIT_MACROS=1") add_definitions("-D__STDC_CONSTANT_MACROS=1") - # Only add -Werror if it's a debug build, done by developers. + # Only add -Werror if it's a Debug or + # RelWithDebInfoWError build, done by developers. # Release builds should not cause extra trouble. if ((${CMAKE_BUILD_TYPE} STREQUAL "Debug") AND HAVE_STD_C11 AND HAVE_STD_CXX11) add_compile_flags("C;CXX" "-Werror") endif() + + if ((${CMAKE_BUILD_TYPE} STREQUAL "RelWithDebInfoWError") + AND HAVE_STD_C11 AND HAVE_STD_CXX11) + add_compile_flags("C;CXX" "-Werror") + message("Trouble Release") + endif() endmacro(enable_tnt_compile_flags) if (HAVE_OPENMP) diff --git a/src/box/sql/alter.c b/src/box/sql/alter.c index a52ba8d..ea4e195 100644 --- a/src/box/sql/alter.c +++ b/src/box/sql/alter.c @@ -157,6 +157,7 @@ sqlite3AlterFinishAddColumn(Parse * pParse, Token * pColDef) if (pParse->nErr || db->mallocFailed) return; assert(v != 0); + (void) v; pNew = pParse->pNewTable; assert(pNew); diff --git a/src/box/sql/pragma.c b/src/box/sql/pragma.c index 23b4c73..1a6314a 100644 --- a/src/box/sql/pragma.c +++ b/src/box/sql/pragma.c @@ -257,7 +257,6 @@ sqlite3Pragma(Parse * pParse, Token * pId, /* First part of [schema.]id field */ char *zLeft = 0; /* Nul-terminated UTF-8 string */ char *zRight = 0; /* Nul-terminated UTF-8 string , or NULL */ char *zTable = 0; /* Nul-terminated UTF-8 string or NULL */ - int rc; /* return value form SQLITE_FCNTL_PRAGMA */ sqlite3 *db = pParse->db; /* The database connection */ Vdbe *v = sqlite3GetVdbe(pParse); /* Prepared statement */ const PragmaName *pPragma; /* The pragma */ @@ -521,8 +520,9 @@ sqlite3Pragma(Parse * pParse, Token * pId, /* First part of [schema.]id field */ box_tuple_t *tuple; box_iterator_t* iter; iter = box_index_iterator(space_id, 0,ITER_ALL, key_buf, key_end); - rc = box_iterator_next(iter, &tuple); + int rc = box_iterator_next(iter, &tuple); assert(rc==0); + (void) rc; for (i = 0; tuple!=NULL; i++, box_iterator_next(iter, &tuple)){ /* 1 is name field number */ const char *str = tuple_field_cstr(tuple, 1); diff --git a/src/box/vy_read_iterator.c b/src/box/vy_read_iterator.c index a265f58..93681a8 100644 --- a/src/box/vy_read_iterator.c +++ b/src/box/vy_read_iterator.c @@ -552,7 +552,7 @@ vy_read_iterator_next_lsn(struct vy_read_iterator *itr, struct tuple **ret) { uint32_t i; bool unused; - struct vy_read_src *src; + struct vy_read_src *src = NULL; assert(itr->curr_stmt != NULL); assert(itr->curr_src < itr->skipped_src); diff --git a/src/box/xrow.c b/src/box/xrow.c index 3ef3d82..d2a0e02 100644 --- a/src/box/xrow.c +++ b/src/box/xrow.c @@ -301,6 +301,7 @@ iproto_reply_vclock(struct obuf *out, uint64_t sync, uint32_t schema_version, size - IPROTO_HEADER_LEN); char *ptr = obuf_alloc(out, size); + (void) ptr; assert(ptr == buf); return 0; } @@ -339,9 +340,12 @@ iproto_write_error(int fd, const struct error *e, uint32_t schema_version, schema_version, sizeof(body) + msg_len); body.v_data_len = mp_bswap_u32(msg_len); - (void) write(fd, header, sizeof(header)); - (void) write(fd, &body, sizeof(body)); - (void) write(fd, e->errmsg, msg_len); + ssize_t r1 = write(fd, header, sizeof(header)); + ssize_t r2 = write(fd, &body, sizeof(body)); + ssize_t r3 = write(fd, e->errmsg, msg_len); + (void) r1; + (void) r2; + (void) r3; } int diff --git a/src/httpc.c b/src/httpc.c index 633e688..a720cf3 100644 --- a/src/httpc.c +++ b/src/httpc.c @@ -249,7 +249,7 @@ httpc_set_low_speed_limit(struct httpc_request *req, long low_speed_limit) void httpc_set_verbose(struct httpc_request *req, bool curl_verbose) { - curl_easy_setopt(req->curl_request.easy, CURLOPT_VERBOSE, curl_verbose); + curl_easy_setopt(req->curl_request.easy, CURLOPT_VERBOSE, (long) curl_verbose); } void diff --git a/src/lua/init.c b/src/lua/init.c index 76e978c..89765e5 100644 --- a/src/lua/init.c +++ b/src/lua/init.c @@ -279,7 +279,8 @@ tarantool_lua_setpaths(struct lua_State *L) { const char *home = getenv("HOME"); char cwd[PATH_MAX] = {'\0'}; - getcwd(cwd, sizeof(cwd)); + char *buf = getcwd(cwd, sizeof(cwd)); + (void) buf; lua_getglobal(L, "package"); int top = lua_gettop(L); diff --git a/src/say.c b/src/say.c index b92514d..0d2fab2 100644 --- a/src/say.c +++ b/src/say.c @@ -853,7 +853,7 @@ say_default(int level, const char *filename, int line, const char *error, line, error, format, ap); if (level == S_FATAL && log_default->fd != STDERR_FILENO) { ssize_t r = write(STDERR_FILENO, buf, total); - (void) r; /* silence gcc warning */ + (void) r; /* silence gcc warning */ } va_end(ap); @@ -1040,13 +1040,15 @@ log_vsay(struct log *log, int level, const char *filename, int line, break; case SAY_LOGGER_SYSLOG: write_to_syslog(log, total); - if (level == S_FATAL && log->fd != STDERR_FILENO) - (void) write(STDERR_FILENO, buf, total); + if (level == S_FATAL && log->fd != STDERR_FILENO) { + ssize_t r = write(STDERR_FILENO, buf, total); + (void) r; /* silence gcc warning */ + } break; case SAY_LOGGER_BOOT: { ssize_t r = write(STDERR_FILENO, buf, total); - (void) r; /* silence gcc warning */ + (void) r; /* silence gcc warning */ break; } default: -- 2.7.4