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/EmacsOpenActivity.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'java/org/gnu/emacs/EmacsOpenActivity.java') diff --git a/java/org/gnu/emacs/EmacsOpenActivity.java b/java/org/gnu/emacs/EmacsOpenActivity.java index 5cca6cfcdff..32a79d1a797 100644 --- a/java/org/gnu/emacs/EmacsOpenActivity.java +++ b/java/org/gnu/emacs/EmacsOpenActivity.java @@ -533,8 +533,7 @@ public final class EmacsOpenActivity extends Activity else uri = intent.getParcelableExtra (Intent.EXTRA_STREAM); - if (uri != null - && (scheme = uri.getScheme ()) != null + if ((scheme = uri.getScheme ()) != null && scheme.equals ("content")) { tem1 = EmacsService.buildContentName (uri); -- cgit v1.3