Skip to content

Conversation

@DustinCampbell
Copy link
Member

@gregg-miskelly, @caslan: If you send me information for the debugger, I'll incorporate it into this PR.

@DustinCampbell DustinCampbell added this to the 1.3 milestone Jul 20, 2016
@DustinCampbell
Copy link
Member Author

Addresses #578

@gregg-miskelly
Copy link
Contributor

@DustinCampbell I was just comping you an email about this. Great minds think alike :). I will send you some text.

@DustinCampbell
Copy link
Member Author

Also addresses #553

@DustinCampbell
Copy link
Member Author

OK, I added the debugger what's new items. I reworded some of the text a bit, so please take a look.

@DustinCampbell
Copy link
Member Author

Also, I arbitrarily ordered them to highlight some of the more important items. Let me know if you disagree with the order and I'll change it.

* Support for conditional breakpoints
* New configuration option to enable [stepping into properties and operators](https://github.com/OmniSharp/omnisharp-vscode/blob/release/debugger.md#stepping-into-properties-and-operators).
* Duplicate warnings and errors should no longer accumulate in Unity projects [#447](https://github.com/OmniSharp/omnisharp-vscode/issues/447)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two other debugger items that I realized I forgot --

  • Support for setting the value of variables in the debugger. To do this, right click on the variable and invoke 'Set Value'.
  • Display of type names for variables changed in the debugger to support VS Code's variable assignments. A shortened form of the type names is now displayed as part of the variable name, and the full type name can be viewed by hovering over the name.

@gregg-miskelly
Copy link
Contributor

Otherwise LGTM

@rajkumar42
Copy link
Contributor

👍

@DustinCampbell
Copy link
Member Author

@gregg-miskelly: Does this look right? These two items appeared to me to be related, so I collapsed them. Was that correct?

@DustinCampbell
Copy link
Member Author

ping?

@gregg-miskelly
Copy link
Contributor

LGTM

@DustinCampbell DustinCampbell merged commit d3764fe into dotnet:master Jul 20, 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.

4 participants