[vlc-devel] [vlc-commits] modules: store object name in a variable

Thomas Guillem thomas at gllm.fr
Tue Mar 5 10:12:52 CET 2019


On Tue, Mar 5, 2019, at 10:02, Rémi Denis-Courmont wrote:
> Yeah. And we never agreed to un-deprecate object names and allow them to be used by plugins back again. And I did even remind of the deprecation at the last workshop less than two weeks ago - not exactly a surprise.

We didn't decided to break the QT module dialog. This feature is very very useful for user support, I don't think we can't ride of it.

And yes, It happens to me a lot: I decide to a technical solution, I try to implement it and I encounter something I didn't planned.

For me, module_t still need to be tied to vlc_object_t

> 
> What you want is that all changes are agreed upon by you. That's not what we agreed.

Not me, but us. If developers that are doing a lot more user support than me (Francois, Steve, Hugo and JB) agree to kill the QT module dialog, I won't be against the common decision.

> 
> Le 5 mars 2019 10:48:21 GMT+02:00, Thomas Guillem <thomas at gllm.fr> a écrit :
>> On Tue, Mar 5, 2019, at 09:43, Rémi Denis-Courmont wrote:
>>> No. I am not accountable for bugs pushed by Antoine (object names), the other Rémi (Qt objects tree) and François (adding code using a feature that's already deprecated and of course without review).
>> 
>> And that is why, we decided to always go to the mailing list when we add/remove new features or modify the VLC/LibVLC core API. It's useless to lament on what was done on the past or find a culprit for bad things that were done a *long* time ago.
>> 
>>> 
>>> Unlike you, I have spent days trying to fix these things with variable success. And now, you are preventing me from fixing that for good.
>>> 
>>> So if somebody's accountable, it's you.
>>> 
>>> Le 5 mars 2019 10:17:34 GMT+02:00, Steve Lhomme <robux4 at ycbcr.xyz> a écrit :
>>>> On 3/5/2019 8:52 AM, Rémi Denis-Courmont wrote:
>>>>> I am not going to hold your hand running git-log or git-blame. The 
>>>>> facts are there, you just don't care to look for them preferring to 
>>>>> band against me.
>>>> 
>>>> You're accountable for the things you push on VLC just like the rest of 
>>>> us. If people disagree with a change you made without discussing you 
>>>> have to explain why, just like the rest of us. Saying it was already 
>>>> discussed 10 years ago (as a flamewar, so definitely not in the SVN 
>>>> logs) with other people is not a way to collaborate.
>>>> 
>>>> >
>>>>>  And it's not exactly rocket science that introspection does not work 
>>>>>  across threads - in general, not just in VLC.
>>>>> 
>>>>>  Le 5 mars 2019 09:03:43 GMT+02:00, Steve Lhomme <robux4 at ycbcr.xyz> a 
>>>>>  écrit :
>>>>> 
>>>>>      On 3/4/2019 9:51 PM, Rémi Denis-Courmont wrote:
>>>>> 
>>>>>          Decreeing that discussions older than 3 is irrelevant is
>>>>>          ridiculous. 
>>>>> 
>>>>> 
>>>>>      Yes and no. Valid points many years ago are likely to be valid today.
>>>>>      But if we're not presented with them there is no way to know. The code
>>>>>      has changed a lot in the last few years and will continue to do so. Some
>>>>>      things that used to be true may not be anymore and vice versa. But we
>>>>>      need the facts, not "it was discussed 9 years ago, end of discussion".     vlc-devel mailing list
>>>>>      To unsubscribe or modify your subscription options:
>>>>>      https://mailman.videolan.org/listinfo/vlc-devel
>>>>> 
>>>>> 
>>>>>  -- 
>>>>>  Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez 
>>>>>  excuser ma brièveté. vlc-devel mailing list
>>>>>  To unsubscribe or modify your subscription options:
>>>>>  https://mailman.videolan.org/listinfo/vlc-devel
>>>> vlc-devel mailing list
>>>> To unsubscribe or modify your subscription options:
>>>> https://mailman.videolan.org/listinfo/vlc-devel
>>> 
>>> -- 
>>> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté. 
>>> _______________________________________________
>>> vlc-devel mailing list
>>> To unsubscribe or modify your subscription options:
>>> https://mailman.videolan.org/listinfo/vlc-devel
>> 
> 
> -- 
> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté. 
> _______________________________________________
> 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/20190305/2e626a98/attachment.html>


More information about the vlc-devel mailing list