Attention is currently required from: neels.
2 comments:
Patchset:
FYI, this patch fell out of my work some months ago, which is now probably obsolete. […]
I'm fine with merging this, I know several places where we are actually generating a string in a stack allocated buf then passing it to this API.
I'd welcome though if you also add to this patch the counterpart for osmo_stream_srv (I think we can skip it for osmo_stream_srv_link, since usually there's no multiple of those which needs string generation).
File src/stream_cli.c:
Patch Set #1, Line 674: osmo_stream_cli_set_name_f(cli, "%s", name);
I'd rather duplicate a couple lines than going through the whole vararg parsing here....
To view, visit change 39310. To unsubscribe, or for help writing mail filters, visit settings.