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
Debugger in DartEditor does not show enough information for closure activation on the stack #4768
Comments
Added this to the M1 milestone. |
It sounds like there are two changes here. One is just a change from <anonymous closure> ==> {}. The other is to include the names of the enclosing scopes (CodeEmitterTask.{} ==> CodeEmitterTask.assembleProgram.{}). Set owner to @mhausner. |
Removed Priority-Medium label. |
Removed this from the M5 milestone. |
This comment was originally written by @mhausner The VM now provides all the additional info. I'll let Devon close this issue once the editor exposes all of it to the user. Set owner to @devoncarew. |
I don't see VM providing enough info. Test code: import 'dart:async'; main() { fooBar() { class A { Stack trace for breakpoint on print('bbb'), see formatted output as the first attached image. [2191.797 <== { "event": "paused", "params": { "reason": "breakpoint", "isolateId": 360142472,"breakpointId": 1,"location": { "url":"file:///Users/scheglov/dart/test2/bin/test2.dart","libraryId":13,"tokenOffset":59}}}] The same is for A.barBaz(). [2676.875 ==> {"id":11,"command":"getStackTrace","params":{"isolateId":360142472}}] Attachments: Removed the owner. |
This comment was originally written by @mhausner My comment #10 is referring to closure objects, not function/closure names. This is still a valid feature wish. Removed Type-Defect label. |
Editor is not supported any more. |
In the stack view (attached picture) the line:
CodeEmitterTask.<anonymous closure>()
fails to show the closure resides inside the assembleProgram method.
How about displaying:
CodeEmitterTask.assembleProgram.{}()
This will make the context clear.
Regards,
Lars
Attachment:
DebuggerClosure.png (209.43 KB)
The text was updated successfully, but these errors were encountered: