drgreen/scrcpy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Create UhidManager only on first use

Browse Source 
There is no need to create a UhidManager instance (with its thread) if
no UHID is used.
This commit is contained in:
Romain Vimont 2024-02-24 22:38:32 +01:00
parent 4f74b9210d
commit ba6c295501
1 changed files with 13 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
server/src/main/java/com/genymobile/scrcpy/Controller.java
View File

@ -26,7 +26,7 @@ public class Controller implements AsyncProcessor {
private Thread thread;
private final UhidManager uhidManager;
private UhidManager uhidManager;
private final Device device;
private final ControlChannel controlChannel;
@ -52,7 +52,13 @@ public class Controller implements AsyncProcessor {
this.powerOn = powerOn;
initPointers();
sender = new DeviceMessageSender(controlChannel);
uhidManager = new UhidManager(sender);
}
private UhidManager getUhidManager() {
if (uhidManager == null) {
uhidManager = new UhidManager(sender);
}
return uhidManager;
}
private void initPointers() {
@ -98,7 +104,9 @@ public class Controller implements AsyncProcessor {
// this is expected on close
} finally {
Ln.d("Controller stopped");
uhidManager.closeAll();
if (uhidManager != null) {
uhidManager.closeAll();
}
listener.onTerminated(true);
}
}, "control-recv");
@ -187,10 +195,10 @@ public class Controller implements AsyncProcessor {
Device.rotateDevice();
break;
case ControlMessage.TYPE_UHID_CREATE:
uhidManager.open(msg.getId(), msg.getData());
getUhidManager().open(msg.getId(), msg.getData());
break;
case ControlMessage.TYPE_UHID_INPUT:
uhidManager.write(msg.getId(), msg.getData());
getUhidManager().write(msg.getId(), msg.getData());
break;
default:
// do nothing