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/.
neels gerrit-no-reply at lists.osmocom.orgneels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/15847 ) Change subject: log: RANAP encode: use RANAP message names instead of BSSAP ...................................................................... Patch Set 1: (1 comment) https://gerrit.osmocom.org/c/osmo-msc/+/15847/1/src/libmsc/ran_msg_iu.c File src/libmsc/ran_msg_iu.c: https://gerrit.osmocom.org/c/osmo-msc/+/15847/1/src/libmsc/ran_msg_iu.c@402 PS1, Line 402: LOG_RAN_IU_ENC(caller_fi, LOGL_DEBUG, "DirectTransfer\n"); > I'm not following this commit. […] When I wrote the RAN message de/encoding, I decided to put the string constants directly in the de/encoding functions; I only need to add 5 strings here, for the encoding; the decoding already has individual strings in each message type's decoding code path. We already have 1:1 separate code paths for each message, it doesn't make much sense to again iterate a value string array. An underlying idea was that each code path could add some details, like this DTAP could also log which enclosed DTAP message type is being encoded. But I didn't get around to that... (I also have some general dislike of the value_string design, because it has to iterate arrays for each and every value, but you can ignore that) -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/15847 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Ib0e0630d775a28958ea86802f70cbeec07087f91 Gerrit-Change-Number: 15847 Gerrit-PatchSet: 1 Gerrit-Owner: neels <nhofmeyr at sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: neels <nhofmeyr at sysmocom.de> Gerrit-CC: pespin <pespin at sysmocom.de> Gerrit-Comment-Date: Thu, 24 Oct 2019 22:36:36 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin at sysmocom.de> Gerrit-MessageType: comment -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20191024/e38f0b51/attachment.htm>