[FFmpeg-devel] [PATCH 12/17] framesync: switch to child_class_iterate()

[Nicolas George]

Anton Khirnov (12020-05-28):
> ---
>  libavfilter/framesync.c | 7 +++++++
>  libavfilter/framesync.h | 3 +++
>  2 files changed, 10 insertions(+)
> 
> diff --git a/libavfilter/framesync.c b/libavfilter/framesync.c
> index b32a5cba6c..04119d686c 100644
> --- a/libavfilter/framesync.c
> +++ b/libavfilter/framesync.c
> @@ -53,6 +53,13 @@ static const AVClass framesync_class = {
>      .parent_log_context_offset = OFFSET(parent),
>  };
>  
> +const AVClass *ff_framesync_child_class_iterate(void **iter)
> +{
> +    const AVClass *c = *iter ? NULL : &framesync_class;

> +    *iter = (void*)(uintptr_t)c;

"void *" please.

> +    return c;
> +}
> +
>  enum {
>      STATE_BOF,
>      STATE_RUN,
> diff --git a/libavfilter/framesync.h b/libavfilter/framesync.h
> index 37743cccb7..51bab16285 100644
> --- a/libavfilter/framesync.h
> +++ b/libavfilter/framesync.h
> @@ -297,6 +297,8 @@ int ff_framesync_dualinput_get(FFFrameSync *fs, AVFrame **f0, AVFrame **f1);
>   */
>  int ff_framesync_dualinput_get_writable(FFFrameSync *fs, AVFrame **f0, AVFrame **f1);
>  
> +const AVClass *ff_framesync_child_class_iterate(void **iter);
> +
>  #define FRAMESYNC_DEFINE_CLASS(name, context, field) \
>  static int name##_framesync_preinit(AVFilterContext *ctx) { \
>      context *s = ctx->priv; \
> @@ -318,6 +320,7 @@ static const AVClass name##_class = { \
>      .version          = LIBAVUTIL_VERSION_INT, \
>      .category         = AV_CLASS_CATEGORY_FILTER, \
>      .child_class_next = name##_child_class_next, \
> +    .child_class_iterate = ff_framesync_child_class_iterate, \
>      .child_next       = name##_child_next, \
>  }
>  

LGTM.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200529/b29cdcf9/attachment.sig>