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

laforge gerrit-no-reply at lists.osmocom.org
Sun Jul 21 10:30:12 UTC 2019


laforge has submitted this change and it was merged. ( 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(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, approved



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-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: laforge <laforge at gnumonks.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190721/7bbc2d9c/attachment.html>


More information about the gerrit-log mailing list