[vlc-devel] [PATCH] Add open_memstream replacement
Derek Buitenhuis
derek.buitenhuis at gmail.com
Thu Aug 25 14:58:57 CEST 2016
On 8/25/2016 1:53 PM, Rémi Denis-Courmont wrote:
>> To be clear, this is kinda what I mean (rad the disclaimed at the top):
>>
>> https://gist.github.com/dwbuiten/66f007058d27bf04663c362732f9be2a
>
> That´s awful.
It's a matter of priorities and taste. Furthermore, that's not even a
legitimate technical review of the idea. It's akin to saying "I don't
like it, so no."
>> That kind of thing, in my mind, is preferable to a 'convenient' solution
>> that is less portable.
>
> I utterly disagree. And FWIW, even the kernel (!) has introduced convenience
> helpers for buffer formatting - for pretty much the same purposes/reasons.
I can only agree to disagree. If it's the difference between using the above,
or not supporting Windows, a major OS, *at all*, in this function, I'd
choose the string-based book-keeping-filled code every day of the week.
For what it's worth, I do not intend to argue this further, as stated before,
it's not my place. I just wanted to voice an alternative idea and opinion
for others to gauge.
Consider it my last mail on the subject
Cheers,
- Derek
More information about the vlc-devel
mailing list