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

[Kartik Ohri]

Hi!
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
minor
change in the VLC API. This will be cumbersome. Also, since Rust will be
used
sparingly in the codebase it is probably better to have separate rust code
that
expose a C interface. I think Thomas and Romain will be able to better
explain
the reasons behind the approach here.

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


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