In `commonMain` I keep getting a warning: > Extension is shadowed by a member Now, I don't care, ...

# multiplatform

[Edoardo Luppi]

In commonMain I keep getting a warning:

Extension is shadowed by a member

Now, I don't care, I'm happy it's shadowed and I want to suppress it.

So I do that, and now it says "redundant suppression" 😞
Can I suppress the "redundant suppression"? lol

[mbonnin]

probably #gradle

[mbonnin]

Ah wait.. maybe not?

[Edoardo Luppi]

@mbonnin I think it's really a Kotlin "core" issue here.
I mean I'm ok with it, but I'd like to suppress the suppression warning

[mbonnin]

Yea, commonMain , sorry I somehow expected that in build.gradle.kts

[mbonnin]

Can you share a snippet that reproduces it?

[Edoardo Luppi]

Sure. Have you got a multiplatform project ready to use?

[mbonnin]

Always!

[mbonnin]

Although TBH this feels like a bug

[Edoardo Luppi]

in `commonMain`:

public expect class ZDeferred<T>

public expect inline fun <T, S> ZDeferred<T>.then(noinline thenFn: (T) -> S): ZDeferred<S>

in `jsMain`:

@Suppress("ACTUAL_TYPE_ALIAS_TO_CLASS_WITH_DECLARATION_SITE_VARIANCE", "ACTUAL_WITHOUT_EXPECT")
public actual typealias ZDeferred<T> = Promise<T>

@Suppress("EXTENSION_SHADOWED_BY_MEMBER")
public actual inline fun <T, S> ZDeferred<T>.then(noinline thenFn: (T) -> S): ZDeferred<S> =
  then(onFulfilled = thenFn)

[Edoardo Luppi]

Always amazing to see the amount of suppressions lmao

[mbonnin]

I don't have any warning here

[mbonnin]

Except the

Expected performance impact from inlining is insignificant. Inlining works best for functions with parameters of functional types

[Edoardo Luppi]

So you compile and you get no warnings?

[mbonnin]

Yep,

> Task :compileKotlinJvm
w: file:///Users/mbonnin/git/test-suppression/src/jvmMain/kotlin/Main.jvm.kt:3:8 Expected performance impact from inlining is insignificant. Inlining works best for functions with parameters of functional types

Same for JS:

> Task :compileKotlinJs
w: file:///Users/mbonnin/git/test-suppression/src/jsMain/kotlin/Main.js.kt:3:8 Expected performance impact from inlining is insignificant. Inlining works best for functions with parameters of functional types

[mbonnin]

https://github.com/martinbonnin/test-suppression

[Edoardo Luppi]

Ok I'll pull it and I'll try to repro

[Edoardo Luppi]

Ahhh you forgot to assign the typealias to a JS Promise<T>

[Edoardo Luppi]

in jsMain it should be

@Suppress("ACTUAL_TYPE_ALIAS_TO_CLASS_WITH_DECLARATION_SITE_VARIANCE", "ACTUAL_WITHOUT_EXPECT")
actual typealias ZDeferred<T> = Promise<T>

[mbonnin]

Ah good catch 👍

[Edoardo Luppi]

You'll see the inline warning will go away, replaced by the shadowed one

[mbonnin]

yep, now I have it 👍

[mbonnin]

I guess it's a Youtrack

[Edoardo Luppi]

So you can suppress it in the JS side, but it will keep bugging you in common.

I guess it's a Youtrack

I guess so, I'll create it tonight.

In the meantime, do you happen to know if something similar to

@Suppress("REDUNDANT_SUPPRESSION")

exists?

[mbonnin]

No idea at all, sorry.

[mbonnin]

It's "just" an IDE thing though, right?

[Edoardo Luppi]

Yeah, it really bugs me. My poor OCD lol

[mbonnin]

I'm hoping future KTIJ update should fix this?

[Edoardo Luppi]

I would hope so. I'll open the issue and link it back here so you can follow it.

[mbonnin]

You can "disable" the inspection

[Edoardo Luppi]

AHHHHHH! DAMN!
I got so used to Kotlin own suppressions that I forgot about the IDE ones.
Yup, just use

@Suppress("EXTENSION_SHADOWED_BY_MEMBER", "KotlinRedundantDiagnosticSuppress")

[Edoardo Luppi]

Worth a YT issue anyway tho

[Jeff Lockhart]

@Suppress("KotlinRedundantDiagnosticSuppress") might be what you're looking for.

[Edoardo Luppi]

@Jeff Lockhart yup exactly, Martin pointed out it was an IDE suppression, and not a Kotlin one

[Jeff Lockhart]

Ah, yes, I see you did reach that conclusion. 👌🏼