Skip to content
This repository has been archived by the owner on Nov 5, 2021. It is now read-only.

csharp: add support for binary literals and _ as separator #16

Merged
merged 1 commit into from
Jun 8, 2017

Conversation

sandyarmstrong
Copy link
Member

Note that proper _ usage is not enforced; this matches vscode behavior.

@sandyarmstrong
Copy link
Member Author

I thought about giving the binary literal its own theme, but that would require changes in vscode and I wasn't sure if it's worth it. Should the theme be extended to support styling hex and binary literals differently?

https://github.com/Microsoft/vscode/blob/1.11.2/src/vs/editor/common/standalone/themes.ts#L25

Note that proper `_` usage is not enforced; this matches vscode behavior.
@sandyarmstrong
Copy link
Member Author

Updated to add some corresponding test cases.

@alexdima alexdima merged commit 9865a5f into microsoft:master Jun 8, 2017
@alexdima
Copy link
Member

alexdima commented Jun 8, 2017

Thank you @sandyarmstrong ! ❤️

@alexdima alexdima modified the milestone: May 2017 Jun 8, 2017
@alexdima alexdima self-assigned this Jun 8, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants