[FFmpeg-devel] [PATCH] avformat/mxfdec: Correct confusing struct tag

Tomas Härdin tjoppen at acc.umu.se
Sun Mar 29 16:22:46 EEST 2020


sön 2020-03-29 klockan 06:06 +0200 skrev Andreas Rheinhardt:
> Don't use typedef struct MXFTrack {...} MXFTimecodeComponent, in
> particular given the fact that MXFTrack is a type of its own.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>  libavformat/mxfdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> index 3374f36a88..fdd0dd2a88 100644
> --- a/libavformat/mxfdec.c
> +++ b/libavformat/mxfdec.c
> @@ -131,7 +131,7 @@ typedef struct MXFSequence {
>      uint8_t origin;
>  } MXFSequence;
>  
> -typedef struct MXFTrack {
> +typedef struct MXFTimecodeComponent {
>      UID uid;
>      enum MXFMetadataSetType type;
>      int drop_frame;

I noticed this as well, just the other day. LGTM

/Tomas



More information about the ffmpeg-devel mailing list