This is an archive of the discontinued LLVM Phabricator instance.

[clangd] in VSCode client, filter extensions properly and only accept file: URIs
ClosedPublic

Authored by sammccall on Dec 18 2017, 1:29 AM.

Details

Summary

The filtering wasn't previously working as intended - the string list is
interpreted as a list of editor modes, not file extensions.
(It happens to mostly work as "c" and "cpp" are the names of modes, but we're
missing objective-c)

The file: restriction is new - clangd needs to be able to convert URI<->path
in order to determine how to build.

Diff Detail

Repository
rL LLVM

Event Timeline

sammccall created this revision.Dec 18 2017, 1:29 AM
hokein accepted this revision.Dec 18 2017, 1:50 AM

LGTM.

clangd/clients/clangd-vscode/src/extension.ts
25 ↗(On Diff #127308)

nit: maybe use ' to be consistent with other places of this file.

This revision is now accepted and ready to land.Dec 18 2017, 1:50 AM
sammccall updated this revision to Diff 127320.Dec 18 2017, 3:30 AM

Fix quotes

This revision was automatically updated to reflect the committed changes.