<div dir="ltr">Hi!<div>If there are no objections, can this patchset be merged ?</div><div>Thanks.</div><div>Regards,</div><div>Kartik</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Sep 20, 2020 at 2:39 AM Kartik Ohri <<a href="mailto:kartikohri13@gmail.com">kartikohri13@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This patchset aims to introduce a rust based cuesheet parsing<br>
module.<br>
<br>
Changes since RFC v2 are as follows:<br>
<br>
        - Added a utils module to vlccore-rs<br>
                This module can contain commonly used utilities for Rust<br>
                modules. For instance, passing Rust allocated strings to<br>
                C code. This is an easy thing to mess up. Rust allocated<br>
                pointers should not be free()`ed in C code. Hence, a<br>
                utility method is provided in vlccore-rs::utils to <br>
                ensure such issues are avoided.<br>
<br>
        - Added cargo workspace to src and modules.<br>
                This will be particularly beneficial while integrating<br>
                rustfmt in CI and running tests for Rust modules.<br>
<br>
        - Make the cuesheet module a mock module.<br>
                Since, there is already a lua based cuesheet parser<br>
                available in VLC and the aim of this module is to serve<br>
                as an example, it makes sense only to build the module<br>
                but not install it.<br>
<br>
        - Use vlc_stream_Control instead of vlc_stream_vaControl<br>
<br>
        - Other suggestions fron Code Review<br>
<br>
Changes left out for now:<br>
<br>
        Since, cargo is not yet available in the CI. Adding rustfmt<br>
        to CI and integrating rust tests in the buildsystem has been<br>
        left for now. Thomas has offered to integrate cargo in the<br>
        CI soon. Once, that is done we can address these issues.<br>
<br>
Future Developments:<br>
<br>
        John Pardo, an intern at Videolabs is working on porting the<br>
        vlc_input_item API to Rust. When that is done, this module<br>
        can almost entirely be extracted into Rust with only the<br>
        module declaration and setting up callabcks in C.<br>
<br>
[PATCH 1/2] libvlc: Add Rust API for writing modules in rust<br>
[PATCH 2/2] stream filter: Add rust based cuesheet module<br>
<br>
</blockquote></div>