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

James Almer jamrial at gmail.com
Sun Jul 3 00:24:29 CEST 2016


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.


More information about the ffmpeg-devel mailing list