This is an archive of the discontinued LLVM Phabricator instance.

[clangd][vscode] Turn on the semantic highlighting by default.
ClosedPublic

Authored by hokein on Sep 26 2019, 2:58 AM.

Reviewers

Commits

Summary

We have turned on the flag manually and used it for a while, and don't see any
major issues, let's enable it by default.

Repository

rG LLVM Github Monorepo

Build Status

Buildable 38583
Build 38582: arc lint + arc unit

LGTM

This revision is now accepted and ready to land.Sep 26 2019, 3:59 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

clang-tools-extra/

clangd/

clients/

clangd-vscode/

2 lines

Show First 20 Lines • Show All 86 Lines • ▼ Show 20 Lines	"contributes": {
"description": "Whether or not to send file events to clangd (File created, changed or deleted). This can be disabled for performance consideration."		"description": "Whether or not to send file events to clangd (File created, changed or deleted). This can be disabled for performance consideration."
},		},
"clangd.trace": {		"clangd.trace": {
"type": "string",		"type": "string",
"description": "Names a file that clangd should log a performance trace to, in chrome trace-viewer JSON format."		"description": "Names a file that clangd should log a performance trace to, in chrome trace-viewer JSON format."
},		},
"clangd.semanticHighlighting": {		"clangd.semanticHighlighting": {
"type": "boolean",		"type": "boolean",
"default": "false",		"default": "true",
"description": "Enable semantic highlighting in clangd"		"description": "Enable semantic highlighting in clangd"
}		}
}		}
},		},
"commands": [		"commands": [
{		{
"command": "clangd-vscode.switchheadersource",		"command": "clangd-vscode.switchheadersource",
"title": "Switch between Source/Header"		"title": "Switch between Source/Header"
Show All 16 Lines