Should we have different highlightings for declarations vs usages? (Although I guess in the end it will be up to the editor if they highlight them differently as the scope is just more specific for declarations)
I guess I personally don't really see the reason as it should be clear from the context if it's a declaration or a function call.
But this might actually be nice if you have macros that declare functions

Fri, Aug 30, 12:29 AM · Restricted Project

jvikstrom accepted D66949: [clangd] Add .vscode-test to .gitignore..

Fri, Aug 30, 12:16 AM · Restricted Project, Restricted Project

Thu, Aug 29

jvikstrom added inline comments to D66928: [clangd] Collecting main file macro expansion locations in ParsedAST..

Thu, Aug 29, 10:07 AM · Restricted Project, Restricted Project

jvikstrom updated the diff for D66928: [clangd] Collecting main file macro expansion locations in ParsedAST..

Added test for prepending concatenations. Also added made tests pass.

Thu, Aug 29, 9:57 AM · Restricted Project, Restricted Project

jvikstrom added inline comments to D66928: [clangd] Collecting main file macro expansion locations in ParsedAST..

Thu, Aug 29, 7:27 AM · Restricted Project, Restricted Project

jvikstrom added inline comments to D66928: [clangd] Collecting main file macro expansion locations in ParsedAST..

Thu, Aug 29, 3:13 AM · Restricted Project, Restricted Project

jvikstrom updated the diff for D66928: [clangd] Collecting main file macro expansion locations in ParsedAST..

Clarified comments. Added tests. Not getting expansions inside other macro expansions.

Thu, Aug 29, 3:10 AM · Restricted Project, Restricted Project

jvikstrom committed rG4683eec2d303: [clangd] Update themeRuleMatcher when color theme changes in vscode extension. (authored by jvikstrom).

[clangd] Update themeRuleMatcher when color theme changes in vscode extension.

Thu, Aug 29, 1:41 AM

jvikstrom committed rL370305: [clangd] Update themeRuleMatcher when color theme changes in vscode extension..

[clangd] Update themeRuleMatcher when color theme changes in vscode extension.

Thu, Aug 29, 1:41 AM

jvikstrom closed D66406: [clangd] Update theme when color theme changes in vscode extension..

Thu, Aug 29, 1:40 AM · Restricted Project, Restricted Project

jvikstrom created D66928: [clangd] Collecting main file macro expansion locations in ParsedAST..

Thu, Aug 29, 1:02 AM · Restricted Project, Restricted Project

jvikstrom retitled D66406: [clangd] Update theme when color theme changes in vscode extension. from [clangd] Update themeRuleMatcher when color theme changes in vscode extension. to [clangd] Update theme when color theme changes in vscode extension..

Thu, Aug 29, 12:39 AM · Restricted Project, Restricted Project

jvikstrom updated the diff for D66406: [clangd] Update theme when color theme changes in vscode extension..

Use conf.affectsConfiguration('workbench.colorTheme') instead of keeping track of the old color theme manually.

Thu, Aug 29, 12:30 AM · Restricted Project, Restricted Project

Wed, Aug 28

jvikstrom accepted D66881: [clangd][vscode] Don't leak the resources.

Wed, Aug 28, 8:05 AM · Restricted Project, Restricted Project

jvikstrom updated the diff for D66738: [clangd] Added highlighting for structured bindings..

Abandoned trying to highlight the same as the bound decl.

Wed, Aug 28, 7:44 AM · Restricted Project, Restricted Project

jvikstrom added a comment to D66881: [clangd][vscode] Don't leak the resources.

Oh, was not aware we needed to dispose the extension as well.

Wed, Aug 28, 7:03 AM · Restricted Project, Restricted Project

jvikstrom updated the diff for D66406: [clangd] Update theme when color theme changes in vscode extension..

Updated to new master.

Wed, Aug 28, 7:03 AM · Restricted Project, Restricted Project

jvikstrom committed rG2dddf3e4ff81: [clangd] Cleans up the semantic highlighting resources if clangd stops. (authored by jvikstrom).

[clangd] Cleans up the semantic highlighting resources if clangd stops.

Wed, Aug 28, 6:47 AM

jvikstrom committed rL370202: [clangd] Cleans up the semantic highlighting resources if clangd stops..

[clangd] Cleans up the semantic highlighting resources if clangd stops.

Wed, Aug 28, 6:45 AM

jvikstrom closed D66743: [clangd] Cleans up the semantic highlighting resources if clangd crashes..

Wed, Aug 28, 6:45 AM · Restricted Project, Restricted Project

jvikstrom added inline comments to D66516: [clangd] Highlight typedefs to template parameters as template parameters.

Wed, Aug 28, 6:41 AM · Restricted Project, Restricted Project

jvikstrom updated the diff for D66516: [clangd] Highlight typedefs to template parameters as template parameters.

Added a RecursiveASTVisitor for finding 'underlying' types.

Wed, Aug 28, 6:17 AM · Restricted Project, Restricted Project

Tue, Aug 27

