take advantage of Cursor being Closeable

alexcohn · Sep 8, 2020 · 765cb5c · 765cb5c
1 parent 80d4b9f
commit 765cb5c
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/android/app/src/main/java/com/arthenica/mobileffmpeg/Config.java b/android/app/src/main/java/com/arthenica/mobileffmpeg/Config.java
@@ -781,16 +781,12 @@ static List<FFmpegExecution> listFFmpegExecutions() {
     private static String getSafParameter(Context context, Uri uri, String openMode) {
 
         String displayName = "unknown";
-        final Cursor cursor = context.getContentResolver().query(uri, null, null, null, null);
-        try {
+        try (Cursor cursor = context.getContentResolver().query(uri, null, null, null, null)) {
             if (cursor != null && cursor.moveToFirst()) {
                 displayName = cursor.getString(cursor.getColumnIndex(DocumentsContract.Document.COLUMN_DISPLAY_NAME));
             }
         } catch (Throwable ex) {
             Log.e(TAG, "failed to get column", ex);
-        } finally {
-            if (cursor != null)
-                cursor.close();
         }
 
         int fd = -1;