Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update C/C++ grammars to files used by VSCode #6937

Closed
wants to merge 42 commits into from

Conversation

alexl0gan
Copy link
Contributor

What it does

Update cpp package to use vscode grammar files as defined here:
https://github.com/jeff-hykin/cpp-textmate-grammar
https://github.com/microsoft/vscode/tree/master/extensions/cpp/syntaxes

How to test

Open a C/C++ file and enjoy a richer syntax colourisation.

Review checklist

Reminder for reviewers

westbury and others added 30 commits October 3, 2019 14:14
This reduces excessive CPU usage on Windows and reducing latency for all

Signed-off-by: Nigel Westbury <[email protected]>
- On Windows, `FileSearchService`.find now returns valid file URIs
as it already did on linux.
- Use LabelProvider to create tooltip field for quick open entries.
- Allow Windows users to search for file paths using either '/' or '\\'.
Signed-off-by: Bogdan Stolojan <[email protected]>

Renaming property

Signed-off-by: Bogdan Stolojan <[email protected]>
Calling `contributionProvider.getContributions` in the constructor
was throwing errors because some of the contributions inject
the LabelProvider.
This allows toolbar items to show content specific to the current
widget. For example, a toolbar item for the terminal could show
a message specific to the currently displayed terminal tab.

Signed-off-by: Matthew Gordon <[email protected]>
The previous implementation did not work because the window was unloaded
before the dialog could be opened.

This implementation gets around some… unusual behaviour of Electron's
beforeunload event by using the will-prevent-unload WebContents event.
If a second instance is opened, it closes immediately and the original
instance receives focus.

Signed-off-by: Matthew Gordon <[email protected]>
Signed-off-by: Nigel Westbury <[email protected]>

Clean up css class names and other code review changes
The version of TypeScript and @types/react used in mbs-ide are not
compatible with the `event` type replaced by this commit.

This commit can be reverted from ARMmbed/theia when:
- Theia updates TS or @types/react such that the original code compiles
- mbs-ide no longer uses @theia/scm in its workspace.
Shut down Mercurial command servers after 10 seconds of inactivity

Clean out unneeded code

Added test for HgCommandServer utils.
Minor touches suggested by Kanthan
Signed-off-by: Federico Bozzini <[email protected]>
MessageService is not available outside of a connection container context,
so using the default init would fail.
For Hg now WorkingDirectoryStatus.currentHead is Branch.tip.sha
Amend support does not work in many edge-cases, this disables amend for Hg leaving the code in-place inside HgScmProvider to be re-enabled when everything will be fixed.
@vince-fugnitto
Copy link
Member

I think this might have been mistakenly committed 🙈

@arekzaluski
Copy link
Contributor

@vince-fugnitto Yes, that was a mistake. It will be rebased on top of master Theia and then reopened :)

@vince-fugnitto
Copy link
Member

@vince-fugnitto Yes, that was a mistake. It will be rebased on top of master Theia and then reopened :)

Please note that @theia/textmate-grammars will be removed #6933 and that cpp support is moved to https://github.com/eclipse-theia/theia-cpp-extensions in case your pull-request addresses either of those.

@alexl0gan alexl0gan deleted the cpp-grammars branch January 28, 2020 11:31
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.

7 participants