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.

3 files changed, 2 insertions, 6 deletions

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);
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;
diff --git a/java/org/gnu/emacs/EmacsService.java b/java/org/gnu/emacs/EmacsService.java
index 3cc37dd992d..33832505333 100644
--- a/java/org/gnu/emacs/EmacsService.java
+++ b/java/org/gnu/emacs/EmacsService.java
@@ -1413,7 +1413,7 @@ public final class EmacsService extends Service
      of OperationCanceledException, SecurityException,
      FileNotFoundException, or UnsupportedOperationException.  */
 
-  private int
+  public int
   documentIdFromName (String tree_uri, String name, String[] id_return)
   {
     /* Start the thread used to run SAF requests if it isn't already