[vlc-devel] [PATCH] activex: first stop playing, only then destroy plugin parts.

[李小晴]

Thanks a lot! Could you tell me the VLC's code changes on linux, please?





At 2012-06-29 18:32:17,"Sergey Radionov" <rsatom at gmail.com> wrote:
>fix crash on closing some host applications when media still playing.
>---
> activex/plugin.cpp |   16 ++++++++--------
> 1 files changed, 8 insertions(+), 8 deletions(-)
>
>diff --git a/activex/plugin.cpp b/activex/plugin.cpp
>index fe5c5b8..17ac732 100644
>--- a/activex/plugin.cpp
>+++ b/activex/plugin.cpp
>@@ -282,6 +282,14 @@ VLCPlugin::VLCPlugin(VLCPluginClass *p_class, LPUNKNOWN pUnkOuter) :
> 
> VLCPlugin::~VLCPlugin()
> {
>+    if( vlc_player::is_open() )
>+    {
>+        if( isPlaying() )
>+            playlist_stop();
>+
>+        player_unregister_events();
>+    }
>+
>     delete vlcSupportErrorInfo;
>     delete vlcOleObject;
>     delete vlcDataObject;
>@@ -304,14 +312,6 @@ VLCPlugin::~VLCPlugin()
>     SysFreeString(_bstr_mrl);
>     SysFreeString(_bstr_baseurl);
> 
>-    if( vlc_player::is_open() )
>-    {
>-        if( isPlaying() )
>-            playlist_stop();
>-
>-        player_unregister_events();
>-    }
>-
>     if( _p_libvlc )  { libvlc_release(_p_libvlc); _p_libvlc=NULL; }
> 
>     _p_class->Release();
>-- 
>1.7.7.1.msysgit.0
>
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>http://mailman.videolan.org/listinfo/vlc-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20120630/1f2b1449/attachment.html>