Is there an `@Ignore` declaration that I can add t...

# intellij

[loke]

Is there an @Ignore declaration that I can add that will silence the warning: Expected performance impact from inlining is insignificant. Inlining works best for functions with parameters of function types.

In my case, I have benchmarked, and it does make a small difference (which matters, because this is in the inner loop of a language interpreter) so I definitely want to keep it.

[Riccardo Lippolis]

you can annotate the function with @Suppress("NOTHING_TO_INLINE")

[loke]

Thanks a lot. That's exactly what I was looking for. For the future, are these listed anywhere?

[Riccardo Lippolis]

I normally discover these options by using the context menu on the warning (alt+enter), and in the submenu for the warning quickfix the relevant suppression is listed (if present)

[loke]

Right, but for this specific warning, there is no such hint.

[loke]

I also note that NOTHING_TO_INLINE appears to not be what I need. This warning is when I inline a function that does not accept a function argument, such as this:

inline fun foo(x: String): Int = x.length

[loke]

If I add NOTHING_TO_INLINE to that, then Idea complains that the suppression is unnecessary.

[Riccardo Lippolis]

oh weird, when I use your code, I get the exact same warning you mentioned

[Riccardo Lippolis]

then I go to more actions (alt+enter) and get the menu I showed earlier

[Riccardo Lippolis]

and then the warning is gone when I apply the suppression

[loke]

Oh wait, you're right. Sometimes it gives me a highlight, and sometimes not.

[loke]

And I never get the warning in IDEA itself, only when I build with gradle.

[loke]

It looks all fine in the editor:

[Riccardo Lippolis]

hmm, when I add the suppression the gradle compile warning also disappears

[loke]

I have to admit, I never tried recompiling with gradle, because when I added it, this warning showed up in the editor:

[Riccardo Lippolis]

and when you remove the suppression on that specific method, the inline warning appears again? 🤔

[Daniel Pitts]

There is actually a suppression for unnecessary suppression warnings. 🙂

[Daniel Pitts]

@Suppress("NOTHING_TO_INLINE", "KotlinRedundantDiagnosticSuppress")

[loke]

Wow. Thanks 🙂