fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmocore/+/14426
Change subject: protocol/gsm_04_08.h: do not check if unsigned is positive ...................................................................... protocol/gsm_04_08.h: do not check if unsigned is positive Change-Id: I6b486b52a3733d5fd5e8ba18acbc9374e2e8bd7e --- M include/osmocom/gsm/protocol/gsm_04_08.h 1 file changed, 3 insertions(+), 3 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/26/14426/1 diff --git a/include/osmocom/gsm/protocol/gsm_04_08.h b/include/osmocom/gsm/protocol/gsm_04_08.h index 16910c3..2be6ed3 100644 --- a/include/osmocom/gsm/protocol/gsm_04_08.h +++ b/include/osmocom/gsm/protocol/gsm_04_08.h @@ -719,7 +719,7 @@ */ static inline void gsm48_barr_acc(struct gsm48_rach_control *rach_control, unsigned int acc) { - OSMO_ASSERT(acc >= 0 && acc <= 15); + OSMO_ASSERT(acc <= 15); if (acc >= 8) rach_control->t2 |= (1 << (acc - 8)); else @@ -733,7 +733,7 @@ */ static inline void gsm48_allow_acc(struct gsm48_rach_control *rach_control, unsigned int acc) { - OSMO_ASSERT(acc >= 0 && acc <= 15); + OSMO_ASSERT(acc <= 15); if (acc >= 8) rach_control->t2 &= ~(1 << (acc - 8)); else @@ -748,7 +748,7 @@ */ static inline bool gsm48_acc_is_barred(struct gsm48_rach_control *rach_control, unsigned int acc) { - OSMO_ASSERT(acc >= 0 && acc <= 15); + OSMO_ASSERT(acc <= 15); if (acc >= 8) return (rach_control->t2 & (1 << (acc - 8))) != 0; return (rach_control->t3 & (1 << (acc))) != 0; -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14426 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I6b486b52a3733d5fd5e8ba18acbc9374e2e8bd7e Gerrit-Change-Number: 14426 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <axilira...@gmail.com> Gerrit-MessageType: newchange