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 <jua...@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

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to