[all-commits] [llvm/llvm-project] e864f9: [clangd] Replace raw lexer code with token buffer ...

Haojian Wu via All-commits all-commits at lists.llvm.org
Wed Jan 29 03:57:34 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e864f937669c996b4dc15db7d0ebe4073527c165
      https://github.com/llvm/llvm-project/commit/e864f937669c996b4dc15db7d0ebe4073527c165
  Author: Haojian Wu <hokein.wu at gmail.com>
  Date:   2020-01-29 (Wed, 29 Jan 2020)

  Changed paths:
    M clang-tools-extra/clangd/ClangdServer.cpp
    M clang-tools-extra/clangd/test/rename.test

  Log Message:
  -----------
  [clangd] Replace raw lexer code with token buffer in prepare rename.

Summary:
there is a slight behavior change in this patch:

- before: `in^t a;`, returns our internal error message (no symbol at given location)
- after: `in^t a, returns null, and client displays their message (e.g.
   e.g. "the element can't be renamed" in vscode).

both are sensible according LSP, and we'd save one `rename` call in the later case.

Reviewers: sammccall

Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, usaxena95, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D73610




More information about the All-commits mailing list