
<!doctype html><html><head><title></title><style type="text/css">

p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body>I doubt that the Android VLC app uses an option that's not in any release yet, and if it does, then that's a pretty irresponsible job from whoever dropped a not even beta VLC version on the general public.<br><br>And your patch set does not do what you say. It's trying and failing to second-guess the module loader.<br><br><div class="gmail_quote">Le 17 janvier 2020 16:13:52 GMT+02:00, Thomas Guillem <thomas@gllm.fr> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div><br></div><div><br></div><div>On Fri, Jan 17, 2020, at 14:49, Rémi Denis-Courmont wrote:<br></div><blockquote type="cite" id="qt"><div>I don't care about nonexistent apps that use an as yet nonexistent input item option. And by the way, zero is not "a lot".<br></div></blockquote><div><br></div><div>VLC for Android is not a nonexistent app, check the play store for usage stats.<br></div><div><br></div><blockquote type="cite" id="qt"><div><br></div><div>There are a number of apps ignoring the documentation and setting avcodec-hw in libvlc_new(). They will break no matter what exactly as the documentation warned they could, and they will have to adjust their syntax.<br></div><div><br></div><div>I do care about VLC and LibVLC working correctly, so not reintroducing parenting bugs and/or module string matching bugs. <br></div></blockquote><div><br></div><div>The dec dev parent will always be the vout, even with my patches. </div><blockquote type="cite" id="qt"><div><br></div><div>I also care about performance and thus not recreating the decoder device at every ES selection<br></div></blockquote><div><br></div><div>The dec dev won't be recreated except if the user configure a different one for each item, which is very unlikely. </div><div><br></div><blockquote type="cite" id="qt"><div><br></div><div>-1.<br></div><div><br></div><div class="qt-gmail_quote"><div>Le 17 janvier 2020 15:18:51 GMT+02:00, Thomas Guillem <thomas@gllm.fr> a écrit :<br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:0pt;margin-left:0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204, 204, 204);padding-left:1ex;" class="qt-gmail_quote"><div> A lot of LibVLC clients have a setting to enable/disable hw decoding. Changing the "avcodec-hw" string by "dec-dev" is quite easy. But having to destroy and create a new libvlc in order to update user preferences is a huge change and require a lot of adaptation in client apps.<br></div><div><br></div><div><br></div><div>On Fri, Jan 17, 2020, at 12:28, Rémi Denis-Courmont wrote:<br></div><blockquote id="qt-qt" type="cite"><div>This "feature" broke anyway since avcodec-hw no longer exists. Users used to be able to set plenty of variables before sout-keep and then input resources were implemented. Moving settings to more global scope is nothing new.<br></div><div><br></div><div class="qt-qt-gmail_quote"><div>Le 17 janvier 2020 10:30:34 GMT+02:00, Thomas Guillem <thomas@gllm.fr> a écrit :<br></div><blockquote class="qt-qt-gmail_quote" style="margin-top:0pt;margin-right:0pt;margin-bottom:0pt;margin-left:0.8ex;border-left-color:rgb(204, 204, 204);border-left-style:solid;border-left-width:1px;padding-left:1ex;"><pre class="qt-qt-k9mail"><div>It's OK to break gapless if the hw dec settings change between 2 inputs. <br></div><div><br></div><div>VLC users were always able to configure hw dec settings via the input (via avcodec-hw and other vars). I thnk we should keep it that way.<br></div><div><br></div><div>Do you have an other solution in mind? <br></div><div><br></div><div>On Thu, Jan 16, 2020, at 19:00, Rémi Denis-Courmont wrote:<br></div><blockquote class="qt-qt-gmail_quote" style="margin-top:0pt;margin-right:0pt;margin-bottom:1ex;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:1px;padding-left:1ex;"><div>Le torstaina 16. tammikuuta 2020, 18.41.36 EET Thomas Guillem a écrit :<br></div><blockquote class="qt-qt-gmail_quote" style="margin-top:0pt;margin-right:0pt;margin-bottom:1ex;margin-left:0.8ex;border-left-color:rgb(173, 127, 168);border-left-style:solid;border-left-width:1px;padding-left:1ex;">Optinal, will be used to get the input_thread_t "dec-dev" variable.<br></blockquote><div>We don't accept output options on input items for years for good reasons. It <br></div><div>was totally broken. And it prevents gap-less.<br></div><div><br></div><div>-- <br></div><div>Реми Дёни-Курмон<br></div><div><a href="http://www.remlab.net/">http://www.remlab.net/</a><hr>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">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div></blockquote><div><hr>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">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div></pre></blockquote></div><div><br></div><div>-- <br></div><div>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté. <br></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div>https://mailman.videolan.org/listinfo/vlc-devel<br></div></blockquote><div><br></div></blockquote></div><div><br></div><div>-- <br></div><div>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté. <br></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div>https://mailman.videolan.org/listinfo/vlc-devel<br></div></blockquote><div><br></div></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>