[all-commits] [llvm/llvm-project] fb5588: [clangd] Propagate remote index errors via Expected

Kirill Bobyrev via All-commits all-commits at lists.llvm.org
Fri Jul 31 02:49:06 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: fb5588b0ad59522031d037b0d1a3fdcf8ada8a79
      https://github.com/llvm/llvm-project/commit/fb5588b0ad59522031d037b0d1a3fdcf8ada8a79
  Author: Kirill Bobyrev <kbobyrev at google.com>
  Date:   2020-07-31 (Fri, 31 Jul 2020)

  Changed paths:
    M clang-tools-extra/clangd/index/remote/Client.cpp
    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] Propagate remote index errors via Expected

This is a refactoring: errors should be logged only on the highest level.
Switch from Optional to Expected in the serialization code.

Reviewed By: kadircet

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




More information about the All-commits mailing list