I got a question about `@Deprecated` and `ReplaceW...

# announcements

[edwardwongtl]

I got a question about @Deprecated and ReplaceWith(), while replacing normal functions works fine, replacing infix function will be like this

@Deprecated(..., replaceWith = ReplaceWith("bar(x)"))
infix fun Foo.foo(x: Int) = ...
infix fun Foo.bar(x: Int) = ...

foo foo 10
// After replacement becomes
foo.bar(10)

is there anyway for the IDE do the replacement such that it becomes foo bar 10?

[gildor]

This replacement works for me, I use Kotlin Plugin 1.2.40

[edwardwongtl]

1.2.40 as well here, could you try the other example?

[gildor]

I tried your example above, works perfectly

[edwardwongtl]

How about a + b foo 10 into a + b bar 10?

[edwardwongtl]

I can only get (a + b).bar(10) or (a+b) bar 10

[gildor]

yes, (a + b).bar(10)

[gildor]

but it’s correct

[gildor]

I see what you mean

[edwardwongtl]

I know it's correct, just does look nice

[edwardwongtl]

The IDE can do better

[gildor]

Create an issue

[gildor]

kotl.in/issue

[gildor]

btw (a + b) foo 10 works perfectly, replaced to (a + b) bar 10

[gildor]

actually this style

a + b foo 10

is very confusing, probably IDE should warn user to avoid it and ask to add parenthesis

[edwardwongtl]

Maybe this example is confusing, but I am doing a bindTo b until event in my project, which should be better?

[gildor]

yes, you right, if you have only infix function it’s much better and not confusing

[edwardwongtl]

I guess I'll let JetBrains decide which approach is better, warning for parenthesis or better replacement.