[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