[all-commits] [llvm/llvm-project] 8ec449: [clangd] Do not end inactiveRegions range at posit...

Nathan Ridge via All-commits all-commits at lists.llvm.org
Mon Jun 5 00:51:36 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8ec44987e54b5366a03716237feb79e37ae0634a
      https://github.com/llvm/llvm-project/commit/8ec44987e54b5366a03716237feb79e37ae0634a
  Author: Nathan Ridge <zeratul976 at hotmail.com>
  Date:   2023-06-05 (Mon, 05 Jun 2023)

  Changed paths:
    M clang-tools-extra/clangd/ClangdServer.cpp
    M clang-tools-extra/clangd/SemanticHighlighting.cpp
    M clang-tools-extra/clangd/SemanticHighlighting.h
    M clang-tools-extra/clangd/unittests/ClangdTests.cpp
    M clang-tools-extra/clangd/unittests/SemanticHighlightingTests.cpp

  Log Message:
  -----------
  [clangd] Do not end inactiveRegions range at position 0 of line

This carries over the fix previously made for semantic highlighting
https://reviews.llvm.org/D92148, to the new inactiveRegions
protocol as well.

In addition, the directives at the beginning and end of an
inactive region are now excluded from the region.

Fixes https://github.com/clangd/clangd/issues/1631
Fixes https://github.com/clangd/clangd/issues/773

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




More information about the All-commits mailing list