diff options
| author | Pau Espin Pedrol <pespin@sysmocom.de> | 2021-05-19 11:58:56 +0200 |
|---|---|---|
| committer | Pau Espin Pedrol <pespin@sysmocom.de> | 2021-05-19 11:58:57 +0200 |
| commit | 6e25119c18146bd3ef23e69444ee91016f3ddf20 (patch) | |
| tree | 52de862354ddd8062c765a3253d4e98389a8c6bf /src | |
| parent | c43570c3519d2c82ecb6a0323c6fe0a6e0efec57 (diff) | |
Clean false positive in newer GCC version checking guard of else clause
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
Diffstat (limited to 'src')
| -rw-r--r-- | src/egprs_rlc_compression.cpp | 7 |
1 files 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; } |