On 2014-02-04 12:06:00 +0100, Vittorio Giovara wrote:
> ---
> 48 is the "high framerate" used in cinema, 25 and 50 are European framerate.
> Not sure it's worth to add 100 and 120 "high framerates" too.
> Vittorio
>
> libavformat/avformat.h | 2 +-
> libavformat/utils.c | 3 ++-
> libavformat/version.h | 2 +-
> 3 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/libavformat/avformat.h b/libavformat/avformat.h
> index 00380d7..26bd54d 100644
> --- a/libavformat/avformat.h
> +++ b/libavformat/avformat.h
> @@ -712,7 +712,7 @@ typedef struct AVStream {
> /**
> * Stream information used internally by av_find_stream_info()
> */
> -#define MAX_STD_TIMEBASES (60*12+5)
> +#define MAX_STD_TIMEBASES (60 * 12 + 8)
> struct {
> int nb_decoded_frames;
> int found_decoder;
> diff --git a/libavformat/utils.c b/libavformat/utils.c
> index fca588b..0b71ad2 100644
> --- a/libavformat/utils.c
> +++ b/libavformat/utils.c
> @@ -2177,7 +2177,8 @@ static int get_std_framerate(int i)
> if (i < 60 * 12)
> return (i + 1) * 1001;
> else
> - return ((const int[]) { 24, 30, 60, 12, 15 })[i - 60 * 12] * 1000 *
> 12;
> + return ((const int[]) { 24, 30, 60, 12, 15, 48, 25, 50 })
> + [i - 60 * 12] * 1000 * 12;
This doesn't match the description, adding them to this list results in
testing 48000/1001, 25000/1001 and 50000/1001. 48, 25, and 50 fps are
already tested through ht ecounter based test. look how
get_std_framerate is used. Changing it to something less confusing is
probably a good idea.
Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
