android-studio-inspection.png

# announcements

[nickk]

android-studio-inspection.png

[Martin Devillers]

If the last argument to a method is a a lambda, it’s more idiomatic to use the syntax where the lambda is moved out of the list of arguments in parentheses.
Prefer

doSomething { }

Rather than

doSomething({})

[leolima]

you can use alt + enter to apply the “fix”

[nickk]

Isn’t there a difference between “can be moved out” and “should be moved out” ?

[Lucas Ł]

its not an error, so you don't have to do it 😉

[leolima]

it is also possible to disable the inspection from the alt + enter menu

[Martin Devillers]

It’s just a convention that Kotlin style prefers for the lambda to be moved out

[nickk]

In this case of complex code, I think the parentheses serve as a visual boundary.

[leolima]

the style guide recommends:

If a call takes a single lambda, it should be passed outside of parentheses whenever possible.

https://kotlinlang.org/docs/reference/coding-conventions.html#lambda-formatting

[nickk]

@leolima In AS 3.1.4, the context menu with Alt+Enter has no option to silence the warning

[leolima]

that’s weird… it should be in the expanded menu

[nickk]

Oh well… We spent too much time on it already. AS 3.1 has plenty of bugs. I was forced to rollback to 3.0 twice.