Attention is currently required from: msuraev. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/28992 )
Change subject: Add osmo_sockaddr_strs_to_str() ...................................................................... Patch Set 6: (4 comments) Patchset: PS6: It would be great if you explain where is this going to be used and the need for it. File include/osmocom/core/sockaddr_str.h: https://gerrit.osmocom.org/c/libosmocore/+/28992/comment/e28c28ce_7f0dc117 PS6, Line 63: #define OSMO_SOCKADDR_STR_FMT_ARGS(R) \ You could actually define these based on the ones below now. It actually makes more sense to swap the order of the 2 groups probably anyway. File src/sockaddr_str.c: https://gerrit.osmocom.org/c/libosmocore/+/28992/comment/518976a9_80b451ca PS6, Line 565: ret = snprintf(buf, rem, "("); FYI, you can have a look at using OSMO_STRBUF APIs, they are useful to implement stuff like done in this function. https://gerrit.osmocom.org/c/libosmocore/+/28992/comment/020c9318_daf05360 PS6, Line 578: OSMO_SOCKADDR_STR_NO_PORT_FMT_ARGS(sa_str[i]), after); maybe add a boolean print_ports to decide whether to add ports or not? -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/28992 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Ic0d7c08f669994e37a2314555ecac85d28c42c89 Gerrit-Change-Number: 28992 Gerrit-PatchSet: 6 Gerrit-Owner: msuraev <msur...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <pes...@sysmocom.de> Gerrit-Attention: msuraev <msur...@sysmocom.de> Gerrit-Comment-Date: Wed, 10 Aug 2022 14:01:33 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment