Channels
#intellij
- k
kirillrakhman
09/01/2017, 12:27 PMthose functions must be inlined - k
karelpeeters
09/01/2017, 12:28 PMCouldn't it generate code like this:Copy code
?if (roundUp) Math.ceil(0.5) else Math.floor(0.5) - k
kirillrakhman
09/01/2017, 12:30 PMmaybe. maybe there are trickier edge cases - k
kirillrakhman
09/01/2017, 12:30 PMI'm gonna file an issue, let's see what JB folks have to say👍 2 - k
kirillrakhman
09/01/2017, 12:34 PM - b
beholder
09/01/2017, 1:37 PMIt's not a bug. Compiler need somehow pass reference to Math::ceil (or ::floor), what is only possible in JVM before 8 by creating a class implemented Function1 - b
beholder
09/01/2017, 1:38 PMThis is inlined:val res = 0.5.let { if (roundUp) Math.ceil(it) else Math.floor(it) } - k
kirillrakhman
09/01/2017, 1:42 PMof course it's not a bug. it's just inconsistent - k
kirillrakhman
09/01/2017, 1:43 PMand maybe unexpected - b
beholder
09/01/2017, 1:44 PMYou can inline code, but not reference to the code - k
kirillrakhman
09/01/2017, 1:45 PM
is inlined0.5.let(Math::ceil) - b
beholder
09/01/2017, 1:45 PMMaybe because it is simple constant expression - k
kirillrakhman
09/01/2017, 1:46 PMwell, yes. because inlining supports lambdas and function/property references - k
kirillrakhman
09/01/2017, 1:47 PMtbf, my example is the same asCopy code
which also shouldn't be inlinedval f = Math::ceil 0.5.let(f) - b
beholder
09/01/2017, 1:50 PMYou are wanting too much from compiler. What resulting bytecode you are expecting in original example? - k
karelpeeters
09/01/2017, 1:51 PMNo we're expecting either a warning or a fix. - k
kirillrakhman
09/01/2017, 1:51 PMPlease read my statements and the issue - b
beholder
09/01/2017, 1:52 PMI think, compiler creating anonymous class is not "danger" it should warn about - a
andreiverdes
09/01/2017, 11:54 PMHello guys! Any idea why the suggestions for Kotlin aren't working like they do in Java? I would usually get the property in Java for this... - u
user
09/01/2017, 11:57 PM@andreiverdes commented on @andreiverdes’s file: looks like because thehttps://kotlinlang.slack.com/files/U6VFQ0CRL/F6WLVC00Y/screen_shot_2017-09-02_at_01.43.38.png▾
property isn't using a capitalmemorystore
, nvm 😁S - n
napperley
09/02/2017, 6:56 AMEncountering an issue where IntelliJ doesn't show the local JAR as a dependency (using Gradle 4.1 with Gradle Kotlin DSL), are there any plans to address this? - t
tapchicoma
09/02/2017, 7:28 AM@napperley open a new ticket here: kotl.in/issue - d
ddsoyka
09/03/2017, 7:41 PMWhy does IntelliJ report an unchecked cast inspection on line 3? Aren't I explicitly using a safe cast operation? Does the answer have something to do with type erasure? - d
diesieben07
09/03/2017, 7:44 PMYes, since generics are erased you cannot actually check if the object is a
or if it's aModel<T>
. You can only check if it's aModel<SomethingElse>
.Model<*> - d
ddsoyka
09/03/2017, 7:45 PMType erasure is the literal worst. Thanks! - d
diesieben07
09/03/2017, 7:47 PMYou'll have to somehow retain the actual type of
, probably using aT
. And no, type erasure is actually quite useful, most of the time. 😉Class<T> - d
ddsoyka
09/03/2017, 7:53 PMWhen does it come in handy? Genuinely curious, my first language was C#, so type erasure just presents as a nuisance to me - d
diesieben07
09/03/2017, 7:55 PMWell, for a start it greatly reduces the code size at runtime, there is one class for
, not 20.List - d
diesieben07
09/03/2017, 7:56 PMWhich easily lets you work with just "any kind of List" - d
ddsoyka
09/03/2017, 8:02 PMI did not know that, thank you 🙂