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/.
Max gerrit-no-reply at lists.osmocom.orgMax has posted comments on this change. ( https://gerrit.osmocom.org/13243 ) Change subject: TBF: update MCS counters ...................................................................... Patch Set 1: (1 comment) This change is ready for review. https://gerrit.osmocom.org/#/c/13243/1/src/tbf.h File src/tbf.h: https://gerrit.osmocom.org/#/c/13243/1/src/tbf.h@651 PS1, Line 651: void update_coding_scheme_counter_dl(enum CodingScheme cs); > Would be nice to have the enum named GprsCodingSCheme It includes both Gprs and Egprs coding schemes so I prefer smth more generic. > Can the class be dropped? Not yet unfortunately. Besides, I don't want to introduce changes which are neither immediate necessity nor convenience for the task at hand (fixing IA RO/TAI), and that class is used all over the place. -- To view, visit https://gerrit.osmocom.org/13243 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I763136c2f356d63aa3d28d09c57fd5faf5336258 Gerrit-Change-Number: 13243 Gerrit-PatchSet: 1 Gerrit-Owner: Max <msuraev at sysmocom.de> Gerrit-Reviewer: Max <msuraev at sysmocom.de> Gerrit-CC: Pau Espin Pedrol <pespin at sysmocom.de> Gerrit-Comment-Date: Wed, 13 Mar 2019 19:22:20 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: No -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190313/2ac38b5b/attachment.htm>