Skip to content
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

Jump to bottom

Closures displayed in stack trace with extra argument #7864

Closed
gbracha opened this issue Jan 11, 2013 · 7 comments
Closed

Closures displayed in stack trace with extra argument #7864

gbracha opened this issue Jan 11, 2013 · 7 comments
Labels
area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends.

Comments

@gbracha
Copy link
Contributor

gbracha commented Jan 11, 2013

The stack trace displays closures with an extra argument. Presumably this is an artifact of the underlying VM implementation, but this is not what a Dart programmer would expect. E.g.,

(String s) => 'string $t'

displays as

(dynamic, String) => dynamic

rather than as

(String) => dynamic
@clayberg
Copy link

Set owner to @devoncarew.
Added this to the M4 milestone.

@iposva-google
Copy link
Contributor

Let us fix this for you.

Removed the owner.
Removed this from the M4 milestone.
Added this to the M3 milestone.
Removed Area-Editor label.
Added Area-VM label.

@iposva-google
Copy link
Contributor

Removed this from the M3 milestone.
Added this to the M4 milestone.

@larsbak
Copy link

larsbak commented May 28, 2013

Removed this from the M4 milestone.
Added this to the M5 milestone.

@iposva-google
Copy link
Contributor

Removed Priority-Medium label.
Added Priority-Unassigned label.

@iposva-google
Copy link
Contributor

Removed this from the M5 milestone.

@gbracha gbracha added Type-Defect area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends. labels Jun 5, 2013
@rmacnak-google
Copy link
Contributor

class A {
foo() {
closure(String x) => "$x i";
print(closure.runtimeType);
}
}

main() {
new A().foo();
}

prints (String) => dynamic
not (dynamic, String) => dynamic

This has probably been fixed for a long time.

@kevmoo kevmoo removed the triaged label Mar 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@kevmoo @gbracha @iposva-google @larsbak @rmacnak-google @clayberg