[PATCH] D89882: [clangd] Migrate to proto2 syntax

2020-10-23 Thread Kirill Bobyrev via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG421a2a0dbbd6: [clangd] Migrate to proto2 syntax (authored by kbobyrev). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://r

[PATCH] D89882: [clangd] Migrate to proto2 syntax

2020-10-23 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev updated this revision to Diff 300246. kbobyrev added a comment. Remove unnecessary comment and fix a typo. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89882/new/ https://reviews.llvm.org/D89882 Files: clang-tools-extra/clangd/index/re

[PATCH] D89882: [clangd] Migrate to proto2 syntax

2020-10-23 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/index/remote/marshalling/Marshalling.cpp:118 Req.IDs = std::move(*IDs); - Req.Filter = static_cast(Message->filter()); +

[PATCH] D89882: [clangd] Migrate to proto2 syntax

2020-10-23 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev updated this revision to Diff 300231. kbobyrev added a comment. Remove a newline. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89882/new/ https://reviews.llvm.org/D89882 Files: clang-tools-extra/clangd/index/remote/Index.proto clang-