Attention is currently required from: fixeria. neels has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/30574 )
Change subject: gsm0808: add gsm0808_enc_speech_codec[_list]2() ...................................................................... Patch Set 2: (2 comments) File include/osmocom/gsm/gsm0808_utils.h: https://gerrit.osmocom.org/c/libosmocore/+/30574/comment/8d641b5b_b7ebdb34 PS1, Line 115: OSMO_DEPRECATED("use gsm0808_enc_speech_codec2() instead") > It's the first time I see such a suggestion/requirement. […] i usually read .h files, and spotting the difference between two function versions goes a lot quicker when something explains it. IMO adding a few words can only improve it, but you do also explain in the API doc. so up to you. File src/gsm/gsm0808_utils.c: https://gerrit.osmocom.org/c/libosmocore/+/30574/comment/b89ce137_b2ea32c4 PS1, Line 299: \a > This is not directly related to this patch, but could you explain why? see the post linked at https://osmocom.org/projects/cellular-infrastructure/wiki/Guidelines_for_API_documentation#Parameters good that i read this again, '\a' is actually not wrong, it just puts the next word in italics. it's just my personal opinion that we should not use \a, for KISS. -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/30574 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I199ffa0ba4a64813238519178155dfc767aa3975 Gerrit-Change-Number: 30574 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <[email protected]> Gerrit-CC: neels <[email protected]> Gerrit-Attention: fixeria <[email protected]> Gerrit-Comment-Date: Tue, 13 Dec 2022 21:58:50 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <[email protected]> Comment-In-Reply-To: fixeria <[email protected]> Gerrit-MessageType: comment
