[vlc-devel] [PATCH 2/3] Refactor RSS display string generation

[Daniel Glaas]

I additionally notices a bug in the generation of a scrolling title. It 
is fixed in the update attached patch.

Am 18.07.20 um 12:40 schrieb Daniel Glaas:
> 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.
>>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Refactor-RSS-display-string-generation.patch
Type: text/x-patch
Size: 3908 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20200721/bc0c170d/attachment.bin>