[FFmpeg-devel] [PATCHv2 2/3] avformat: factorize iso 8601 timestamp writer to a dictionary avutil function

[Marton Balint]

On Sat, 2 Jul 2016, James Almer wrote:

> On 7/2/2016 7:10 PM, Marton Balint wrote:
>> Signed-off-by: Marton Balint <cus at passwd.hu>
>> ---
>>  libavformat/internal.h |  1 +
>>  libavformat/utils.c    | 16 ++--------------
>>  libavutil/dict.c       | 17 +++++++++++++++++
>>  libavutil/internal.h   | 10 ++++++++++
>>  4 files changed, 30 insertions(+), 14 deletions(-)
>> 
>> diff --git a/libavformat/internal.h b/libavformat/internal.h
>> index 647ad65..3ec4b0c 100644
>> --- a/libavformat/internal.h
>> +++ b/libavformat/internal.h
>> @@ -24,6 +24,7 @@
>>  #include <stdint.h>
>>
>>  #include "libavutil/bprint.h"
>> +#include "libavutil/internal.h"
>
> utils.c already includes this, so no need to include it here.
>
> Don't resend the patch just for this. Wait for a real review, or
> locally fix and apply if there's no need for another patch revision.

Thanks, fixed locally. Ping for the patch and for the rest of the series.

Regards,
Marton