demus/Gadgetbridge
1
0
Fork 0
mirror of https://codeberg.org/Freeyourgadget/Gadgetbridge.git synced 2026-07-31 07:44:24 +02:00
Code Issues 1 Packages Projects Releases Wiki Activity

ble: always use the TransactionBuilder provided to createTransactionBuilder

Browse Source 
The use of initializeDevice's TransactionBuilder was inconsistent:
onServicesDiscovered used the input argument
performInitialized used the return
This commit is contained in:
Thomas Kuehne
2025-07-01 23:04:03 +02:00
committed by José Rebelo
parent cb9db98480
commit d98e51190d
2 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/btle/AbstractBTLEMultiDeviceSupport.java
+2 -1
View File
@@ -249,7 +249,8 @@ public abstract class AbstractBTLEMultiDeviceSupport extends AbstractBTLEDeviceS
// first, add a transaction that performs device initialization
TransactionBuilder builder = createTransactionBuilder("Initialize device");
builder.add(new CheckInitializedAction(devices[deviceIdx]));
initializeDevice(builder, deviceIdx).queue(getQueue(deviceIdx));
initializeDevice(builder, deviceIdx);
builder.queue(getQueue(deviceIdx));
}
return createTransactionBuilder(taskName);
}
app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/btle/AbstractBTLESingleDeviceSupport.java
+2 -1
View File
@@ -184,7 +184,8 @@ public abstract class AbstractBTLESingleDeviceSupport extends AbstractBTLEDevice
// first, add a transaction that performs device initialization
TransactionBuilder builder = createTransactionBuilder("Initialize device");
builder.add(new CheckInitializedAction(gbDevice));
initializeDevice(builder).queue(getQueue());
initializeDevice(builder);
builder.queue(getQueue());
}
return createTransactionBuilder(taskName);
}