Fix: IllegalStateException: message is already in use

jobs_xie · jobs_xie · commit e04bfdf9eff1 · 2021-04-22T10:58:44.000+08:00
diff --git a/lib/utilcode/src/main/java/com/blankj/utilcode/util/MessengerUtils.java b/lib/utilcode/src/main/java/com/blankj/utilcode/util/MessengerUtils.java
@@ -317,15 +317,17 @@ public int onStartCommand(Intent intent, int flags, int startId) {
         }
 
         private void sendMsg2Client(final Message msg) {
+           final Message obtain = Message.obtain(msg); //Copy the original
             for (Messenger client : mClientMap.values()) {
                 try {
                     if (client != null) {
-                        client.send(Message.obtain(msg));
+                        client.send(obtain);
                     }
                 } catch (RemoteException e) {
                     e.printStackTrace();
                 }
             }
+            obtain.recycle(); //Recycled copy
         }
 
         private void consumeServerProcessCallback(final Message msg) {

Original file line number	Diff line number	Diff line change
`@@ -317,15 +317,17 @@ public int onStartCommand(Intent intent, int flags, int startId) {`
`317`	`317`	`}`
`318`	`318`
`319`	`319`	`private void sendMsg2Client(final Message msg) {`
	`320`	`+ final Message obtain = Message.obtain(msg); //Copy the original`
`320`	`321`	`for (Messenger client : mClientMap.values()) {`
`321`	`322`	`try {`
`322`	`323`	`if (client != null) {`
`323`		`- client.send(Message.obtain(msg));`
	`324`	`+ client.send(obtain);`
`324`	`325`	`}`
`325`	`326`	`} catch (RemoteException e) {`
`326`	`327`	`e.printStackTrace();`
`327`	`328`	`}`
`328`	`329`	`}`
	`330`	`+ obtain.recycle(); //Recycled copy`
`329`	`331`	`}`
`330`	`332`
`331`	`333`	`private void consumeServerProcessCallback(final Message msg) {`