Patch Set 1: (1 comment)
https://gerrit.osmocom.org/#/c/4407/1/src/utils.c File src/utils.c: Line 375: size_t ret = src ? strlen(src) : 0; > Why not simply return here right away? I thought this is the preferred styl I guess it's a good idea to still zero-terminate 'dst' even if 'src' is NULL. -- To view, visit https://gerrit.osmocom.org/4407 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ieba7ba262ace2e370a4b9a550b3131fb13f07413 Gerrit-PatchSet: 1 Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Max <[email protected]> Gerrit-HasComments: Yes
