[PATCH] libosmocore[master]: sms_test.c: Remove test of deprecated wrapper function

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 Jun 18 08:43:28 UTC 2017


Review at  https://gerrit.osmocom.org/2950

sms_test.c: Remove test of deprecated wrapper function

The deprecated function is just a wrapper, and it triggers a compilation
warning.

Change-Id: Iaac701bdf0c0e66c343f68cc31d82550a024ef57
---
M tests/sms/sms_test.c
M tests/sms/sms_test.ok
2 files changed, 0 insertions(+), 27 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/50/2950/1

diff --git a/tests/sms/sms_test.c b/tests/sms/sms_test.c
index 776c45a..efbdf7e 100644
--- a/tests/sms/sms_test.c
+++ b/tests/sms/sms_test.c
@@ -288,16 +288,6 @@
 
 	/* test 7-bit encoding */
 	for (i = 0; i < ARRAY_SIZE(test_encode); ++i) {
-		/* Test legacy function (return value only) */
-		septet_length = gsm_7bit_encode(coded,
-						(const char *) test_encode[i].input);
-		printf("Legacy encode case %d: "
-		       "septet length %d (expected %d)\n"
-		       , i
-		       , septet_length, test_encode[i].expected_septet_length
-		      );
-		OSMO_ASSERT (septet_length == test_encode[i].expected_septet_length);
-
 		/* Test new function */
 		memset(coded, 0x42, sizeof(coded));
 		septet_length = gsm_7bit_encode_n(coded, sizeof(coded),
@@ -377,15 +367,6 @@
 
 	/* test 7-bit decoding */
 	for (i = 0; i < ARRAY_SIZE(test_decode); ++i) {
-		/* Test legacy function (return value only) */
-		if (!test_decode[i].ud_hdr_ind) {
-			nchars = gsm_7bit_decode(result, test_decode[i].input,
-						 test_decode[i].expected_septet_length);
-			printf("Legacy decode case %d: "
-			       "return value %d (expected %d)\n",
-			       i, nchars, test_decode[i].expected_septet_length);
-		}
-
 		/* Test new function */
 		memset(result, 0x42, sizeof(result));
 		nchars = gsm_7bit_decode_n_hdr(result, sizeof(result), test_decode[i].input,
diff --git a/tests/sms/sms_test.ok b/tests/sms/sms_test.ok
index fa536ea..a71567d 100644
--- a/tests/sms/sms_test.ok
+++ b/tests/sms/sms_test.ok
@@ -1,19 +1,11 @@
 SMS testing
-Legacy encode case 0: septet length 9 (expected 9)
 Encode case 0: Octet length 8 (expected 8, computed 8), septet length 9 (expected 9)
-Legacy encode case 1: septet length 41 (expected 41)
 Encode case 1: Octet length 36 (expected 36, computed 36), septet length 41 (expected 41)
-Legacy encode case 2: septet length 39 (expected 39)
 Encode case 2: Octet length 35 (expected 35, computed 35), septet length 39 (expected 39)
-Legacy encode case 3: septet length 40 (expected 40)
 Encode case 3: Octet length 35 (expected 35, computed 35), septet length 40 (expected 40)
-Legacy decode case 0: return value 9 (expected 9)
 Decode case 0: return value 9 (expected 9)
-Legacy decode case 1: return value 41 (expected 41)
 Decode case 1: return value 40 (expected 40)
-Legacy decode case 2: return value 39 (expected 39)
 Decode case 2: return value 31 (expected 31)
-Legacy decode case 3: return value 40 (expected 40)
 Decode case 3: return value 32 (expected 32)
 Decode case 4: return value 153 (expected 153)
 Decode case 5: return value 40 (expected 40)

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iaac701bdf0c0e66c343f68cc31d82550a024ef57
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>



More information about the gerrit-log mailing list