diff options
| author | Dan Williams <dcbw@redhat.com> | 2019-04-02 09:16:12 -0500 |
|---|---|---|
| committer | Dan Williams <dcbw@redhat.com> | 2019-04-02 10:48:40 -0500 |
| commit | 88fac7f14a8898a7856a6868bbd94d4cf576723a (patch) | |
| tree | 700753959d987c211d00cbf675af51d1eaadcceb /src/mm-broadband-bearer.c | |
| parent | 093174c84da265059eda2591cf79593abb567f3a (diff) | |
build: fix -Wunused-but-set-variable support and handle clang better
By default the build used -Wno-unused-but-set-variable but that's
clearly not what we want. Fix those problems.
When using clang, the compiler doesn't error on unknown compile
options, so let's add -Werror to the checking too.
CC='clang'
configure:3408: checking for gcc
configure:3435: result: clang
...
configure:17644: checking whether gcc understands -Wno-unused-but-set-variable
configure:17657: clang -c -Wall -std=gnu89 -g -O2 -Wmissing-declarations -Wmissing-prototypes -Wdeclaration-after-statement -Wstrict-prototypes -Wno-unused-parameter -Wno-sign-compare -Wno-deprecated-declarations -Wno-unused-but-set-variable conftest.c >&5
warning: unknown warning option '-Wno-unused-but-set-variable'; did you mean '-Wno-unused-const-variable'? [-Wunknown-warning-option]
1 warning generated.
configure:17657: $? = 0
configure:17666: result: yes
Reported-by: Torsten Hilbrich <torsten.hilbrich@secunet.com>
Diffstat (limited to 'src/mm-broadband-bearer.c')
| -rw-r--r-- | src/mm-broadband-bearer.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/mm-broadband-bearer.c b/src/mm-broadband-bearer.c index 16acb2b1..aa6654e4 100644 --- a/src/mm-broadband-bearer.c +++ b/src/mm-broadband-bearer.c @@ -261,10 +261,8 @@ dial_cdma_ready (MMBaseModem *modem, static void cdma_connect_context_dial (GTask *task) { - MMBroadbandBearer *self; DetailedConnectContext *ctx; - self = g_task_get_source_object (task); ctx = g_task_get_task_data (task); mm_base_modem_at_command_full (ctx->modem, |