Age | Commit message (Collapse) | Author |
|
CHANGELOG:
- activate the extension on ObjC files.
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@353309 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
The version bumps are a result of running `npm audit`, which found
3 security issues in previous versions of our dependencies.
Also add 'package-lock.json' to the repo, it's a common practice to
check in those files into the repository to get consistent versions of
dependencies when running on different machines.
Reviewers: hokein
Reviewed By: hokein
Subscribers: dschuff, ioeric, MaskRay, jkorous, arphaman, kadircet, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D57814
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@353296 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary: Thanks to Andreas Ostermeyer for raising this on the mailing list.
Reviewers: hokein
Reviewed By: hokein
Subscribers: ioeric, MaskRay, jkorous, arphaman, kadircet, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D57813
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@353295 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
CHANGELOG:
- cleanup filestatus caches when clangd crashes
- extension workwith LSP v3.14.0, support go-to-declaration feature
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@353027 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
This allows us to use latest LSP v3.14.0 (for go-to-declaration feature).
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@353026 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
Clear the cached file statuses, otherwise we will leave some garbage texts on
the status bar when clangd crashes.
Reviewers: ilya-biryukov
Subscribers: ioeric, MaskRay, jkorous, arphaman, kadircet, cfe-commits
Differential Revision: https://reviews.llvm.org/D56540
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@352049 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
In last year's update (D48219) it was suggested that the release manager
might want to do this, so here we go.
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@351194 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@350324 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
The file status will be shown in the status bar.
Depends on D55363.
Reviewers: ilya-biryukov
Subscribers: ioeric, MaskRay, jkorous, arphaman, kadircet, cfe-commits
Differential Revision: https://reviews.llvm.org/D55374
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@350210 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@347969 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
When using the vscode clangd plugin, lots and lots of junk output is printed to the output window, which constantly reopens itself.
Example output:
I[11:13:17.733] <-- textDocument/codeAction(4)
I[11:13:17.733] --> reply:textDocument/codeAction(4) 0 ms
I[11:13:17.937] <-- textDocument/codeAction(5)
I[11:13:17.937] --> reply:textDocument/codeAction(5) 0 ms
I[11:13:18.557] <-- textDocument/hover(6)
I[11:13:18.606] --> reply:textDocument/hover(6) 48 ms
This should prevent that from happening.
Patch by James Findley!
Reviewers: ioeric, ilya-biryukov, hokein
Reviewed By: ioeric
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, cfe-commits
Tags: #clang-tools-extra
Differential Revision: https://reviews.llvm.org/D55052
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@347968 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@347737 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
Alt+o is used on Windows/Linux and Option+Cmd+o on macOS.
Signed-off-by: Marc-Andre Laperle <malaperle@gmail.com>
Reviewers: hokein, ilya-biryukov, ioeric
Reviewed By: ioeric
Subscribers: sammccall, ilya-biryukov, ioeric, MaskRay, jkorous, arphaman, kadircet, cfe-commits
Differential Revision: https://reviews.llvm.org/D54781
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@347509 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Reviewers: ioeric, sammccall, simark
Reviewed By: sammccall
Subscribers: MaskRay, jkorous, arphaman, kadircet, cfe-commits
Differential Revision: https://reviews.llvm.org/D52311
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@347498 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
When opening package.json, vscode shows:
Use 'Programming Languages' instead
Replacing "Languages" with this fixes it.
Reviewers: ilya-biryukov
Subscribers: arphaman, ilya-biryukov, ioeric, MaskRay, jkorous, cfe-commits
Differential Revision: https://reviews.llvm.org/D49253
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@337697 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@329583 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
This allows the extension to work with LSP 3.0 and is useful for testing.
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewers: ilya-biryukov
Subscribers: hokein, klimek, ilya-biryukov, jkorous-apple, ioeric, MaskRay, cfe-commits
Differential Revision: https://reviews.llvm.org/D45285
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@329574 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@327130 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@327019 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
For features like go-to-definition, clangd can point clients to symlink paths
(e.g. in bazel execroot) which might not be desired if the symlink points to a
file in the workspace. Clangd might not be able to build the file, and users
might prefer to edit the file on the real path.
This change converts file paths from clangd to real path (e.g. resolving symlinks).
Long term, we might want to the symlink handling logic to clangd where clangd
can better decide whether symlinks should be resolved according to e.g. compile
commands.
Reviewers: sammccall
Subscribers: klimek, ilya-biryukov, jkorous-apple, cfe-commits
Differential Revision: https://reviews.llvm.org/D44158
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@327009 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@326548 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@326547 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@326452 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@325503 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
extension.
Reviewers: sammccall
Subscribers: klimek, ilya-biryukov, jkorous-apple, ioeric, cfe-commits
Differential Revision: https://reviews.llvm.org/D43455
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@325499 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Reviewers: sammccall
Subscribers: klimek, ilya-biryukov, jkorous-apple, cfe-commits, ioeric
Differential Revision: https://reviews.llvm.org/D43452
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@325484 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
Setting the CLANGD_TRACE environment variable directly is awkward with VSCode's
"reload from the command palette" workflow.
Reviewers: ilya-biryukov
Subscribers: klimek, jkorous-apple, ioeric, cfe-commits
Differential Revision: https://reviews.llvm.org/D43385
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@325477 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
o Replace the existing clangd::URI with a wrapper of FileURI which also
carries a resolved file path.
o s/FileURI/URI/
o Get rid of the URI hack in vscode extension.
Reviewers: sammccall
Reviewed By: sammccall
Subscribers: klimek, ilya-biryukov, jkorous-apple, cfe-commits
Differential Revision: https://reviews.llvm.org/D42419
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@323660 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
extension.
Summary:
Previously, we use a separate GitHub repository (https://github.com/llvm-vs-code-extensions/vscode-clangd)
for publishing `vscode-clangd` extension to marketplace.
To reduce the maintain burden, we will use the vscode extension in the
clang-tools-extra, and deprecate the one on GitHub.
Test in https://marketplace.visualstudio.com/items?itemName=llvm-vs-code-extensions.clangd-vscode-test
Reviewers: sammccall, krasimir
Reviewed By: sammccall
Subscribers: klimek, ilya-biryukov, cfe-commits
Differential Revision: https://reviews.llvm.org/D41391
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@321252 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
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.
Reviewers: hokein
Subscribers: klimek, ilya-biryukov, cfe-commits
Differential Revision: https://reviews.llvm.org/D41343
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@320972 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
When the user selects a fix-it (or any code action with commands), it is
possible to let the client forward the selected command to the server.
When the clangd.applyFix command is handled on the server, it can send a
workspace/applyEdit request to the client. This has the advantage that
the client doesn't explicitly have to know how to handle
clangd.applyFix. Therefore, the code to handle clangd.applyFix in the VS
Code extension (and any other Clangd client) is not required anymore.
Reviewers: ilya-biryukov, sammccall, Nebiroth, hokein
Reviewed By: hokein
Subscribers: ioeric, hokein, rwols, puremourning, bkramer, ilya-biryukov
Tags: #clang-tools-extra
Differential Revision: https://reviews.llvm.org/D39276
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@317322 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
The client can send notifications when it detects watched files have
changed. This patch adds the protocol handling for this type of notification.
For now, the notification will be passed down to the ClangdServer, but it will
not be acted upon. However, this will become useful for the indexer to react
to file changes.
The events could also potentially be used to invalidate other caches
(compilation database, etc).
This change also updates the VSCode extension so that it sends the events.
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewers: ilya-biryukov, Nebiroth
Subscribers: ilya-biryukov
Tags: #clang-tools-extra
Differential Revision: https://reviews.llvm.org/D38422
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@314693 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Enhances CompletionItemsCollector in such a way that snippet
completions can be presented to the client. Enable snippet completion
items by specifying -enable-snippets while invoking the clangd
executable.
See: https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md#completion-request
See: https://github.com/Microsoft/vscode/blob/master/src/vs/editor/contrib/snippet/browser/snippet.md
Patch by Raoul Wols.
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@313029 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
The URI conversion logic was returning 'undefined' when going from server to
VSCode which broke the Go to Definition functionality.
Reviewers: krasimir
Reviewed By: krasimir
Subscribers: cfe-commits
Tags: #clang-tools-extra
Differential Revision: https://reviews.llvm.org/D35215
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@307715 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Patch contributed by stanionascu!
rfc8089#appendix-E.2 specifies that paths can begin with a drive letter e.g. as file:///c:/.
In this case just consuming front file:// is not enough and the 3rd slash must be consumed to produce a valid path on windows.
The patch introduce a generic way of converting an uri to a filesystem path and back.
Differential Revision: https://reviews.llvm.org/D31401
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@299758 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary: Adds vscode workspace level configuration options for path to clangd binary and its arguments.
Contributed by stanionascu!
Reviewers: cfe-commits, bkramer, krasimir
Reviewed By: krasimir
Differential Revision: https://reviews.llvm.org/D31121
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@298696 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
This uses CodeActions to show 'apply fix' actions when code actions are
requested for a location. The actions themselves make use of a
clangd.applyFix command which has to be implemented on the editor side. I
included an implementation for vscode.
This also adds a -run-synchronously flag which runs everything on the main
thread. This is useful for testing.
Reviewers: krasimir
Subscribers: cfe-commits
Differential Revision: https://reviews.llvm.org/D30498
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@296636 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary: This patch adds bare-bone VS Code integration for development purposes of clangd.
Reviewers: klimek, bkramer, mprobst
Reviewed By: bkramer
Subscribers: mprobst
Differential Revision: https://reviews.llvm.org/D30102
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@296618 91177308-0d34-0410-b5e6-96231b3b80d8
|