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 <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to