[vlc-devel] [PATCH 2/3] Refactor RSS display string generation
Daniel Glaas
daniel.glaas at freenet.de
Sat Jul 18 12:40:33 CEST 2020
You mean that instead of snprintf() the function vlc_memstream() is used?
Ok, it was not my purpose of this patch to replace the snprintf() calls
with another functions. I just wanted to make it easier readable with
function parameters are passed into snprintf(). You think that if these
lines are touched, then the usage of vlc_memstream should be used?
Am 18.07.20 um 09:38 schrieb Rémi Denis-Courmont:
> Hi,
>
> Le lauantaina 18. heinäkuuta 2020, 5.05.14 EEST Daniel Glaas a écrit :
>> Refactor RSS display string generation
>>
>> The RSS display string generation contained lot of code duplications. With
>> commit, structures that are used several time are factor out to make the
>> final
>> snprintf() statements which write the text into the marquee buffer better
>> readable.
> Yes but we use vlc_memstream for that sort of things nowadays. It's far
> simpler and less error-prone.
>
More information about the vlc-devel
mailing list