Skip to content
Sign in
Success

Changes

Summary

  1. Close resource streams for mixin configs. (#185) (commit: 5250ed3) (details)
  2. Improvement: Avoid reflection in Bytecode opcode name utils. (#186) (commit: 76b38b8) (details)
  3. Fix: Respect injection point specifier with shift. (#187) (commit: a3cd815) (details)
  4. Improvement: Avoid ClassInfo lookups wherever possible without changing behaviour. (commit: 292070a) (details)
  5. Fix: Use null as a proper ClassInfo cache value. (commit: 2c6ea18) (details)
  6. Improvement: Remove warning for string-targeted public classes. (commit: 9f8bc5f) (details)
  7. Improvement: Check inner class visibility without using ClassInfos. (commit: b0be3b2) (details)
  8. Improvement: Only check for final method overrides if `debug.verify` is enabled. (commit: 10af730) (details)
Commit 5250ed3069c064b697d585177c6c90b347ef2aeb by noreply
Close resource streams for mixin configs. (#185)

* Close resource streams for mixin configs.

* Close resource streams for mixin configs.

* Simplify

* Remove Unneeded changes & Import

---------

Co-authored-by: cputnam-a11y <unknown@cputnam-a11y.github.io>
 (commit: 5250ed3)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/transformer/MixinConfig.java (diff)
Commit 76b38b8a2a1f0ebc5a309c569a0e30fbc386bcd9 by noreply
Improvement: Avoid reflection in Bytecode opcode name utils. (#186)

 (commit: 76b38b8)
The file was modifiedsrc/main/java/org/spongepowered/asm/util/Bytecode.java (diff)
Commit a3cd81528f0f12d94a2aafc18696fdd14833c1c7 by noreply
Fix: Respect injection point specifier with shift. (#187)

Gated since this is technically breaking.
 (commit: a3cd815)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/injection/InjectionPoint.java (diff)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/FabricUtil.java (diff)
The file was modifiedgradle.properties (diff)
Commit 292070a3789da55caa2e8842a220f36fb6941464 by l3gomindstorms
Improvement: Avoid ClassInfo lookups wherever possible without changing behaviour.
 (commit: 292070a)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/transformer/MixinInfo.java (diff)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/struct/MemberRef.java (diff)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/transformer/MixinTargetContext.java (diff)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/transformer/ClassInfo.java (diff)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/transformer/MixinPreProcessorStandard.java (diff)
Commit 2c6ea1834edc24b9107fad5b2408493515d81c27 by l3gomindstorms
Fix: Use null as a proper ClassInfo cache value.
 (commit: 2c6ea18)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/transformer/ClassInfo.java (diff)
Commit 9f8bc5fdcee0cebf51319eefd2275a775b8506b5 by l3gomindstorms
Improvement: Remove warning for string-targeted public classes.

This can be expensive to compute since it requires loading ClassInfos for outer classes, and it is a useless warning anyway which only worsens logspam.
 (commit: 9f8bc5f)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/transformer/MixinInfo.java (diff)
Commit b0be3b255095ac3f107d085086b86a4312b324d7 by l3gomindstorms
Improvement: Check inner class visibility without using ClassInfos.

The semantics of staticness are likely not identical, but I haven't been able to find a case where they differ. The old one is horribly heuristic anyway.
 (commit: b0be3b2)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/transformer/MixinInfo.java (diff)
Commit 10af73058a135df88140f10be3924c26b265baa2 by l3gomindstorms
Improvement: Only check for final method overrides if `debug.verify` is enabled.

The verifier will catch this anyway, this just gives us a better error which blames the particular mixin, but it's very expensive to check so best avoided in the default case.
 (commit: 10af730)
The file was modifiedsrc/main/java/org/spongepowered/asm/mixin/transformer/MixinTargetContext.java (diff)