[Android] MediaWrapper: remove constructor with LibVLC

Thomas Guillem git at videolan.org
Fri Jan 23 13:01:19 CET 2015


vlc-ports/android | branch: master | Thomas Guillem <thomas at gllm.fr> | Fri Jan 23 10:25:46 2015 +0100| [9880ab8a5069d7b36d2d6006523d6034edc7fdbe] | committer: Thomas Guillem

MediaWrapper: remove constructor with LibVLC

This constructor was parsing a media and could take a long time. Replace it
with a mrl constructor that don't parse a media.

Indeed, MediaWrapper is only a wrapper and is often used in the UI Thread. It
shouldn't do any parsing (but it still fetch data that is already parsed when
constructed from a media).

Add "// FIXME" for all media that are parsed in UI Thread.

> http://git.videolan.org/gitweb.cgi/vlc-ports/android.git/?a=commit;h=9880ab8a5069d7b36d2d6006523d6034edc7fdbe
---

 vlc-android/src/org/videolan/vlc/MediaLibrary.java |   10 +++++---
 vlc-android/src/org/videolan/vlc/MediaWrapper.java |   17 +++++--------
 .../src/org/videolan/vlc/MediaWrapperList.java     |    4 +---
 .../org/videolan/vlc/MediaWrapperListPlayer.java   |    6 ++++-
 .../src/org/videolan/vlc/audio/AudioService.java   |   25 +++++++++++++-------
 .../src/org/videolan/vlc/gui/DirectoryAdapter.java |   10 +++++---
 .../vlc/gui/video/VideoPlayerActivity.java         |    6 ++++-
 vlc-android/src/org/videolan/vlc/util/Util.java    |    9 +++++--
 8 files changed, 54 insertions(+), 33 deletions(-)

Diff:   http://git.videolan.org/gitweb.cgi/vlc-ports/android.git/?a=commitdiff;h=9880ab8a5069d7b36d2d6006523d6034edc7fdbe


More information about the Android mailing list