[FFmpeg-devel] ftp protocol support

[Lukasz M]

2013/5/16 Clément Bœsch <ubitux at gmail.com>:
> On Thu, May 16, 2013 at 06:31:03PM +0200, Lukasz M wrote:
>> > Did you look at the AVBPrint API? It might be relevant to use it
>> > eventually.
>>
>> You mean to use ABVprint API inside functions where commands are
>> currently created with snprintf?
>
> Yes that was the idea.
>
>> Will this change anything? I think it will be harder to maintain.
>>
>
> It was an innocent question, not a request. It can makes sense if you're
> dealing with ± large buffers and incrementally adding stuff to them. That
> might not be the case in your code, so feel free to ignore.
>

Updated patch in attachment. I skipped AVBPrintf remark.
Regarding valgrind, yes, I tested and no leaks detected.

Best Regards,
Lukasz Marek
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-FTP-protocol-support.patch
Type: application/octet-stream
Size: 24758 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130516/7bfdfae6/attachment.obj>