<div dir="ltr"><div><div><div>This is more to be extra-safe.<br></div>But it could be:<br></div>- if user doesn't grant storage read permission at runtime (Android 6+)<br></div>- If video player is launched from 3rd party app (until we manage to partially init the ML for this case)<br></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jan 4, 2017 at 2:40 PM Hugo Beauzée-Luyssen <<a href="mailto:hugo@beauzee.fr">hugo@beauzee.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 01/03/2017 05:13 PM, Geoffrey Métais wrote:<br class="gmail_msg">
> vlc-android | branch: master | Geoffrey Métais <<a href="mailto:geoffrey.metais@gmail.com" class="gmail_msg" target="_blank">geoffrey.metais@gmail.com</a>> | Tue Jan  3 17:08:40 2017 +0100| [e9a55720d59e7ae17ada0f480fb90a6cbac818e1] | committer: Geoffrey Métais<br class="gmail_msg">
><br class="gmail_msg">
> Check ML is init when adding an external media<br class="gmail_msg">
><br class="gmail_msg">
>> <a href="https://code.videolan.org/videolan/vlc-android/commit/e9a55720d59e7ae17ada0f480fb90a6cbac818e1" rel="noreferrer" class="gmail_msg" target="_blank">https://code.videolan.org/videolan/vlc-android/commit/e9a55720d59e7ae17ada0f480fb90a6cbac818e1</a><br class="gmail_msg">
> ---<br class="gmail_msg">
><br class="gmail_msg">
>  medialibrary/src/org/videolan/medialibrary/Medialibrary.java | 2 +-<br class="gmail_msg">
>  1 file changed, 1 insertion(+), 1 deletion(-)<br class="gmail_msg">
><br class="gmail_msg">
> diff --git a/medialibrary/src/org/videolan/medialibrary/Medialibrary.java b/medialibrary/src/org/videolan/medialibrary/Medialibrary.java<br class="gmail_msg">
> index 93b1fda..67f674a 100644<br class="gmail_msg">
> --- a/medialibrary/src/org/videolan/medialibrary/Medialibrary.java<br class="gmail_msg">
> +++ b/medialibrary/src/org/videolan/medialibrary/Medialibrary.java<br class="gmail_msg">
> @@ -206,7 +206,7 @@ public class Medialibrary {<br class="gmail_msg">
>      }<br class="gmail_msg">
><br class="gmail_msg">
>      public MediaWrapper addMedia(String mrl) {<br class="gmail_msg">
> -        return nativeAddMedia(mrl);<br class="gmail_msg">
> +        return mIsInitiated ? nativeAddMedia(mrl) : null;<br class="gmail_msg">
>      }<br class="gmail_msg">
><br class="gmail_msg">
>      public long getId() {<br class="gmail_msg">
><br class="gmail_msg">
> _______________________________________________<br class="gmail_msg">
> Android mailing list<br class="gmail_msg">
> <a href="mailto:Android@videolan.org" class="gmail_msg" target="_blank">Android@videolan.org</a><br class="gmail_msg">
> <a href="https://mailman.videolan.org/listinfo/android" rel="noreferrer" class="gmail_msg" target="_blank">https://mailman.videolan.org/listinfo/android</a><br class="gmail_msg">
><br class="gmail_msg">
Is there a case where the ML could be uninitialized while the user is<br class="gmail_msg">
able to insert an external media?<br class="gmail_msg">
_______________________________________________<br class="gmail_msg">
Android mailing list<br class="gmail_msg">
<a href="mailto:Android@videolan.org" class="gmail_msg" target="_blank">Android@videolan.org</a><br class="gmail_msg">
<a href="https://mailman.videolan.org/listinfo/android" rel="noreferrer" class="gmail_msg" target="_blank">https://mailman.videolan.org/listinfo/android</a><br class="gmail_msg">
</blockquote></div>