-
-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Lambdas do not get remapped #4
Comments
Presumably this issue is meant for Lorenz? (don't close, I can move it) Though could you supply some example mappings that are causing issues? |
Ah, sorry for putting this on the wrong project. Specifically, I was remapping Minecraft Forge universal (everything under the net.minecraftforge package) from SRG to obfuscated. I don't remember exactly where the error cropped up, though -- I forgot to create this issue for a few weeks. I could create a smaller test case to demonstrate it when I have more time in a week or so. |
Did you get chance to follow this up at all? |
public interface TestLambda {
static final TestLambda TEST = () -> System.out.println("test");
void hello();
public static void main(String... args) {
TEST.hello();
}
}
Simple enough test case. It is borked. This is my fix in SS for it, gunna hack it into my fork for the time being. LexManos/SpecialSource@ef5ac81 |
When remapping a lambda, everything appears to get remapped properly except the name of the method to implement. (The
invokedName
parameter of LambdaMetafactory.metafactory.) This results in an AbstractMethodError when the lambda is called.The text was updated successfully, but these errors were encountered: