[vlc-devel] commit: Use the signal interface and fix signal handling ( Rémi Denis-Courmont )

[bl4]

Rémi Denis-Courmont wrote:
> Le lundi 2 juin 2008 23:14:27 bl4, vous avez écrit :
>   
>> I'm afraid this commit broke the vlc gui in my case. When I pass a
>> filename on the command line, vlc starts playing, but the status bar is
>> empty, the title bar is unchanged, the progress bar doesn't move and the
>> Pause/Play button doesn't work. Everything goes back to normal when I
>> stop and play again. Everything is fine when I use Media | Open File
>> menu to open a file.
>>     
>
> I cannot reproduce the problem, but this should be fixed now anyway.
>   

Well, the problem still occurs. One of your commits removed b_play 
property and all the code that depended on it. It means that 
playlist_Control( THEPL, PLAYLIST_PLAY, false ) in qt4.cpp is no longer 
executed (or something similar in other gui modules, I only tested qt). 
Instead, the playlist is started in vlc.c with libvlc_playlist_play 
(vlc, -1, 0, NULL, NULL).

This change causes exactly the same problem with my gui, I can't explain 
why. I fixed it by restoring the code in qt4.cpp, making it work without 
b_play property and removing the code for starting playlist in vlc.c. 
But the other gui modules would have to be changed too and it looks 
pointless because it's not what removing b_play was for. When I have 
some time I'll try to find out why it doesn't work for me.

-- 
bl4