This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
pespin gerrit-no-reply at lists.osmocom.orgpespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/26132 ) Change subject: vty: show tbf: Drop unneeded check for non-null ms ...................................................................... vty: show tbf: Drop unneeded check for non-null ms Since a while a go, a TBF is guaranteed to always have a MS assigned. Hence, there's no point in checking it. Change-Id: I89e062432ac671c73731ce68c889aeb5e24277f5 --- M src/pcu_vty_functions.cpp 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved diff --git a/src/pcu_vty_functions.cpp b/src/pcu_vty_functions.cpp index fb7897d..c2cc950 100644 --- a/src/pcu_vty_functions.cpp +++ b/src/pcu_vty_functions.cpp @@ -62,7 +62,7 @@ tbf->first_ts, tbf->first_common_ts, tbf->control_ts, tbf->ms_class(), - tbf->ms() ? ms_egprs_ms_class(tbf->ms()) : -1, + ms_egprs_ms_class(tbf->ms()), VTY_NEWLINE); vty_out(vty, " TS_alloc="); for (int i = 0; i < 8; i++) { -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/26132 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: I89e062432ac671c73731ce68c889aeb5e24277f5 Gerrit-Change-Number: 26132 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin at sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de> Gerrit-Reviewer: pespin <pespin at sysmocom.de> Gerrit-MessageType: merged -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211108/3d795e45/attachment.htm>