Skip to content

Commit

Permalink
fix: update ASMTransformerWrapper visitor class version
Browse files Browse the repository at this point in the history
  • Loading branch information
kappa-maintainer committed Mar 1, 2024
1 parent daa0154 commit 2d935d4
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ private static byte[] makeWrapper(String fileName)

ClassWriter writer = new ClassWriter(ClassWriter.COMPUTE_FRAMES);

writer.visit(Opcodes.V17, Opcodes.ACC_PUBLIC, name, null, wrapper.getInternalName(), null);
writer.visit(Opcodes.V21, Opcodes.ACC_PUBLIC, name, null, wrapper.getInternalName(), null);

This comment has been minimized.

Copy link
@Ecdcaeb

Ecdcaeb Mar 1, 2024

Contributor

I dont think this version should be updated.
I think remain it Java 1.7 is OK (not 17 or 21)
since the content is just a 1.7 class.

This comment has been minimized.

Copy link
@kappa-maintainer

kappa-maintainer Mar 1, 2024

Author Collaborator

Nah, whole Forge is 21 now, and we will get transformers with new structure sooner or later


Method m = Method.getMethod("void <init> ()");
GeneratorAdapter mg = new GeneratorAdapter(Opcodes.ACC_PUBLIC, m, null, null, writer);
Expand Down

0 comments on commit 2d935d4

Please sign in to comment.