[FFmpeg-devel] [PATCH] avutil/avstring: add av_strreplace API into avstring

Steven Liu lingjiujianke at gmail.com
Sat Apr 1 05:58:24 EEST 2017


2017-03-31 20:11 GMT+08:00 Steven Liu <lingjiujianke at gmail.com>:

>
>
> 2017-03-31 17:51 GMT+08:00 Nicolas George <george at nsup.org>:
>
>> Le decadi 10 germinal, an CCXXV, Steven Liu a écrit :
>> > refer to: http://creativeandcritical.net/str-replace-c
>> > add av_strreplace API for replace string operations.
>> >
>> > Signed-off-by: Steven Liu <lq at chinaffmpeg.org>
>> > ---
>> >  libavutil/avstring.c | 77 ++++++++++++++++++++++++++++++
>> ++++++++++++++++++++++
>> >  libavutil/avstring.h |  5 ++++
>> >  2 files changed, 82 insertions(+)
>>
>> It looks like it is duplicating a lot of functionality already
>> implemented in BPrint. I suggest you try implementing it using BPrint
>> instead.
>>
> I think just similar, not duplicating, and the simple functionality use
> BPrint maybe too complex.
>
>>
>> Regards,
>>
>> --
>>   Nicolas George
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>
> Pushed



Thanks!


More information about the ffmpeg-devel mailing list