[FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

[Michael Niedermayer]

On Tue, Jul 30, 2019 at 06:47:23PM -0700, Juan De León wrote:
> Removed AVQuantizationParamsArray to prevent ambiguous memory allocation.
> For simplicity, the side data will be allocated as an array of AVQuantizationParams and the last element of the array will have w and h set to 0.
> 
> Better explained in the doc.
> design doc: https://docs.google.com/document/d/1WClt3EqhjwdGXhEw386O0wfn3IBQ1Ib-_5emVM1gbnA/edit?usp=sharing
> 
> Signed-off-by: Juan De León <juandl at google.com>
> ---
>  libavutil/Makefile              |   2 +
>  libavutil/frame.h               |   6 ++
>  libavutil/quantization_params.c |  41 +++++++++++++
>  libavutil/quantization_params.h | 101 ++++++++++++++++++++++++++++++++
>  4 files changed, 150 insertions(+)
>  create mode 100644 libavutil/quantization_params.c
>  create mode 100644 libavutil/quantization_params.h

doesnt build here

libavutil/quantization_params.c: In function ‘av_get_qp_type_string’:
libavutil/quantization_params.c:33:72: error: ‘NULL’ undeclared (first use in this function)
             return index < AV_QP_ARR_SIZE_H264 ? QP_NAMES_H264[index] :NULL;
                                                                        ^
libavutil/quantization_params.c:33:72: note: each undeclared identifier is reported only once for each function it appears in
libavutil/quantization_params.c:41:1: error: control reaches end of non-void function [-Werror=return-type]
 }
 ^
cc1: some warnings being treated as errors
make: *** [libavutil/quantization_params.o] Error 1

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190802/e73f2847/attachment.sig>