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/.
Harald Welte gerrit-no-reply at lists.osmocom.orgHarald Welte has posted comments on this change. ( https://gerrit.osmocom.org/12317 ) Change subject: ctrl: pass config struct to all simple_ctrl_*() ...................................................................... Patch Set 1: Code-Review-1 (1 comment) https://gerrit.osmocom.org/#/c/12317/1/src/simple_ctrl.c File src/simple_ctrl.c: https://gerrit.osmocom.org/#/c/12317/1/src/simple_ctrl.c@243 PS1, Line 243: const struct ctrl_cfg *cfg, struct simple_ctrl_handle *sch I'm sorry, but this is absolutely not an improvement to me. If you need the config from the handle, make the handle point to the config. Adding one extra argument to almost each function in the entire module is not an improvement. -- To view, visit https://gerrit.osmocom.org/12317 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-sysmon Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I783dda27dfc5fd57401d971b2e970ede0efc7b2c Gerrit-Change-Number: 12317 Gerrit-PatchSet: 1 Gerrit-Owner: Max <msuraev at sysmocom.de> Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Comment-Date: Fri, 14 Dec 2018 15:15:15 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: Yes -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20181214/fd8a6b20/attachment.htm>