Change in ...libosmocore[master]: tdef: remove bogus OSMO_ASSERT(unsigned long >= 0)

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/.

laforge gerrit-no-reply at lists.osmocom.org
Sun Jul 21 07:25:41 UTC 2019


laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmocore/+/14857


Change subject: tdef: remove bogus OSMO_ASSERT(unsigned long >= 0)
......................................................................

tdef: remove bogus OSMO_ASSERT(unsigned long >= 0)

Change-Id: I7a544d2d43b83135def296674f777e48fe5fd80a
Closes: CID#190866
---
M src/tdef.c
1 file changed, 0 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/57/14857/1

diff --git a/src/tdef.c b/src/tdef.c
index 9d5d736..3cfb17c 100644
--- a/src/tdef.c
+++ b/src/tdef.c
@@ -187,7 +187,6 @@
 {
 	const struct osmo_tdef *t = osmo_tdef_get_entry((struct osmo_tdef*)tdefs, T);
 	if (!t) {
-		OSMO_ASSERT(val_if_not_present >= 0);
 		return val_if_not_present;
 	}
 	return osmo_tdef_round(t->val, t->unit, as_unit);

-- 
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14857
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I7a544d2d43b83135def296674f777e48fe5fd80a
Gerrit-Change-Number: 14857
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge at gnumonks.org>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190721/0e75592b/attachment.htm>


More information about the gerrit-log mailing list