yeah the 1.2 plugin seems really buggy

any details? code examples, screenshots?

on sec

you know, these “seems really buggy” comments without much context are quite unpleasant to read for the developers…

I realize that, I am sorry. Can you reproduce this? 1. run something in debug mode, break on breakpoint 2. introduce error/warning/weak warning into code by changing something before the breakpoint 3. hover over error 4. no popup

if I stop the running process the popups work again

yes, this is standard IntelliJ behavior since forever. when debugging, hovering is used to show variable values instead of warning/error details

😄

ok nevermind then

i just updated to idea 2017.3 (with kotlin 1.2) and now it shows all the classes as unused (i can still click on the class in other classes and idea takes me there).

so it's definetly not unused 🙂

inval and restart?

will try .. 1 sec

@Czar commented on @Paul Woitaschek’s file https://kotlinlang.slack.com/files/U0KBF8D7V/F87H4AFQT/-.kt: Well, first of all, what does combineLatest do? Second, seems you're doing two things in one expression: calling function combineLatest which does something with params and creating FoodSummaryModel, why? I have this questions because you only shared small part of your code, if you share more, I'll probably understand, but that becomes code review 😉 And finally third, if you have lambda as last parameter, why not get it out of parenthesis?

@trevjones thx, did the trick!

@Czar commented on @Paul Woitaschek’s file https://kotlinlang.slack.com/files/U0KBF8D7V/F87H4AFQT/-.kt: If you have FoodSummaryModel which combines those same 4 params, probably you [should?] have something like FoodModel which you can pass instead of 4 params, plus you could use same FoodModel as param to FoodSummaryModel. Then you'd have something like:

val foodModel = FoodModel(amountOfEnergy, energyRecommendation, viewModels, recommendedFoods)
return combineLatest(foodModel, ::FoodSummaryModel)

@Czar commented on @Paul Woitaschek’s file https://kotlinlang.slack.com/files/U0KBF8D7V/F87H4AFQT/-.kt: What I've wrote could be utter nonsense of course depending on what is really going on in your app, hence: code review 🙂

@Paul Woitaschek commented on @Paul Woitaschek’s file https://kotlinlang.slack.com/files/U0KBF8D7V/F87H4AFQT/-.kt: I was really not looking for a code review 😄

@Paul Woitaschek commented on @Paul Woitaschek’s file https://kotlinlang.slack.com/files/U0KBF8D7V/F87H4AFQT/-.kt: You know; the day before you gave the tip add a line break before the .defer so the map was not completely in the middle of nowhere

@Paul Woitaschek commented on @Paul Woitaschek’s file https://kotlinlang.slack.com/files/U0KBF8D7V/F87H4AFQT/-.kt: That was really the same kind of question, purely code formatting related

@Czar commented on @Paul Woitaschek’s file https://kotlinlang.slack.com/files/U0KBF8D7V/F87H4AFQT/-.kt: Heh, then take

And finally third, if you have lambda as last parameter, why not get it out of parenthesis?

and maybe my second comment on the snippet, depending on it being nonsense or not 🙂 Cheers 🙂

@Czar commented on @Paul Woitaschek’s file https://kotlinlang.slack.com/files/U0KBF8D7V/F87H4AFQT/-.kt: Oh, and rename

a, b, c, d

to

a, e, v, r

if you're keeping 4 separate params 🙂

Okay I make an example that is less related to code

@Paul Woitaschek uploaded a file: Untitled

@Paul Woitaschek I think we've strayed off-topic here 🙂 Maybe next question of this sort should be in #random 🙂

Quick question, is there a way to execute

move lambda argument outside of parentheses

for whole file at once? This is a pretty frequent task for us when converting Java to Kotlin.

This refactor looks wrong to me. Want to make sure I'm not being dumb. First version:

for (element in this) if (!predicate(element)) return false
return true

says "if any element does not match the predicate, return false; otherwise, return true". Second version:

return this.any { predicate(it) }

says "if any element matches the predicate, return true; otherwise, return false". These are not equivalent, right?

I have this too, maybe that exists because run configiarations etc can be different depending on the branch you're on. It's definitely new since

2017.3

.

It's here as the first point under VCS: https://blog.jetbrains.com/idea/2017/11/intellij-idea-2017-3-smarter-coding-assistance-better-debugger-run-dashboard-frameworks-support-and-more/