jvikstrom updated the diff for D66743: [clangd] Cleans up the semantic highlighting resources if clangd crashes..

Dispose of all resources when disposing.

Tue, Aug 27, 7:24 AM · Restricted Project, Restricted Project

jvikstrom added inline comments to D66743: [clangd] Cleans up the semantic highlighting resources if clangd crashes..

Tue, Aug 27, 4:43 AM · Restricted Project, Restricted Project

jvikstrom updated the diff for D66743: [clangd] Cleans up the semantic highlighting resources if clangd crashes..

Renamed disposables to subscriptions and removed clangd crashes in api.

Tue, Aug 27, 4:43 AM · Restricted Project, Restricted Project

Mon, Aug 26

jvikstrom abandoned D65637: [clangd] [WIP] Semantic highlighting prototype for the vscode extension..

Mon, Aug 26, 9:52 AM · Restricted Project

jvikstrom created D66743: [clangd] Cleans up the semantic highlighting resources if clangd crashes..

Mon, Aug 26, 7:26 AM · Restricted Project, Restricted Project

jvikstrom committed rL369911: [clangd] Handling text editor/document lifetimes in vscode extension..

[clangd] Handling text editor/document lifetimes in vscode extension.

Mon, Aug 26, 7:25 AM

jvikstrom committed rG1c9aa70b042b: [clangd] Handling text editor/document lifetimes in vscode extension. (authored by jvikstrom).

[clangd] Handling text editor/document lifetimes in vscode extension.

Mon, Aug 26, 7:19 AM

jvikstrom closed D66735: [clangd] Handling text editor/document lifetimes in vscode extension..

Mon, Aug 26, 7:19 AM · Restricted Project

jvikstrom updated the diff for D66735: [clangd] Handling text editor/document lifetimes in vscode extension..

Fixed changes that weren't supposed to be made.

Mon, Aug 26, 6:37 AM · Restricted Project

jvikstrom updated the diff for D66735: [clangd] Handling text editor/document lifetimes in vscode extension..

Address comments.

Mon, Aug 26, 6:31 AM · Restricted Project

jvikstrom created D66738: [clangd] Added highlighting for structured bindings..

Mon, Aug 26, 6:22 AM · Restricted Project, Restricted Project

jvikstrom added inline comments to D66735: [clangd] Handling text editor/document lifetimes in vscode extension..

Mon, Aug 26, 6:04 AM · Restricted Project

jvikstrom created D66735: [clangd] Handling text editor/document lifetimes in vscode extension..

Mon, Aug 26, 5:26 AM · Restricted Project

jvikstrom committed rG3b6d7040cf4f: [clangd] Added a colorizer to the vscode extension. (authored by jvikstrom).

[clangd] Added a colorizer to the vscode extension.

Mon, Aug 26, 4:36 AM

jvikstrom committed rL369893: [clangd] Added a colorizer to the vscode extension..

[clangd] Added a colorizer to the vscode extension.

Mon, Aug 26, 4:35 AM

jvikstrom closed D66219: [clangd] Added a colorizer to the vscode extension..

Mon, Aug 26, 4:35 AM · Restricted Project, Restricted Project

jvikstrom added inline comments to D66516: [clangd] Highlight typedefs to template parameters as template parameters.

Mon, Aug 26, 4:20 AM · Restricted Project, Restricted Project

jvikstrom added inline comments to D66219: [clangd] Added a colorizer to the vscode extension..

Mon, Aug 26, 2:51 AM · Restricted Project, Restricted Project

jvikstrom updated the diff for D66219: [clangd] Added a colorizer to the vscode extension..

Made tests more readable.

Mon, Aug 26, 2:51 AM · Restricted Project, Restricted Project

Thu, Aug 22

jvikstrom added inline comments to D66516: [clangd] Highlight typedefs to template parameters as template parameters.

Thu, Aug 22, 9:12 AM · Restricted Project, Restricted Project

jvikstrom updated the diff for D66219: [clangd] Added a colorizer to the vscode extension..

Added missing protected and comment.

Thu, Aug 22, 7:51 AM · Restricted Project, Restricted Project

jvikstrom added a comment to D66219: [clangd] Added a colorizer to the vscode extension..

I had completely forgotten we could just override the applyHighlightings method in the tests, everything is much simpler now.

Thu, Aug 22, 7:51 AM · Restricted Project, Restricted Project

jvikstrom updated the diff for D66219: [clangd] Added a colorizer to the vscode extension..

Rewrote the Highlighter class as we can override the highlighting method for the tests.

Thu, Aug 22, 7:44 AM · Restricted Project, Restricted Project

jvikstrom accepted D66541: [clangd] Send highlighting diff beyond the end of the file..

It feels a bit strange to be sending highlighting (even if they are empty) beyond eof. But I guess the proposal does not specify this and it would make life for the vscode extension (much) simpler.
Theia also does not crash from this and if we apply decorations to vscode outside the file nothing happens as well so I guess it should be fine. (even when vscode finally implements this feature into lsp)

Thu, Aug 22, 1:44 AM · Restricted Project, Restricted Project