Hi,<div><br></div><div><meta http-equiv="content-type" content="text/html; charset=utf-8"><div>This patch fixes a potential memory leak when a media is stopped and played again or a new media is selected and play.</div><div>

(I don't trust dalvik's GC)</div></div><div><br></div><div>As of current state, when I play a media file, then quit and then reopen VLC, it would hang, sometimes I see no video.</div><div>The problem I think is: even when finish() is called to quit, vlc.android stays as a process, so because it's not cleaning up, restarting it causes issues.</div>

<div><br></div><div>If LibVlc/VLC were a service, it would run in background, and I could still hear music when I goto for ex. on home screen, as opposed to it being an activity which pauses when I switch to home screen. So I'm proposing, LibVlc to run as a Service and VLC as an Activity/basically the gui, comment?</div>

<div><br></div><div>Best Regards,</div><div>Rohit Yadav</div><div><br><div class="gmail_quote">On Fri, Mar 18, 2011 at 1:56 AM, Adrien Maglo <span dir="ltr"><<a href="mailto:magsoft@videolan.org">magsoft@videolan.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Le 17/03/2011 19:58, Rohit Yadav a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div></div><div class="h5">
Patch 1: Forces preferred installation location to external storage<br>
Patch 2: Release any previous media player instance, before creating a<br>
new one<br>
Patch 3: Trivial, use black as bg for vout<br>
If anyone care, vlc-android neon build with wip gui:<br>
<a href="http://dl.rohityadav.in/VLC-neon-unstable.apk" target="_blank">dl.rohityadav.in/VLC-neon-unstable.apk</a><br></div></div>
<<a href="http://dl.rohityadav.in/VLC-neon-unstable.apk" target="_blank">http://dl.rohityadav.in/VLC-neon-unstable.apk</a>><br>
</blockquote>
<br>
Patch 2 applied.<br>
<br>
Thanks for your work,<br><font color="#888888">
<br>
<br>
-- <br>
MagSoft<br>
</font></blockquote></div><br></div>