Re: [FFmpeg-devel] [PATCH] libaom: Dont use aom_codec_av1_dx_algo.

2021-11-02 Thread James Zern
On Mon, Nov 1, 2021 at 6:11 PM Matt Oliver wrote: > > On Tue, 2 Nov 2021 at 04:41, James Zern > wrote: > > > On Fri, Oct 29, 2021 at 8:51 PM Matt Oliver wrote: > > > > > > On Sat, 30 Oct 2021 at 14:13, Matt Oliver wrote: > > > > > > > Does this compile and link with libaom 1.0.0? We

Re: [FFmpeg-devel] [PATCH] libaom: Dont use aom_codec_av1_dx_algo.

2021-11-01 Thread Matt Oliver
On Tue, 2 Nov 2021 at 04:41, James Zern wrote: > On Fri, Oct 29, 2021 at 8:51 PM Matt Oliver wrote: > > > > On Sat, 30 Oct 2021 at 14:13, Matt Oliver wrote: > > > > > Does this compile and link with libaom 1.0.0? We unfortunately still > > >> support that version (despite being unusable speed

Re: [FFmpeg-devel] [PATCH] libaom: Dont use aom_codec_av1_dx_algo.

2021-11-01 Thread James Zern
On Fri, Oct 29, 2021 at 8:51 PM Matt Oliver wrote: > > On Sat, 30 Oct 2021 at 14:13, Matt Oliver wrote: > > > Does this compile and link with libaom 1.0.0? We unfortunately still > >> support that version (despite being unusable speed wise) because distros > >> like Debian still ship it. > >> >

Re: [FFmpeg-devel] [PATCH] libaom: Dont use aom_codec_av1_dx_algo.

2021-11-01 Thread James Zern
On Fri, Oct 29, 2021 at 8:13 PM Matt Oliver wrote: > > > > > Does this compile and link with libaom 1.0.0? We unfortunately still > > support that version (despite being unusable speed wise) because distros > > like Debian still ship it. > > > > Just looked it up and yes version 1.0.0 includes

Re: [FFmpeg-devel] [PATCH] libaom: Dont use aom_codec_av1_dx_algo.

2021-10-29 Thread Matt Oliver
On Sat, 30 Oct 2021 at 14:13, Matt Oliver wrote: > Does this compile and link with libaom 1.0.0? We unfortunately still >> support that version (despite being unusable speed wise) because distros >> like Debian still ship it. >> > > Just looked it up and yes version 1.0.0 includes the required >

Re: [FFmpeg-devel] [PATCH] libaom: Dont use aom_codec_av1_dx_algo.

2021-10-29 Thread Matt Oliver
> > Does this compile and link with libaom 1.0.0? We unfortunately still > support that version (despite being unusable speed wise) because distros > like Debian still ship it. > Just looked it up and yes version 1.0.0 includes the required aom_codec_av1_dx() function (line 37 of aomdx.h for

Re: [FFmpeg-devel] [PATCH] libaom: Dont use aom_codec_av1_dx_algo.

2021-10-29 Thread James Almer
On 10/29/2021 11:43 PM, Matt Oliver wrote: lgtm. Thanks, I will push this assuming no other comments. Does this compile and link with libaom 1.0.0? We unfortunately still support that version (despite being unusable speed wise) because distros like Debian still ship it.

Re: [FFmpeg-devel] [PATCH] libaom: Dont use aom_codec_av1_dx_algo.

2021-10-29 Thread Matt Oliver
> > lgtm. > Thanks, I will push this assuming no other comments. ___ 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

Re: [FFmpeg-devel] [PATCH] libaom: Dont use aom_codec_av1_dx_algo.

2021-10-20 Thread James Zern
On Wed, Oct 20, 2021 at 9:59 AM Matt Oliver wrote: > > This fixes linking errors where variables cannot be correctly linked in > from an external shared library such as with msvc (requires dllimport which > is not used by libaom). Instead just call the function that returns the > same variable. >

[FFmpeg-devel] [PATCH] libaom: Dont use aom_codec_av1_dx_algo.

2021-10-20 Thread Matt Oliver
This fixes linking errors where variables cannot be correctly linked in from an external shared library such as with msvc (requires dllimport which is not used by libaom). Instead just call the function that returns the same variable. --- libavcodec/libaomdec.c | 2 +- 1 file changed, 1