Patch Set 8: Code-Review-1 (1 comment)
https://gerrit.osmocom.org/#/c/4413/8/src/bts.cpp File src/bts.cpp: Line 165: rlc_late_block(bts_data()->ctrs); I'm sorry, I really think it's ugly to pass around a counter array to various functions that don't take any other argument. Either the counters can be resolved from some larger struct/object of which the counters are a member, but which makes actual sense to the related function, or the respective function like rlc_late_block() would have to be a method of an object which has access to the counters? -- To view, visit https://gerrit.osmocom.org/4413 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I795631752af3b9077a846c3123217e1deda52e86 Gerrit-PatchSet: 8 Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Owner: Max <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-HasComments: Yes
