[all-commits] [llvm/llvm-project] ce6338: [clangd] Drop version from remote index proto name...

Sam McCall via All-commits all-commits at lists.llvm.org
Fri Oct 23 06:28:32 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ce63383e45f4c833dbff6a89b242bfd1d188786e
      https://github.com/llvm/llvm-project/commit/ce63383e45f4c833dbff6a89b242bfd1d188786e
  Author: Sam McCall <sam.mccall at gmail.com>
  Date:   2020-10-23 (Fri, 23 Oct 2020)

  Changed paths:
    M clang-tools-extra/clangd/index/remote/Index.proto
    M clang-tools-extra/clangd/index/remote/marshalling/Marshalling.cpp
    M clang-tools-extra/clangd/index/remote/marshalling/Marshalling.h
    M clang-tools-extra/clangd/index/remote/server/Server.cpp
    M clang-tools-extra/clangd/unittests/remote/MarshallingTests.cpp

  Log Message:
  -----------
  [clangd] Drop version from remote index proto names, fix clangd-index-server

We only need to version these messages if they actually diverge.
Unlike the service, the namespace name isn't part of the wire format.

clangd-index-server was broken by 81e5f298c431555d809f898c196945ca879c1150
as the namespace names weren't updated there, this fixes it (by adding
them for the service, and not requiring them elsewhere).




More information about the All-commits mailing list