MinecraftDev icon indicating copy to clipboard operation
MinecraftDev copied to clipboard

correct targets often cannot be identified when an ordinal is already specified

Open tildejustin opened this issue 1 year ago • 2 comments

Minecraft Development for IntelliJ plugin version

2023.3-1.6.12-452

IntelliJ version

2023.3.2 (Ultimate Edition)

Operating System

EndeavorOS Linux, kernel 6.7.0-arch3-1

Target platform

Mixins

Description of the bug

I don't quite know how to articulate this, but whenever you write a mixin and set the ordinal, to change what is targeted you have to remove the ordinal for mcdev to recognize the possibilities. This happens to me all the time, recently I was fixing this old style NEW invocation. image The only option given to me is not the one I'm looking for. image I remove the ordinal and the option i'm looking for shows up. image And after adding the ordinal back it can find the referenced code if I use the "go to target" button, but the issue at the start with it not showing up still is a problem. image

tildejustin avatar Jan 22 '24 04:01 tildejustin

This is clearly a bug so I'm keeping this open, but a tip for code style, you should use slice instead of such high ordinal values.

Earthcomputer avatar Jan 22 '24 12:01 Earthcomputer

this was just a bit of maintenance work on an old version of this mod to make a hotfix so I didn't really want to make more changes then necessary, but I'll keep that in mind for the future. thanks!

tildejustin avatar Jan 22 '24 16:01 tildejustin