[vlc-devel] [PATCHv2 7/8] codec: add rav1e encoder module

[Tristan Matthews]

Hi,

On Wed, Aug 26, 2020 at 11:12 AM Rémi Denis-Courmont <remi at remlab.net> wrote:
>
>         Hi,
>
> One would hope that you can install rav1e without contribs, and this patch can
> be reviewed and pushed independently of contrib patches.

I believe it can, but reviewers may not want the trouble of having to
figure out how to install rav1e etc. on their hosts as it isn't
necessarily available in package managers yet, so I appreciate the
effort made here to facilitate testing.

> However, do we really want to use Rav1e via C? That seems a bit odd at a time
> when we are also pondering how to write Rust modules.

I don't really see any downsides of using rav1e via C (zero-cost
abstractions™ and all that) instead of Rust. If we're only at the
"pondering" stage of writing Rust modules (where IMO, the biggest
value-added would be for demuxers et al, not encoder library modules)
I don't think this should block this patchset.

Best,
-t

>
> --
> レミ・デニ-クールモン
> http://www.remlab.net/
>
>
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel