Skip to content

Always allow breakpoints to be set on same line if possible#1918

Merged
dsyme merged 1 commit intodotnet:masterfrom
vasily-kirichenko:fix-breakpoint-validation
Dec 3, 2016
Merged

Always allow breakpoints to be set on same line if possible#1918
dsyme merged 1 commit intodotnet:masterfrom
vasily-kirichenko:fix-breakpoint-validation

Conversation

@vasily-kirichenko
Copy link
Contributor

@vasily-kirichenko vasily-kirichenko commented Dec 2, 2016

This fixes #1902

However, I just removed a peace of logic that didn't allow set the breakpoint. I'm not sure it's opened the door of setting breakpoints at inappropriate places.

@cartermp
Copy link
Contributor

cartermp commented Dec 2, 2016

Great work @vasily-kirichenko . Below are all examples where it would previously fail, but now it works:

debugging-works

@dsyme
Copy link
Contributor

dsyme commented Dec 3, 2016

This is a good fix, thanks.

@dsyme dsyme merged commit 90aac4d into dotnet:master Dec 3, 2016
@cartermp
Copy link
Contributor

cartermp commented Dec 3, 2016

🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inconsistent behavior or two ways of adding breakpoints: with mouse and with F9

4 participants