[vlc-devel] Android: Installation location, memory leak, vout background

[Rohit Yadav]

Hi,

This patch fixes a potential memory leak when a media is stopped and played
again or a new media is selected and play.
(I don't trust dalvik's GC)

As of current state, when I play a media file, then quit and then reopen
VLC, it would hang, sometimes I see no video.
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.

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?

Best Regards,
Rohit Yadav

On Fri, Mar 18, 2011 at 1:56 AM, Adrien Maglo <magsoft at videolan.org> wrote:

> Le 17/03/2011 19:58, Rohit Yadav a écrit :
>
>> Patch 1: Forces preferred installation location to external storage
>> Patch 2: Release any previous media player instance, before creating a
>> new one
>> Patch 3: Trivial, use black as bg for vout
>> If anyone care, vlc-android neon build with wip gui:
>> dl.rohityadav.in/VLC-neon-unstable.apk
>> <http://dl.rohityadav.in/VLC-neon-unstable.apk>
>>
>
> Patch 2 applied.
>
> Thanks for your work,
>
>
> --
> MagSoft
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110318/f959e880/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Android-Aout-Release-AudioTrack-resources-before-cre.patch
Type: text/x-patch
Size: 1719 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110318/f959e880/attachment.bin>