pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmo-netif/+/30190 )

Change subject: osmux: Drop noop OR during assignment
......................................................................

osmux: Drop noop OR during assignment

osmuxh->rtp_m is zeroed at that point, it makes no sense to bit-OR at
that point.

Change-Id: I09c920035b1aa1af139a2a63f7cb4a3390bb9d36
---
M src/osmux_input.c
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/src/osmux_input.c b/src/osmux_input.c
index 866588f..a50a179 100644
--- a/src/osmux_input.c
+++ b/src/osmux_input.c
@@ -154,7 +154,7 @@
                                                      sizeof(struct osmux_hdr));
                osmuxh->ft = OSMUX_FT_VOICE_AMR;
                osmuxh->ctr = 0;
-               osmuxh->rtp_m = osmuxh->rtp_m || state->rtph->marker;
+               osmuxh->rtp_m = state->rtph->marker;
                osmuxh->seq = state->circuit->seq++;
                osmuxh->circuit_id = state->circuit->ccid;
                osmuxh->amr_ft = state->amrh->ft;

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

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I09c920035b1aa1af139a2a63f7cb4a3390bb9d36
Gerrit-Change-Number: 30190
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to