[MERGED] libsmpp34[master]: smpp34_dumpBuf.c: remove unused variable

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Pau Espin Pedrol gerrit-no-reply at lists.osmocom.org
Sun Sep 17 14:59:44 UTC 2017


Pau Espin Pedrol has submitted this change and it was merged.

Change subject: smpp34_dumpBuf.c: remove unused variable
......................................................................


smpp34_dumpBuf.c: remove unused variable

Fixes compilation warning below:
smpp34_dumpBuf.c:52:9: warning: variable ‘lefterror’ set but not used [-Wunused-but-set-variable]
     int lefterror = 0;

Change-Id: Iea8ff80d3e4d16aaca6e37d64593b50fa27c8cfb
---
M src/smpp34_dumpBuf.c
1 file changed, 0 insertions(+), 2 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/smpp34_dumpBuf.c b/src/smpp34_dumpBuf.c
index 77dfdba..dd0bad4 100644
--- a/src/smpp34_dumpBuf.c
+++ b/src/smpp34_dumpBuf.c
@@ -49,14 +49,12 @@
     int size;
     uint8_t ind = 3;
     uint8_t *buffer = NULL;
-    int lefterror = 0;
 
     size   = srcL;
     buffer = src;
 
     memset(smpp34_strerror, 0, sizeof(smpp34_strerror));
     ptrerror = smpp34_strerror;
-    lefterror = sizeof(smpp34_strerror);
 
     /* dump buffer character by character until size is reached */
     for(i = 0; i < size; i++){

-- 
To view, visit https://gerrit.osmocom.org/3964
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iea8ff80d3e4d16aaca6e37d64593b50fa27c8cfb
Gerrit-PatchSet: 1
Gerrit-Project: libsmpp34
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>



More information about the gerrit-log mailing list