when I execute this code, I expected to get a "fal...

# announcements

[marcelo]

when I execute this code, I expected to get a "false" but instead, I always get true. Not sure what I am doing wrong.

metrics.timer(timerName, successTag).time {
                throw IllegalArgumentException("Stop")`

Timer1.kt

[Shawn]

are you actually calling toTime() in your real code

[Shawn]

line 16 in your snippet @marcelo

[marcelo]

figured I would by just me adding things like:

metrics.timer(timerName, successTag).time {
                "Test".capitalize()            }

[marcelo]

figured that this would take the block of code, no?

[marcelo]

😕

[Shawn]

right but in the implementation of Timer1.time() it looks like you’re accepting a block but never actually executing it

[Shawn]

you just have

try {
  toTime

sans ()

[Shawn]

maybe you just omitted them when transposing to slack, but I figured it’s worth asking just in case you accidentally omitted them in your actual code

[marcelo]

Dude!!

[marcelo]

🐑 feel so sheepish

[marcelo]

thanks

[Shawn]

lmao no worries, we all make silly mistakes from time to time

[Shawn]

the IDE should warning highlight it though with The expression is unused

[Shawn]

though I guess that’s maybe kind of a vague warning when you’re intentionally not using the return value of the lambda in question

[Shawn]

also maybe consider making this an inline fun to reduce overhead, but I suppose it shouldn’t matter if you’re just comparing runs against each other

[marcelo]

thanks Shawn, just a bit shy of a month using Kotlin. Really liking it but I haven't seen the coroutines yet. I will look into it. What's the best place to get info - Kotlin docs?

[Shawn]

Kotlin docs are comprehensive and a great place to start

[Shawn]

Stackoverflow is where I go if I can’t find an answer in the docs