<div dir="auto">Most of the identical code concerns reading the metadata of audio file. I think around 150 lines could be taken out common. Thats 25% of the module codebase. </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 24, 2020, 3:18 AM Jean-Baptiste Kempf <<a href="mailto:jb@videolan.org">jb@videolan.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><u></u><div><div>Depends, how much is in common, and how much is not?<br></div><div><br></div><div>On Sun, Feb 23, 2020, at 22:44, Kartik Ohri wrote:<br></div><blockquote type="cite" id="m_4736607960368471560qt"><div dir="auto">Yes, that is correct. Should I extract the common code in a utilities file and then have the modules depend on it ?<br></div><div><br></div><div><div dir="ltr">On Mon, Feb 24, 2020, 2:56 AM Jean-Baptiste Kempf <<a href="mailto:jb@videolan.org" target="_blank" rel="noreferrer">jb@videolan.org</a>> wrote:<br></div><blockquote style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(204,204,204);border-left-style:solid;border-left-width:1px;padding-left:1ex"><div><br></div><div><br></div><div>On Sun, Feb 23, 2020, at 22:20, Kartik Ohri<a rel="noreferrer noreferrer" href="mailto:kartikohri13@gmail.com" target="_blank">kartikohri13@gmail.com</a> wrote:<br></div><div> > VLC already has the audioscrobbler module to submit scrobbles to<br></div><div> > <a rel="noreferrer noreferrer noreferrer" href="http://last.fm" target="_blank">last.fm</a> and other services with a similar API. This module extends<br></div><div> > that functionality to allow submission of listens to ListenBrainz.<br></div><div> > The existing audioscrobbler module is incompatible with ListenBrainz<br></div><div> > due to difference in authentication procedures and REST API for<br></div><div> > submissions.<br></div><div> <br></div><div> But cannot you reuse some code of the other module?<br></div><div> <br></div><div> <br></div><div> -- <br></div><div> Jean-Baptiste Kempf -  President<br></div><div> +33 672 704 734<br></div><div> _______________________________________________<br></div><div> vlc-devel mailing list<br></div><div> To unsubscribe or modify your subscription options:<br></div><div> <a rel="noreferrer noreferrer noreferrer" href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div></blockquote></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div><a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank" rel="noreferrer">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div></blockquote><div><br></div><div id="m_4736607960368471560sig60240713"><div>--<br></div><div>Jean-Baptiste Kempf -  President<br></div><div>+33 672 704 734<br></div><div> <br></div><div><br></div></div><div><br></div></div>_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div>