-
Notifications
You must be signed in to change notification settings - Fork 127
Optional parameters in extension methods not working #692
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
Comments
EotT123
pushed a commit
to EotT123/manifold-test
that referenced
this issue
Apr 19, 2025
EotT123
pushed a commit
to EotT123/manifold-test
that referenced
this issue
Apr 19, 2025
I've fixed the example, I forgot to add a default value for one of the parameters. The following warning is also displayed: |
The generated extension methods don't seem to be anotated with @Extension
public class FooExt {
public static String test(@This Foo foo, String bar, String foobar) {
return bar + foobar;
}
@manifold_params("foo,opt$bar,foobar")
public static String $test(Foo foo, boolean $isBar, String bar, String foobar) {
String var10001 = $isBar ? bar : "";
String var10002 = $isBar ? bar : "";
return test(foo, var10001, foobar);
}
@manifold_params("foo,bar,foobar")
public static String test(Foo foo, String foobar) {
return $test(foo, false, (String)null, foobar);
}
static {
IBootstrap.dasBoot();
}
} |
EotT123
pushed a commit
to EotT123/manifold-test
that referenced
this issue
May 23, 2025
manifold-systems/manifold#692 #692 #692
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
It appears that optional and named parameters in extension methods aren't functioning correctly.
The text was updated successfully, but these errors were encountered: