laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/29620 ) Change subject: msgb: do not use msgb_l4 instead of msgb_sms ...................................................................... msgb: do not use msgb_l4 instead of msgb_sms The macro msgb_sms() is basically an alias for msgb_l4(). Lets use msgb_l4() in msgb_l4len() instead of msgb_sms(). Change-Id: I90d4f5c07fbaadd9e022752a2c64c4855f0b4227 --- M include/osmocom/core/msgb.h 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, approved Jenkins Builder: Verified diff --git a/include/osmocom/core/msgb.h b/include/osmocom/core/msgb.h index fbf1742..1e2b023 100644 --- a/include/osmocom/core/msgb.h +++ b/include/osmocom/core/msgb.h @@ -184,7 +184,7 @@ static inline unsigned int msgb_l4len(const struct msgb *msgb) { OSMO_ASSERT(msgb->l4h); - return msgb->tail - (uint8_t *)msgb_sms(msgb); + return msgb->tail - (uint8_t *)msgb_l4(msgb); } /*! determine the length of the header -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/29620 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I90d4f5c07fbaadd9e022752a2c64c4855f0b4227 Gerrit-Change-Number: 29620 Gerrit-PatchSet: 2 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged