<div dir="ltr"><div>LGTM.<br><br></div>But you should have mailed it to <a href="mailto:android@videolan.org" target="_blank">android@videolan.org</a> instead ;)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 25, 2014 at 2:24 PM, Andrey Khalyavin <span dir="ltr"><<a href="mailto:halyavin@google.com" target="_blank">halyavin@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This patch makes code nicer and getStorageDirectories() does not<br>
return null on exception any more.<br>
<br>
With best regards,<br>
<br>
Andrey Khalyavin<br>
<br>
>From 1fb0f9d19fbefbf30e8019a47a3b241c5fe1f6b0 Mon Sep 17 00:00:00 2001<br>
From: Andrey Khalyavin <<a href="mailto:halyavin@google.com">halyavin@google.com</a>><br>
Date: Tue, 25 Nov 2014 15:31:43 +0300<br>
Subject: [PATCH] Do not return null on exception in<br>
 AndroidDevices.getStorageDirectories().<br>
<br>
---<br>
 vlc-android/src/org/videolan/vlc/util/AndroidDevices.java | 8 +-------<br>
 1 file changed, 1 insertion(+), 7 deletions(-)<br>
<br>
diff --git a/vlc-android/src/org/videolan/vlc/util/AndroidDevices.java<br>
b/vlc-android/src/org/videolan/vlc/util/AndroidDevices.java<br>
index d71f55f..ae8f473 100644<br>
--- a/vlc-android/src/org/videolan/vlc/util/AndroidDevices.java<br>
+++ b/vlc-android/src/org/videolan/vlc/util/AndroidDevices.java<br>
@@ -88,7 +88,6 @@ public class AndroidDevices {<br>
     }<br>
<br>
     public static String[] getStorageDirectories() {<br>
-        String[] dirs = null;<br>
         BufferedReader bufReader = null;<br>
         ArrayList<String> list = new ArrayList<String>();<br>
         list.add(Environment.getExternalStorageDirectory().getPath());<br>
@@ -127,11 +126,6 @@ public class AndroidDevices {<br>
                 if (Strings.StartsWith(deviceWL, device) &&<br>
(typeWL.contains(type) || Strings.StartsWith(mountWL, mountpoint)))<br>
                     list.add(mountpoint);<br>
             }<br>
-<br>
-            dirs = new String[list.size()];<br>
-            for (int i = 0; i < list.size(); i++) {<br>
-                dirs[i] = list.get(i);<br>
-            }<br>
         }<br>
         catch (FileNotFoundException e) {}<br>
         catch (IOException e) {}<br>
@@ -143,7 +137,7 @@ public class AndroidDevices {<br>
                 catch (IOException e) {}<br>
             }<br>
         }<br>
-        return dirs;<br>
+        return list.toArray(new String[list.size()]);<br>
     }<br>
<br>
     public static String[] getMediaDirectories() {<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.1.0.rc2.206.gedb03e5<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
</font></span></blockquote></div><br></div>