[vlc-devel] [RFC][PATCH] fix mozplug toolbar
Jean-Paul Saman
jpsaman at videolan.org
Tue Jun 17 22:43:14 CEST 2008
Anthony Loiseau wrote:
> As discussed on the IRC,
> Mozilla-plugin ODS-toolbar (X11) seems to not work.
That is my fault I broke it while trying to rewrite it. The original
codes was not ok so I set out to rewrite it, but due to lack of time it
stalled.
> A while ago, I have managed to patch it but had not found the time to
> make my patches clean, so did't propose it (part of patch 0002)
> As asked, here there are (I've done a pull rebase before).
I applied 0002 after some rewrite, but the approach seemed ok
> I have no time to test it, sorry. The reason why I add RFC as subject.
> I find them not clean because:
>
> - one patch fix 2 things (need a rebase -i to be cleaner)
> - timeline didn't worked, iirc (but play/pause worked)
Timeline never worked in the orignal patch.
> I don't remember if last patch (play/pause) was very usefull, you should
> test with and without. It was about pausing after the end of the video
> and pausing several times.
Indeed 0003 is not usefull.
> The flag is_playing did not come back to "play" when quitting pause
> calling libvlc_play, so I had thought to quit it with another
> libvlc_pause (this is a toggle function).
>
>
> Hope I am clear enough,
> good luck with those patches and sorry for the lack of time to send them
> just clean.
> _anthony (thannoy)
>
Thanks for sending these patches.
Gtz
Jean-Paul Saman.
More information about the vlc-devel
mailing list