caspervk/AndroidAPS
1
0
Fork 0
Code Releases Activity

combov2: Disable internal Pump.connect() connection attempt limiter

Browse source 
QueueThread already has a timeout; the limiter is redundant.

Signed-off-by: Carlos Rafael Giani <crg7475@mailbox.org>
This commit is contained in:
Carlos Rafael Giani 2022-11-20 23:05:24 +01:00
parent 90480ab1d9
commit 61519e43f2
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
pump/combov2/src/main/kotlin/info/nightscout/androidaps/plugins/pump/combov2/ComboV2Plugin.kt
View file

@ -498,7 +498,10 @@ class ComboV2Plugin @Inject constructor (
var forciblyDisconnectDueToError = false var forciblyDisconnectDueToError = false
try { try {
pump?.connect() // Set maxNumAttempts to null to turn off the connection attempt limit inside the connect() call.
// The AAPS queue thread will anyway cause the connectionSetupJob to be canceled when its
// connection timeout expires, so the Pump class' own connection attempt limiter is redundant.
pump?.connect(maxNumAttempts = null)
// No need to set the driver state here, since the pump's stateFlow will announce that. // No need to set the driver state here, since the pump's stateFlow will announce that.