From 265ffe59fbd7c01e2bd5a8c6dcac9035c00de004 Mon Sep 17 00:00:00 2001 From: Po Lu Date: Wed, 29 Nov 2023 10:46:42 +0800 Subject: Clean up some redundant or otherwise suspect code * java/org/gnu/emacs/EmacsOpenActivity.java (onCreate): Don't check if URI is non-NULL, for instanceof checks this also. * java/org/gnu/emacs/EmacsSafThread.java (accessDocument1): Don't check !writable twice. * java/org/gnu/emacs/EmacsService.java (documentIdFromName): Designate this function as public, since it is called through JNI. --- java/org/gnu/emacs/EmacsSafThread.java | 3 --- 1 file changed, 3 deletions(-) (limited to 'java/org/gnu/emacs/EmacsSafThread.java') diff --git a/java/org/gnu/emacs/EmacsSafThread.java b/java/org/gnu/emacs/EmacsSafThread.java index 1b62662b4fc..7917e2d4880 100644 --- a/java/org/gnu/emacs/EmacsSafThread.java +++ b/java/org/gnu/emacs/EmacsSafThread.java @@ -1430,9 +1430,6 @@ public final class EmacsSafThread extends HandlerThread /* If so, don't check for FLAG_SUPPORTS_WRITE. Check for FLAG_DIR_SUPPORTS_CREATE instead. */ - if (!writable) - return 0; - index = cursor.getColumnIndex (Document.COLUMN_FLAGS); if (index < 0) return -3; -- cgit v1.3