You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
https://dart-review.googlesource.com/c/sdk/+/409280 wires up the messages for failed breakpoints to the UI for DAP-based clients (like VS Code) so that users can better understand the reason for grey/invalid breakpoints (for example whether they're in invalid locations, or the script just hasn't been compiled yet).
The current messages thar we get back (as RPCErrors) are very wordy and contain redundant information (in this context):
addBreakpointWithScriptUri: Cannot add breakpoint at line 8. Error occurred when resolving breakpoint location: No debuggable code where breakpoint was requested.
This doesn't fit nicely into a tooltip, and:
"addBreakpointWithScriptUri" is an internal method name that is irrelevant to the user
"Cannot add breakpoint at line 8" doesn't add anything that isn't already obvious in the context of a grey breakpoint on line 8
"Error occurred when resolving breakpoint location" also provides no useful information
"No debuggable code where breakpoint was requested" is the only text we really want to show the user in a tooltip on the grey breakpoint. We're currently using a regex to extract this, but if the overall message changes (or if there are other messages that the regex doesn't match) then the errors might not be ideal.
It would be nice if we could get a terse string for just "No debuggable code where breakpoint was requested" (and any other errors likely to be returned from addBreakpointWithScriptUri) that we could use directly instead.
The text was updated successfully, but these errors were encountered:
https://dart-review.googlesource.com/c/sdk/+/409280 wires up the messages for failed breakpoints to the UI for DAP-based clients (like VS Code) so that users can better understand the reason for grey/invalid breakpoints (for example whether they're in invalid locations, or the script just hasn't been compiled yet).
The current messages thar we get back (as
RPCError
s) are very wordy and contain redundant information (in this context):This doesn't fit nicely into a tooltip, and:
"No debuggable code where breakpoint was requested" is the only text we really want to show the user in a tooltip on the grey breakpoint. We're currently using a regex to extract this, but if the overall message changes (or if there are other messages that the regex doesn't match) then the errors might not be ideal.
It would be nice if we could get a terse string for just "No debuggable code where breakpoint was requested" (and any other errors likely to be returned from
addBreakpointWithScriptUri
) that we could use directly instead.The text was updated successfully, but these errors were encountered: