From 6e25119c18146bd3ef23e69444ee91016f3ddf20 Mon Sep 17 00:00:00 2001 From: Pau Espin Pedrol Date: Wed, 19 May 2021 11:58:56 +0200 Subject: [PATCH] Clean false positive in newer GCC version checking guard of else clause MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Got this today with newer gcc (11.1.0) after system upgrade: egprs_rlc_compression.cpp:693:9: error: this ‘else’ clause does not guard... [-Werror=misleading-indentation] The indentation was indeed wrong, provoking a warning in GCC. From code flow point of view, however, the previous state was fine too, so no logical change is involved in this commit. Change-Id: I37bfc8e85daaabbbf10dfd907b305e3e0ec31863 --- src/egprs_rlc_compression.cpp | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/egprs_rlc_compression.cpp b/src/egprs_rlc_compression.cpp index 8f4fd872..fae529c3 100644 --- a/src/egprs_rlc_compression.cpp +++ b/src/egprs_rlc_compression.cpp @@ -690,8 +690,7 @@ int egprs_compress::compress_rbb( if (clen >= uclen) /* No Gain is observed, So no need to compress */ return 0; - else - LOGP(DRLCMACUL, LOGL_DEBUG, "CRBB bitmap = %s\n", osmo_hexdump(crbb_vec->data, (crbb_vec->cur_bit+7)/8)); - /* Add compressed bitmap to final buffer */ - return 1; + LOGP(DRLCMACUL, LOGL_DEBUG, "CRBB bitmap = %s\n", osmo_hexdump(crbb_vec->data, (crbb_vec->cur_bit+7)/8)); + /* Add compressed bitmap to final buffer */ + return 1; }