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 Hofmeyr gerrit-no-reply at lists.osmocom.orgPatch Set 1: > Why returning int if it's always positive value? Better use size_t > or other unsigned type. I considered it, but then decided to keep int, as in the original implementation from some C file in libosmo-abis: firstly, practical fact: if we want to count the len by iterating an llist that has more than INT_MAX elements, something is seriously wrong. Coding convenience: it is painful / dangerous to compare signed and unsigned ints. We usually use int for iterators, indexes, list lengths, so it's convenient to return an int here. Also enables the caller to compare with negative values -- no idea why we'd want that, but is conceivable that it might be useful. So I agree that it is wrong in the sense of type logic to return a signed value, but if we return unsigned, we need to add type casts and checks or use unsigned int for iterators everywhere... (which is a bit like saying "then we need to use the correct type for iterators everywhere") unsigned is more "correct", but I wasn't certain enough to change it from the int return type I found. What do other reviewers think? -- To view, visit https://gerrit.osmocom.org/1563 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic49adc7a346f5722bf624d7d3b4a735e4220ae15 Gerrit-PatchSet: 1 Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Max <msuraev at sysmocom.de> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de> Gerrit-HasComments: No