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

[Steven Liu]

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
>