This adds a couple of missed tests from existing clang-rename ones and
introduces several new ones (e.g. static class member). This patch is required
to ensure feature parity for migration off Clang-Rename API D71880.
Details
Details
- Reviewers
hokein - Commits
- rGa1152482826c: [clangd] Add missing tests to rename feature
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
clang-tools-extra/clangd/unittests/RenameTests.cpp | ||
---|---|---|
203 | hmm, I'm not sure this is needed, I think it is identical with C. | |
618 | nit: we have groups of templates on ~Line 215 already, I'd suggest to move these template-related cases to there. | |
684 | nit: add a comment, this is variable template. | |
703 | this test case is about function templates, move to Line 683. |
clang-tools-extra/clangd/unittests/RenameTests.cpp | ||
---|---|---|
618 | this looks like still undone. I would move this section (from Line638-Line710) to Line307. |
hmm, I'm not sure this is needed, I think it is identical with C.