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

Marton Balint cus at passwd.hu
Sun Aug 14 21:18:43 EEST 2016



On Mon, 11 Jul 2016, Marton Balint wrote:

>
> 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.
>

Will apply this soon.

Regards,
Marton


More information about the ffmpeg-devel mailing list