libosmo-netif[master]: osmux: Re-write osmux_snprintf

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/.

Pablo Neira Ayuso gerrit-no-reply at lists.osmocom.org
Mon Aug 21 13:28:32 UTC 2017


Patch Set 1:

size is the original buffer size, number of bytes you can fit into the buffer you provide.

len is the number of byte that has been written into the buffer.

offset is where you should continue to append more bytes.

If you want to retain snprintf semantics, ie. return the number of bytes that would have been written, then you need to keep size and offset, because offset should not ever go over the boundary.

Retaining snprintf semantics can be good to log how many bytes are missing in the buffer you provide.

If you don't want snprintf semantics, then you can remove len and use offset in any case.

-- 
To view, visit https://gerrit.osmocom.org/3537
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I695771d099833842db37a415b636035d17f1bba7
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Pablo Neira Ayuso <pablo at gnumonks.org>
Gerrit-HasComments: No



More information about the gerrit-log mailing list