Attention is currently required from: pespin.
neels has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bsc/+/31607
)
Change subject: bsc_vty.c write_msc(): fix weird printf format
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
any explanation what was that doing and ther ein first
place and why don't we want it?
the codec names like 'fr1' have numbers
that are supposed to be single digit. Now, the '%.1u' may seem like it is
enforcing a single digit, or something; There is obviously no printf format limiting the
amount of printed integer digits, because that concept makes no sense. The '%.1u'
format only applies to floating point numbers, limiting fractional digits *after* the
decimal dot. So it is completely weird and lost and confusing.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bsc/+/31607
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I856c2d620b89efcf3239186ef53c6941577dbccc
Gerrit-Change-Number: 31607
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 01 Mar 2023 22:10:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment