diff --git a/clang-tools-extra/clangd/index/remote/Client.cpp b/clang-tools-extra/clangd/index/remote/Client.cpp --- a/clang-tools-extra/clangd/index/remote/Client.cpp +++ b/clang-tools-extra/clangd/index/remote/Client.cpp @@ -52,7 +52,7 @@ unsigned FailedToParse = 0; while (Reader->Read(&Reply)) { if (!Reply.has_stream_result()) { - FinalResult = Reply.final_result(); + FinalResult = Reply.final_result().has_more(); continue; } auto Response = ProtobufMarshaller->fromProtobuf(Reply.stream_result()); diff --git a/clang-tools-extra/clangd/index/remote/Index.proto b/clang-tools-extra/clangd/index/remote/Index.proto --- a/clang-tools-extra/clangd/index/remote/Index.proto +++ b/clang-tools-extra/clangd/index/remote/Index.proto @@ -10,6 +10,9 @@ package clang.clangd.remote; +// Common final result for streaming requests. +message FinalResult { optional bool has_more = 1; } + message LookupRequest { repeated string ids = 1; } // The response is a stream of symbol messages and the terminating message @@ -17,7 +20,7 @@ message LookupReply { oneof kind { Symbol stream_result = 1; - bool final_result = 2; + FinalResult final_result = 2; } } @@ -36,7 +39,7 @@ message FuzzyFindReply { oneof kind { Symbol stream_result = 1; - bool final_result = 2; // HasMore + FinalResult final_result = 2; } } @@ -51,7 +54,7 @@ message RefsReply { oneof kind { Ref stream_result = 1; - bool final_result = 2; // HasMore + FinalResult final_result = 2; } } @@ -116,7 +119,7 @@ message RelationsReply { oneof kind { Relation stream_result = 1; - bool final_result = 2; // HasMore + FinalResult final_result = 2; } } diff --git a/clang-tools-extra/clangd/index/remote/server/Server.cpp b/clang-tools-extra/clangd/index/remote/server/Server.cpp --- a/clang-tools-extra/clangd/index/remote/server/Server.cpp +++ b/clang-tools-extra/clangd/index/remote/server/Server.cpp @@ -109,7 +109,7 @@ ++Sent; }); LookupReply LastMessage; - LastMessage.set_final_result(true); + LastMessage.mutable_final_result()->set_has_more(true); Reply->Write(LastMessage); SPAN_ATTACH(Tracer, "Sent", Sent); SPAN_ATTACH(Tracer, "Failed to send", FailedToSend); @@ -142,7 +142,7 @@ ++Sent; }); FuzzyFindReply LastMessage; - LastMessage.set_final_result(HasMore); + LastMessage.mutable_final_result()->set_has_more(HasMore); Reply->Write(LastMessage); SPAN_ATTACH(Tracer, "Sent", Sent); SPAN_ATTACH(Tracer, "Failed to send", FailedToSend); @@ -173,7 +173,7 @@ ++Sent; }); RefsReply LastMessage; - LastMessage.set_final_result(HasMore); + LastMessage.mutable_final_result()->set_has_more(HasMore); Reply->Write(LastMessage); SPAN_ATTACH(Tracer, "Sent", Sent); SPAN_ATTACH(Tracer, "Failed to send", FailedToSend); @@ -207,7 +207,7 @@ ++Sent; }); RelationsReply LastMessage; - LastMessage.set_final_result(true); + LastMessage.mutable_final_result()->set_has_more(true); Reply->Write(LastMessage); SPAN_ATTACH(Tracer, "Sent", Sent); SPAN_ATTACH(Tracer, "Failed to send", FailedToSend);