Change in libosmocore[master]: stat, rate_ctr: Allow setting group name and use it at report time

fixeria gerrit-no-reply at lists.osmocom.org
Tue Jun 1 21:00:51 UTC 2021


fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/24467 )

Change subject: stat,rate_ctr: Allow setting group name and use it at report time
......................................................................


Patch Set 2: Code-Review+1

I am really happy to see this change because it was only a few days ago when I was puzzled over picking the proper index values for the shadow timeslots. What we currently do is: (TRXN + 1) * 10 + TN, and I find this approach ugly. Giving my CR+1 to in hope to see it merged soon.


-- 
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/24467
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I0dc510783dd9ae8436dae8005a7b3330e80d36f3
Gerrit-Change-Number: 24467
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: osmith <osmith at sysmocom.de>
Gerrit-Comment-Date: Tue, 01 Jun 2021 21:00:51 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210601/b46a67ea/attachment.htm>


More information about the gerrit-log mailing list