emacs-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

master 617bd85cd0b: Correct typos in and omissions from earlier Android


From: Po Lu
Subject: master 617bd85cd0b: Correct typos in and omissions from earlier Android port changes
Date: Fri, 5 Apr 2024 23:19:29 -0400 (EDT)

branch: master
commit 617bd85cd0bd1d0a73d405d623ee3dd41c7e8a37
Author: Po Lu <luangruo@yahoo.com>
Commit: Po Lu <luangruo@yahoo.com>

    Correct typos in and omissions from earlier Android port changes
    
    * java/AndroidManifest.xml.in: Remove duplicate permissions.
    
    * java/org/gnu/emacs/EmacsWindowManager.java
    (removeWindowConsumer): Disregard isFinishing for non-initial
    activities.
---
 java/AndroidManifest.xml.in                |  5 +----
 java/org/gnu/emacs/EmacsWindowManager.java | 13 ++++++++-----
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/java/AndroidManifest.xml.in b/java/AndroidManifest.xml.in
index 563914fb02c..f1047ac41d8 100644
--- a/java/AndroidManifest.xml.in
+++ b/java/AndroidManifest.xml.in
@@ -31,7 +31,7 @@ along with GNU Emacs.  If not, see 
<https://www.gnu.org/licenses/>. -->
          android:versionName="@version@">
 
   <!-- Paste in every permission in existence so Emacs can do
-       anything.  -->
+       everything.  -->
 
   <uses-permission android:name="android.permission.READ_CONTACTS" />
   <uses-permission android:name="android.permission.WRITE_CONTACTS" />
@@ -95,8 +95,6 @@ along with GNU Emacs.  If not, see 
<https://www.gnu.org/licenses/>. -->
   <uses-permission android:name="android.permission.READ_MEDIA_VIDEO" />
   <uses-permission 
android:name="android.permission.READ_MEDIA_VISUAL_USER_SELECTED" />
   <uses-permission android:name="android.permission.READ_PHONE_NUMBERS" />
-  <uses-permission android:name="android.permission.READ_SMS" />
-  <uses-permission android:name="android.permission.RECEIVE_MMS" />
   <uses-permission android:name="android.permission.RECEIVE_WAP_PUSH" />
   <uses-permission android:name="android.permission.SCHEDULE_EXACT_ALARM" />
   <uses-permission 
android:name="android.permission.SMS_FINANCIAL_TRANSACTIONS" />
@@ -185,7 +183,6 @@ along with GNU Emacs.  If not, see 
<https://www.gnu.org/licenses/>. -->
   <uses-permission android:name="android.permission.USE_EXACT_ALARM" />
   <uses-permission android:name="android.permission.USE_FINGERPRINT" />
   <uses-permission android:name="android.permission.WRITE_PROFILE" />
-  <uses-permission android:name="android.permission.WRITE_SMS" />
   <uses-permission android:name="android.permission.WRITE_SOCIAL_STREAM" />
   <uses-permission android:name="android.permission.WRITE_SYNC_SETTINGS" />
   <uses-permission android:name="android.permission.WRITE_USER_DICTIONARY" />
diff --git a/java/org/gnu/emacs/EmacsWindowManager.java 
b/java/org/gnu/emacs/EmacsWindowManager.java
index a239fdc8ac2..41ea3a15809 100644
--- a/java/org/gnu/emacs/EmacsWindowManager.java
+++ b/java/org/gnu/emacs/EmacsWindowManager.java
@@ -221,11 +221,14 @@ public final class EmacsWindowManager
       {
        consumer.detachWindow ();
 
-       /* Though pruneWindows will likely remove the same windows, call
-          onActivityDetached anyway if isFinishing is set, as in
-          obscure circumstances pruneWindows will not remove frames
-          bound to the system-started task.  */
-       if (isFinishing)
+       /* Though pruneWindows will likely remove the same window(s),
+          call onActivityDetached anyway if isFinishing is set, if
+          CONSUMER not be a multitasking activity, as in obscure
+          circumstances pruneWindows will not remove frames bound to
+          the system-started task.  */
+       if (isFinishing
+           && (!(consumer instanceof EmacsMultitaskActivity)
+               || Build.VERSION.SDK_INT < Build.VERSION_CODES.Q))
          window.onActivityDetached ();
       }
 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]