Joseph Burton
Joseph Burton
This PR removes some hardcoded lists of mixin annotations, and adds some code to detect if an annotation is an injector, and assigns a default handler to it if no...
Currently the auto-reporter is spammy which makes it hard to process the actual issues. I propose the following changes to the auto-reporter: - *Edit* existing issues to specify the number...
Currently the mixin is always added to the `mixins` array in mixin configs, however if the target class is client-side-only, it should be easy to infer that it should go...
This PR rewrites the SideOnly inspections. For details, see [my gist](https://gist.github.com/Earthcomputer/c9aded5e879fbad6b84ab513e121a16c). - [x] Add a new SideOnly inspection to replace the old ones. - [x] Add `SideOnly`, `OnlyIn` and `Environment`...
Since Fabric now supports access wideners, it would be very helpful to add a few features * Highlight access widener files and allow for jumping to the reference * Detect...
It's an easier quick reference than the wiki, if you forget the order.
It's impossible to modify a null constant (that is, an `aconst_null` instruction) with `@ModifyConstant`. This seems unintentional as `@Constant(nullValue = true)` is a thing. Although I'm a little unsure exactly...
Example: ```java public class MultipleConstructors { public MultipleConstructors() {} public MultipleConstructors(int i) {} } public class TargetClass { public void foo() { new MultipleConstructors(); // new MultipleConstructors; invokespecial MultipleConstructors.()V new...
... To end the reflection hacks and improper ASM once and for all! :P Rationale --- - Sometimes adding enum constants is the only way to do something you want,...
For example, if I try to target `*()Lnet/minecraft/world/gen/GeneratorOptions;` in the `GeneratorOptions` class in Fabric, it works in dev, but since the refmap does not contain this entry, it breaks in...