[vlc-devel] [PATCH] Add Rust backed cue sheet parsing module

Kartik Ohri kartikohri13 at gmail.com
Thu Aug 20 10:51:13 CEST 2020

On Thu, Aug 20, 2020 at 1:43 PM Steve Lhomme <robux4 at ycbcr.xyz> wrote:

> On 2020-08-19 17:22, Alexandre Janniaux wrote:
> > Hi,
> >
> > On Wed, Aug 19, 2020 at 05:04:08PM +0200, RĂ©mi Denis-Courmont wrote:
> >> And then I would expect that the interface from Rust is not ad-hoc, but
> rather wrapping the plugin API directly.
> >
> > I'm not sure to understand this correctly.
> >
> > Does it mean that rust should use vlccore function for the plugin
> > behaviour, or that the plugin should directly expose itself without
> > a C wrapper, or something else?
> Have generic Rust code that handles the VLC filter API and then each
> filter be written in Rust without caring of the C binding. That seems
> cleaner IMO so when we update the filter API we only have this binding
> to update.
> And then we could do the same kind of generic binding for codecs and
> pretty much every other module type.

Before beginning the development on this, I had a discussion with Romain and
Thomas. It was decided that this approach is not very good. The rationale
is that
the Rust Bindings for the VLC API will need to be regenerated for even a
change in the VLC API. This will be cumbersome. Also, since Rust will be
sparingly in the codebase it is probably better to have separate rust code
expose a C interface. I think Thomas and Romain will be able to better
the reasons behind the approach here.

> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20200820/f30a0560/attachment.html>

More information about the vlc-devel mailing list