[FFmpeg-devel] [PATCH v3] avutil/avstring: improve av_strreplace implement
James Almer
jamrial at gmail.com
Wed Apr 5 19:03:40 EEST 2017
On 4/5/2017 11:05 AM, Steven Liu wrote:
> 2017-04-03 20:32 GMT+08:00 Nicolas George <george at nsup.org>:
>
>> 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
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>>
> ping
You can't rename the function. It's already in the 3.3 branch.
This is one of the reasons why giving people enough time for reviews
is a must before pushing new public symbols.
The rest should be ok since Nicolas reviewed it.
More information about the ffmpeg-devel
mailing list