karelpeeters
10/02/2017, 7:26 AMdave08
10/02/2017, 7:30 AMdave08
10/02/2017, 7:42 AMdave08
10/02/2017, 7:49 AMbod
10/02/2017, 8:30 AMbod
10/02/2017, 8:54 AMkarelpeeters
10/02/2017, 9:00 AMbod
10/02/2017, 9:02 AMdave08
10/03/2017, 9:18 AMdave08
10/03/2017, 9:19 AMdave08
10/03/2017, 9:38 AMmegamiun
10/05/2017, 2:51 AMedvin
10/06/2017, 7:48 PMedvin
10/06/2017, 7:49 PMmkobit
10/06/2017, 8:48 PMInt
)
any thoughts on if this should be files as an issue or not?Shawn
10/06/2017, 8:53 PMShawn
10/06/2017, 8:53 PMUnit
as a return type is redundant because functions that don’t have a specified return type are inferred to return Unit
anyhowmkobit
10/06/2017, 8:56 PMfun thisFunction(myThing: MyThing) = myThing.doOtherThing()
you may want to clarify the return type for these single expression functions - does that make sense?mkobit
10/06/2017, 8:57 PMmyThing.doOtherThing()
type signature changes or something down the line - my main question was that the IntelliJ intention says it is redundant for Unit
but not for other types like Int
mkobit
10/06/2017, 8:57 PM{}
because you always need to specify return type if it is not Unit
Shawn
10/06/2017, 9:01 PMShawn
10/06/2017, 9:01 PMShawn
10/06/2017, 9:02 PMmkobit
10/06/2017, 9:02 PMUnit
because IntelliJ suggested itShawn
10/06/2017, 9:03 PMUnit
to anything else, that probably warrants a renaming of the function, or maybe marking it as @Deprecated
and replacing it with a more appropriately named callShawn
10/06/2017, 9:04 PMnapperley
10/07/2017, 1:45 AMnapperley
10/07/2017, 1:52 AMyole
10/07/2017, 7:56 AM