caspervk/AndroidAPS
1
0
Fork 0
Code Releases Activity

combov2: Minor cleanup to reduce some code smell

Browse source 
* Kotlin code convention specifies that ignored exceptions must be named
  "_", not "ignored."
* Add explanation about empty overridden method implementations.

Signed-off-by: Carlos Rafael Giani <crg7475@mailbox.org>
This commit is contained in:
Carlos Rafael Giani 2022-11-06 16:29:07 +01:00
parent 8fda1ab0b8
commit cdfb197c54
2 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -619,7 +619,7 @@ class ComboV2Plugin @Inject constructor (
}
} catch (e: CancellationException) {
throw e
} catch (ignored: Exception) {
} catch (_: Exception) {
}
}
@ -1370,7 +1370,7 @@ class ComboV2Plugin @Inject constructor (
suspend fun providePairingPIN(pairingPIN: PairingPIN) {
try {
pairingPINChannel?.send(pairingPIN)
} catch (ignored: ClosedSendChannelException) {
} catch (_: ClosedSendChannelException) {
}
}
@ -1384,8 +1384,8 @@ class ComboV2Plugin @Inject constructor (
val pump = pumpManager?.acquirePump(bluetoothAddress) ?: return@runBlocking
pump.unpair()
pumpManager?.releasePump(bluetoothAddress)
} catch (ignored: ComboException) {
} catch (ignored: BluetoothException) {
} catch (_: ComboException) {
} catch (_: BluetoothException) {
}
}

4
pump/combov2/src/main/kotlin/info/nightscout/androidaps/plugins/pump/combov2/activities/ComboV2PairingActivity.kt
View file

@ -69,9 +69,13 @@ class ComboV2PairingActivity : NoSplashAppCompatActivity() {
// xxx xxx xxxx
binding.combov2PinEntryEdit.addTextChangedListener(object : TextWatcher {
override fun beforeTextChanged(s: CharSequence?, start: Int, count: Int, after: Int) {
// Nothing needs to be done here; overridden method only exists
// to properly and fully implement the TextWatcher interface.
}
override fun onTextChanged(s: CharSequence?, start: Int, before: Int, count: Int) {
// Nothing needs to be done here; overridden method only exists
// to properly and fully implement the TextWatcher interface.
}
override fun afterTextChanged(s: Editable?) {