Le 14/06/2012 15:35, David Girault a écrit :
> This new decoder will allow to decode menu streams found
> in m2ts files in Bluray or AVCHD HDMV disks.
>
> Signed-off-by: David Girault <[email protected]>
> ---
>  version 6:
>  - rename decoder functions
>  - more stylistical fixes

Hi all,

This new version introduce a bug in that prevent menu to be displayed
correctly. :-(
I've fixed it already (see below) but have more change in queue (code
cleanup, better igs stream decoding, libbluray input, ...).


>  version 5:
>  - add documentation, version and changelog updates
>  - add missing malloc checks
>  - fixes most stylistical remarks
>  version 4:
>  - Added some fixes for x264 demo disk that have button without
>  picture.
>  - Change palette conversion to avoid convertion in avplay.
>
>  Changelog               |    1 +
>  doc/general.texi        |    9 +
>  libavcodec/Makefile     |    1 +
>  libavcodec/allcodecs.c  |    3 +
>  libavcodec/avcodec.h    |   55 ++++
>  libavcodec/igsmenudec.c |  801 
> +++++++++++++++++++++++++++++++++++++++++++++++
>  libavcodec/utils.c      |   36 +++
>  libavcodec/version.h    |    2 +-
>  libavutil/avutil.h      |    1 +
>  9 files changed, 908 insertions(+), 1 deletion(-)
>  create mode 100644 libavcodec/igsmenudec.c
>
> [...]
>
> diff --git a/libavcodec/igsmenudec.c b/libavcodec/igsmenudec.c
> new file mode 100644
> index 0000000..2719203
> --- /dev/null
> +++ b/libavcodec/igsmenudec.c
> [...]
> +    ove->start_display_time = 0;       // display immediatly
> +    ove->end_display_time   = 3600000; // todo: use specified user timeout
> +    ove->format             = 0;
> +
> +    ove->rects = av_mallocz(sizeof(AVOverlayRect*) * ove->num_rects);
ove->rects = av_mallocz(sizeof(AVOverlayRect*) * page->bogs_count);

> +    if (!ove->rects)
> +        return 1; // return with zero rects
> +
> +    ove->num_rects = page->bogs_count;
> +
> +    av_log(avctx, AV_LOG_INFO, "Displaying menu page %d, %d bog, button %d "
> +           "%s.\n",
> +           ctx->page, ove->num_rects, page->sel_button,
> +                (activated ? "activated" : "selected"));
> [...]


Regards,
David
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to