[FFmpeg-devel] [PATCH v2 2/3] avcodec: estimate output bitrate for ffv1/huffyuv codecs

[Tobias Rapp]

On 06.02.2017 13:33, Tobias Rapp wrote:
> Allows to get a more realistic total bitrate (and estimated file size)
> in avi_write_header. Previously a static default value of 200k was
> assumed.
>
> Signed-off-by: Tobias Rapp <t.rapp at noa-archive.com>
> ---
>  libavcodec/ffv1enc.c                           |  4 ++++
>  libavcodec/huffyuvenc.c                        |  4 ++++
>  libavcodec/internal.h                          |  6 ++++++
>  libavcodec/utils.c                             | 21 +++++++++++++++++++++
>  tests/ref/vsynth/vsynth1-ffv1                  |  2 +-
>  tests/ref/vsynth/vsynth1-ffv1-v0               |  2 +-
>  tests/ref/vsynth/vsynth1-ffv1-v3-bgr0          |  2 +-
>  tests/ref/vsynth/vsynth1-ffv1-v3-rgb48         |  2 +-
>  tests/ref/vsynth/vsynth1-ffv1-v3-yuv420p       |  2 +-
>  tests/ref/vsynth/vsynth1-ffv1-v3-yuv422p10     |  2 +-
>  tests/ref/vsynth/vsynth1-ffv1-v3-yuv444p16     |  2 +-
>  tests/ref/vsynth/vsynth1-ffvhuff               |  2 +-
>  tests/ref/vsynth/vsynth1-ffvhuff420p12         |  2 +-
>  tests/ref/vsynth/vsynth1-ffvhuff422p10left     |  2 +-
>  tests/ref/vsynth/vsynth1-ffvhuff444            |  2 +-
>  tests/ref/vsynth/vsynth1-ffvhuff444p16         |  2 +-
>  tests/ref/vsynth/vsynth1-huffyuv               |  2 +-
>  tests/ref/vsynth/vsynth1-huffyuvbgr24          |  2 +-
>  tests/ref/vsynth/vsynth1-huffyuvbgra           |  2 +-
>  tests/ref/vsynth/vsynth2-ffv1                  |  2 +-
>  tests/ref/vsynth/vsynth2-ffv1-v0               |  2 +-
>  tests/ref/vsynth/vsynth2-ffv1-v3-bgr0          |  2 +-
>  tests/ref/vsynth/vsynth2-ffv1-v3-rgb48         |  2 +-
>  tests/ref/vsynth/vsynth2-ffv1-v3-yuv420p       |  2 +-
>  tests/ref/vsynth/vsynth2-ffv1-v3-yuv422p10     |  2 +-
>  tests/ref/vsynth/vsynth2-ffv1-v3-yuv444p16     |  2 +-
>  tests/ref/vsynth/vsynth2-ffvhuff               |  2 +-
>  tests/ref/vsynth/vsynth2-ffvhuff420p12         |  2 +-
>  tests/ref/vsynth/vsynth2-ffvhuff422p10left     |  2 +-
>  tests/ref/vsynth/vsynth2-ffvhuff444            |  2 +-
>  tests/ref/vsynth/vsynth2-ffvhuff444p16         |  2 +-
>  tests/ref/vsynth/vsynth2-huffyuv               |  2 +-
>  tests/ref/vsynth/vsynth2-huffyuvbgr24          |  2 +-
>  tests/ref/vsynth/vsynth2-huffyuvbgra           |  2 +-
>  tests/ref/vsynth/vsynth3-ffv1                  |  2 +-
>  tests/ref/vsynth/vsynth3-ffv1-v0               |  2 +-
>  tests/ref/vsynth/vsynth3-ffv1-v3-bgr0          |  2 +-
>  tests/ref/vsynth/vsynth3-ffv1-v3-rgb48         |  2 +-
>  tests/ref/vsynth/vsynth3-ffv1-v3-yuv420p       |  2 +-
>  tests/ref/vsynth/vsynth3-ffv1-v3-yuv422p10     |  2 +-
>  tests/ref/vsynth/vsynth3-ffv1-v3-yuv444p16     |  2 +-
>  tests/ref/vsynth/vsynth3-ffvhuff               |  2 +-
>  tests/ref/vsynth/vsynth3-ffvhuff420p12         |  2 +-
>  tests/ref/vsynth/vsynth3-ffvhuff422p10left     |  2 +-
>  tests/ref/vsynth/vsynth3-ffvhuff444            |  2 +-
>  tests/ref/vsynth/vsynth3-ffvhuff444p16         |  2 +-
>  tests/ref/vsynth/vsynth3-huffyuv               |  2 +-
>  tests/ref/vsynth/vsynth3-huffyuvbgr24          |  2 +-
>  tests/ref/vsynth/vsynth3-huffyuvbgra           |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffv1              |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffv1-v0           |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffv1-v3-bgr0      |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffv1-v3-rgb48     |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffv1-v3-yuv420p   |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffv1-v3-yuv422p10 |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffv1-v3-yuv444p16 |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffvhuff           |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffvhuff420p12     |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffvhuff422p10left |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffvhuff444        |  2 +-
>  tests/ref/vsynth/vsynth_lena-ffvhuff444p16     |  2 +-
>  tests/ref/vsynth/vsynth_lena-huffyuv           |  2 +-
>  tests/ref/vsynth/vsynth_lena-huffyuvbgr24      |  2 +-
>  tests/ref/vsynth/vsynth_lena-huffyuvbgra       |  2 +-
>  64 files changed, 95 insertions(+), 60 deletions(-)
>
>[...]

I'd like to apply this patch soon, if there are no objections.

Regards,
Tobias