Skip to content

Commit

Permalink
Merge pull request #215 from stiemannkj1/ARIES-1219-support-jsr-ret-i…
Browse files Browse the repository at this point in the history
…n-spi-fly

ARIES-1219 Weaving of Aries SPI Fly bundle produces RuntimeException: JSR/RET are not supported with computeFrames option
  • Loading branch information
timothyjward committed Apr 8, 2023
2 parents a816658 + 9df919f commit 55b2b34
Showing 1 changed file with 6 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,14 @@

import org.apache.aries.spifly.Util;
import org.apache.aries.spifly.WeavingData;
import org.objectweb.asm.ClassReader;
import org.objectweb.asm.ClassVisitor;
import org.objectweb.asm.Label;
import org.objectweb.asm.MethodVisitor;
import org.objectweb.asm.Opcodes;
import org.objectweb.asm.Type;
import org.objectweb.asm.commons.GeneratorAdapter;
import org.objectweb.asm.commons.JSRInlinerAdapter;
import org.objectweb.asm.commons.Method;

import aQute.bnd.annotation.baseline.BaselineIgnore;
Expand Down Expand Up @@ -77,7 +79,10 @@ public boolean isWoven() {
public MethodVisitor visitMethod(int access, String name, String desc,
String signature, String[] exceptions) {
MethodVisitor mv = super.visitMethod(access, name, desc, signature, exceptions);
return new TCCLSetterMethodVisitor(mv, access, name, desc);
mv = new TCCLSetterMethodVisitor(mv, access, name, desc);
mv = new JSRInlinerAdapter(mv, access, name, desc, signature, exceptions);

return mv;
}

@Override
Expand Down

1 comment on commit 55b2b34

@vbansal22
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @timothyjward , Is it possible to get a release build (1.3.7) with this PR incorporated in the trunk?

Please sign in to comment.