[FFmpeg-devel] [PATCH 2/2] avisynth: support pix_fmts added to AviSynth+

[Roger Pack]

Does this require some version check in configure to be added as well?

On 8/16/16, Michael Niedermayer <michael at niedermayer.cc> wrote:
> On Mon, Aug 15, 2016 at 12:37:31PM -0400, Stephen Hutchinson wrote:
>> A number of new pix_fmts have been added to AviSynth+:
>> 16-bit packed RGB and RGBA
>> 10-, 12-, 14, and 16-bit YUV 4:2:0, 4:2:2, and 4:4:4
>> 8-, 10-, 12-, 14-, and 16-bit Planar RGB
>> 8-, 10-, 12-, 14-, and 16-bit Planar YUVA and Planar RGBA*
>> 10-, 12-, 14-, and 16-bit GRAY variants*
>> 32-bit floating point Planar YUV(A), Planar RGB(A), and GRAY*
>>
>> *some of which are not currently available pix_fmts here and were
>>  not added to the demuxer due to this
>> ---
>>  libavformat/avisynth.c | 176
>> ++++++++++++++++++++++++++++++++++++++++++++++++-
>>  1 file changed, 175 insertions(+), 1 deletion(-)
>
> breaks build
>
> libavformat/avisynth.c:100:40: error: ‘AVS_PLANAR_G’ undeclared here (not in
> a function)
> libavformat/avisynth.c:100:54: error: ‘AVS_PLANAR_B’ undeclared here (not in
> a function)
> libavformat/avisynth.c:101:43: error: ‘AVS_PLANAR_R’ undeclared here (not in
> a function)
> libavformat/avisynth.c:103:57: error: ‘AVS_PLANAR_A’ undeclared here (not in
> a function)
> libavformat/avisynth.c: In function ‘avisynth_read_packet_video’:
> libavformat/avisynth.c:686:36: error: ‘AviSynthLibrary’ has no member named
> ‘avs_is_planar_rgb’
> libavformat/avisynth.c:687:36: error: ‘AviSynthLibrary’ has no member named
> ‘avs_is_planar_rgba’
> make: *** [libavformat/avisynth.o] Error 1
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> Rewriting code that is poorly written but fully understood is good.
> Rewriting code that one doesnt understand is a sign that one is less smart
> then the original author, trying to rewrite it will not make it better.
>