*Disclaimer*: Since I use the `when` expression it...

# android

[Lilly]

Disclaimer: Since I use the when expression it is kotlin related 😌

Can someone tell me why lint is warning it would be always true? This warning is new and must have come up after doing some depedency updates/ IDE update.

[José González D'Amico]

Since updates were done, have you tried invalidate cache/restart? I remember vaguely having seen weird lint errors after updating libraries

[Lilly]

Unfortunately this didn't help. Shoud I reinstall? Any other ideas?

[José González D'Amico]

hmm other than reverting the library updates and see if the problem goes away, no

[Lilly]

ok easy. There is still a possibility this is a bug. I consider to file an issue

[José González D'Amico]

Wait, I read too quickly. Have you checked that connectAsClient is not being called with the secure parameter with a true value in all cases?

[Lilly]

@José González D'Amico good point but even when I remove the when it still warns about being always true. Meanwhile, I'm pretty sure this is a bug

[Chrimaeon]

From my point of view the warning is fine - you always return true in the run.

[Chrimaeon]

Move the return true outside the when call and you are fine.

[Lilly]

Damn you are right. But what has changed that it suddenly occurs?

[Chrimaeon]

Just a new lint check maybe. 🤷‍♂️

[Lilly]

Makes sense. Thanks boss

[Tuna]

I don't have the answer for you question, just suggest to rewrite

try {
    val bluetoothSocket = if (secure) {
        ...
    } else {
        ...
    }
    socket = bluetoothSocket
    bluetoothSocket.connect()
    true
} catch (e: IOException) {
    false
}