dart-archive / eclipse3

DEPRECATED - NO LONGER maintained/supported - Eclipse plugins and Dart Editor
8 stars 6 forks source link

Code being stepped over is clearly not the code that is in the editor #62

Open DartBot opened 9 years ago

DartBot commented 9 years ago

Issue by lukechurch Originally opened as https://github.com/dart-lang/sdk/issues/23090


Create a file Add breakpoint Run Comment out some lines Step Step

Notice that the editor now thinks it is stepping over comments. Notice also, e.g. how the editor thinks it has executed a line that prints 'done'. However there is no such line in the console.

Dart Editor version 1.10.0.dev_00_01 (DEV) Dart SDK version 1.10.0-dev.0.1

I suspect this won't be enough to repro. Let me know what else you need.


Attachment: [stepping over comments.png](https://storage.googleapis.com/google-code-attachments/dart/issue-23090/comment-0/stepping over comments.png) (393.14 KB)

DartBot commented 9 years ago

Comment by lukechurch


Sorry the above comments didn't make it clear. I restarted the program between making the changes and stepping.

After running a re-analyze sources, it behaves itself again.

DartBot commented 9 years ago

Comment by keertip


Luke,

So is this still an issue? Or does this now work as expected?


cc @lukechurch.

DartBot commented 9 years ago

Comment by lukechurch


No, it does not work as expected, this is still an issue.

I don't expect to have to run 're-analyze sources' between every change in order to be able to use the debugger reliably :-)

DartBot commented 9 years ago

Comment by lukechurch


PS - It is possible that the root problem here is that the debugger doesn't tell the user if the source has been modified between when they started the execution, and so the state in the VM no longer matches.

If that is the root cause here, then it may be a UI problem rather than a debugger problem per-say, but the behaviour is still a high priority bug.

Ultimately when the debugger shows state in the UI, the code that it is being debugged should be the same state that is displayed in the UI. Anything else is really misleading from the users' POV.

DartBot commented 9 years ago

Comment by bwilkerson


Removed Area-Analyzer label. Added Area-Editor label.