[FFmpeg-devel] [PATCH v3] avutil/avstring: improve av_strreplace implement

Nicolas George george at nsup.org
Mon Apr 3 15:32:36 EEST 2017


Le quartidi 14 germinal, an CCXXV, Steven Liu a écrit :
> change name from av_strreplace to av_strireplace
> Use AVBprint to implement av_strireplace
> add av_strireplace test case TEST_STRIREPLACE
> 
> Signed-off-by: Steven Liu <lq at chinaffmpeg.org>
> ---
>  libavutil/avstring.c       | 76 +++++++---------------------------------------
>  libavutil/avstring.h       |  2 +-
>  libavutil/tests/avstring.c | 16 ++++++++++
>  3 files changed, 28 insertions(+), 66 deletions(-)

This version looks fine to me.

Maybe wait a little more for advice about the rename and the whole
thing.

Thanks.

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170403/2e7abf17/attachment.sig>


More information about the ffmpeg-devel mailing list