https://kotlinlang.org logo
Channels
Powered by
#kobalt
  • k

    kirillrakhman

    12/27/2015, 6:42 PM
    -.txt
    -.txt
  • c

    cedric

    12/27/2015, 6:42 PM
    (the issues was incorrect bytecode generation and a few others)
  • c

    cedric

    12/27/2015, 6:43 PM
    mmmh ok this seems to be coming from the new compiler indeed, guess I should upgrade
  • k

    kirillrakhman

    12/27/2015, 6:43 PM
    dang, "Error:(158, 21) Kotlin: 'return' is not allowed here" is caused because of the default method from JDK8 winning over the extension method
  • c

    cedric

    12/27/2015, 6:43 PM
    So wait, I want to make sure this is a legitimate error and not a bug in the current compiler
  • c

    cedric

    12/27/2015, 6:44 PM
    Not convinced it’s legitimate right now
  • c

    cedric

    12/27/2015, 6:44 PM
    If you can fix the error and kobalt seems to work fine, I’ll update
  • k

    kirillrakhman

    12/27/2015, 6:45 PM
    you can mitigate this by using 
    kotlin.List
    instead of 
    ArrayList
  • k

    kirillrakhman

    12/27/2015, 6:45 PM
    either by casting or by using 
    listOf
    instead of 
    arrayListOf
  • c

    cedric

    12/27/2015, 6:46 PM
    Where exactly?
  • k

    kirillrakhman

    12/27/2015, 6:46 PM
    RepoFinder line 147
  • c

    cedric

    12/27/2015, 6:47 PM
    Oh yeah and it should be a 
    List
    anyway
  • k

    kirillrakhman

    12/27/2015, 6:47 PM
    in KFiles 286, the 
    forEach
    can by replaced by a 
    map
  • k

    kirillrakhman

    12/27/2015, 6:47 PM
    I'll make a PR
  • c

    cedric

    12/27/2015, 6:47 PM
    Awesome
  • c

    cedric

    12/27/2015, 6:48 PM
    It’s interesting to read this code I wrote just a few months ago and that I would write completely differently today
  • k

    kirillrakhman

    12/27/2015, 6:48 PM
    it always is
  • k

    kirillrakhman

    12/27/2015, 6:49 PM
    man the new overloading rules kinda suck
  • k

    kirillrakhman

    12/27/2015, 6:49 PM
    Copy code
    env.forEach {it ->
                pbEnv.put(it.key, it.value)
            }
    and
    Copy code
    env.forEach {
                pbEnv.put(it.key, it.value)
            }
    have different meanings now
  • c

    cedric

    12/27/2015, 6:50 PM
    Yeah I saw that, not crazy about it. Hopefully it doesn’t happen too often
  • k

    kirillrakhman

    12/27/2015, 6:51 PM
    it did in kobalt
  • k

    kirillrakhman

    12/27/2015, 6:51 PM
    oh jeses
  • c

    cedric

    12/27/2015, 6:51 PM
    Does the compiler complain or does it compile silently except that now it behaves differently?
  • k

    kirillrakhman

    12/27/2015, 6:52 PM
    it doesn't compile
  • k

    kirillrakhman

    12/27/2015, 6:52 PM
    there is a method 
    Thread.yield
  • k

    kirillrakhman

    12/27/2015, 6:52 PM
    which cannot be used without backticks anymore
  • c

    cedric

    12/27/2015, 6:52 PM
    Unfortunate but not the end of the world to me, I don’t use that method very often
  • k

    kirillrakhman

    12/27/2015, 6:53 PM
    once to be pricese
  • k

    kirillrakhman

    12/27/2015, 6:54 PM
    how to tell kobalt which version of kotlin to use?
  • c

    cedric

    12/27/2015, 6:54 PM
    KotlinCompiler.kt:36
1...189190191...826Latest