Attention is currently required from: pespin. neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/30125 )
Change subject: rtp_stream: allow multiple codecs / use codec filter from Assignment ...................................................................... Patch Set 4: (1 comment) Patchset: PS4: > I have the feeling this commit is still renaming and changing logic both > things at the same time, wh […] i had this sort of separate once, but here we are now. my opinion now is that it is better to rename "codec" to "codecs" at the same time when it changes to multiple codecs involved where there was only one before: this shows in one patch all the places that are affected by the change, e.g. 'git blame' will hint at this patch. Especially when adding the function ".._add_one_codec()" it is better to have the other function named "..._add_codecs()", not ".._add_codec()". -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/30125 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: If9c67b298b30f893ec661f84c9fc622ad01b5ee5 Gerrit-Change-Number: 30125 Gerrit-PatchSet: 4 Gerrit-Owner: neels <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Comment-Date: Tue, 07 Mar 2023 15:51:46 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <[email protected]> Gerrit-MessageType: comment
