pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
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(-)

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 change 30190. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I09c920035b1aa1af139a2a63f7cb4a3390bb9d36
Gerrit-Change-Number: 30190
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged