Attention is currently required from: pespin. neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/29864 )

Change subject: mgw: rx_rtp(): reorder checks and handlings ......................................................................

Patch Set 1:

(2 comments)

Commit Message:

https://gerrit.osmocom.org/c/osmo-mgw/+/29864/comment/b8296e75_934230b2 PS1, Line 9: Let's first validate the origin of the message, then the content of the (you're using "Let's" regularly, but from https://osmocom.org/projects/cellular-infrastructure/wiki/Coding_standards : "You should stick to the imperative form -- e.g. "add foo", "fix bar", "drop baz" -- and avoid empty phrasing like "this patch implements". This is shortest to write and, more importantly, most efficient to read for reviewers.")

Patchset:

PS1: I am missing the point of the patch, can you explain the motivation / the effects in the commit log?