[MERGED] libosmocore[master]: lapd_test.c: Convert compilation warning to level message

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
Wed Jun 21 13:56:38 UTC 2017


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

Change subject: lapd_test.c: Convert compilation warning to level message
......................................................................


lapd_test.c: Convert compilation warning to level message

This way we clean the build of warnings completely and we can enable
-Werror but we still see this FIXME message while compiling.

Change-Id: I24c11226cd5bb5b6032f4ece7d1a0695dc82f0dd
---
M tests/lapd/lapd_test.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Max: Looks good to me, but someone else must approve
  Neels Hofmeyr: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/tests/lapd/lapd_test.c b/tests/lapd/lapd_test.c
index e322314..0dccaa4 100644
--- a/tests/lapd/lapd_test.c
+++ b/tests/lapd/lapd_test.c
@@ -358,7 +358,7 @@
 
 		/* Verify the added RSL_IE_L3_INFO but we have a bug here */
 		OSMO_ASSERT(msg->data[6] == RSL_IE_L3_INFO);
-		#warning "RSL_IE_L3_INFO 16 bit length is wrong"
+		#pragma message ("RSL_IE_L3_INFO 16 bit length is wrong")
 		/* This should be okay but it is actually 0x0, 0x9c on ia-32 */
 		/* OSMO_ASSERT(msg->data[7] == 0x0 && msg->data[8] == 0x0); */
 	} else if (state->ms_read == 1) {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I24c11226cd5bb5b6032f4ece7d1a0695dc82f0dd
Gerrit-PatchSet: 2
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>



More information about the gerrit-log mailing list