Parameter inlay hint offsetting the argument names

[bjorn3]

[bjorn3]

The variable module is not defined, but I would expect it either not give parameter inlay hints, or show the correct names based on the only function called declare_data.

[lnicola]

Also, the linkage hint is on the wrong argument.

[bjorn3]

Yes, also writable is the first bool and tls is the second bool. I also notice now that id should be called name, so I guess there is another function which is also called declare_data.

[flodiebold]

What does 'go to definition' on the method call go to?

[bjorn3]

It goes to https://docs.rs/cranelift-module/0.56.0/cranelift_module/trait.Backend.html#tymethod.declare_data instead of the https://docs.rs/cranelift-module/0.56.0/cranelift_module/struct.Module.html#method.declare_data I meant. The former does indeed have id has first param, but that method has one more param than the later (it has one too much for this call)

[flodiebold]

Method resolution doesn't look at the parameters (except self)... I'm wondering why it got the idea that module implements this trait though -- is there any kind of blanket impl for it?

[bjorn3]

I don't think there is any blanket impl for Backend.

[to-mas-kral]

Also, is there a reason why there are no hints for non-literals ?

[SomeoneToIgnore]

Also, is there a reason why there are no hints for non-literals ?

This is the reason, I believe: #1618

[bjorn3]

I haven't seen this problem in a long time.