<br><br><div class="gmail_quote">On Fri, Sep 4, 2009 at 10:49 PM, Niles Bindel <span dir="ltr"><<a href="mailto:zaggal69@gmail.com">zaggal69@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I've separated the patch into two stages...the first being the conversion to Media List Player and the second a cosmetic patch to be applied after the first patch.  I also found that my first patch was missing the Mozilla plugin changes I made so I've added those changes to these patches as well.<br>

<br></blockquote><div>In the first patch there is a change in method onActivateInPlace() which is not related to the modifications as outlined in the commitlog. Could you please explain why you took that out of the if( _b_usermode ) branch? (And preferably separate into another patch.)<br>
 <br></div>@@ -717,17 +770,18 @@ HRESULT VLCPlugin::onActivateInPlace(LPMSG lpMesg, HWND hwndParent, LPCRECT lprc<br>     HRGN clipRgn = CreateRectRgnIndirect(&clipRect);<br>     SetWindowRgn(_inplacewnd, clipRgn, TRUE);<br>
 <br>+    /* will run vlc if not done already */<br>+    libvlc_instance_t* p_libvlc;<br>+    HRESULT result = getVLC(&p_libvlc);<br>+    if( FAILED(result) )<br>+        return result;<br>+<br>+    set_player_window(NULL);<br>
+<br>     if( _b_usermode )<br>     {<br>-        /* will run vlc if not done already */<br>-        libvlc_instance_t* p_libvlc;<br>-        HRESULT result = getVLC(&p_libvlc);<br>-        if( FAILED(result) )<br>-            return result;<br>
-<br>-        if( _b_autoplay && playlist_select(0,NULL) )<br>+        if( _b_autoplay && playlist_play_item(0, NULL) )<br>         {<br>-            libvlc_media_player_play(_p_mplayer,NULL);<br>             fireOnPlayEvent();<br>
         }<br>     }<br><br>Gtz<br>Jean-Paul Saman<br></div><br>