diff --git a/test/cpp/end2end/async_end2end_test.cc b/test/cpp/end2end/async_end2end_test.cc index 96068bfa1f7..eae0ad0d540 100644 --- a/test/cpp/end2end/async_end2end_test.cc +++ b/test/cpp/end2end/async_end2end_test.cc @@ -208,7 +208,7 @@ bool plugin_has_sync_methods(std::unique_ptr& plugin) { // that needs to be tested here. class ServerBuilderSyncPluginDisabler : public ::grpc::ServerBuilderOption { public: - void UpdateArguments(ChannelArguments* arg) override {} + void UpdateArguments(ChannelArguments* /*arg*/) override {} void UpdatePlugins( std::vector>* plugins) override { @@ -1821,7 +1821,7 @@ TEST_P(AsyncEnd2endServerTryCancelTest, ServerBidiStreamingTryCancelAfter) { TestBidiStreamingServerCancel(CANCEL_AFTER_PROCESSING); } -std::vector CreateTestScenarios(bool test_secure, +std::vector CreateTestScenarios(bool /*test_secure*/, bool test_message_size_limit) { std::vector scenarios; std::vector credentials_types; diff --git a/test/cpp/end2end/client_callback_end2end_test.cc b/test/cpp/end2end/client_callback_end2end_test.cc index 88a9dd52f62..2567af90f0d 100644 --- a/test/cpp/end2end/client_callback_end2end_test.cc +++ b/test/cpp/end2end/client_callback_end2end_test.cc @@ -326,8 +326,8 @@ class ClientCallbackEnd2endTest }; activate_(); } - void OnWriteDone(bool ok) override { StartWritesDone(); } - void OnReadDone(bool ok) override { + void OnWriteDone(bool /*ok*/) override { StartWritesDone(); } + void OnReadDone(bool /*ok*/) override { EchoResponse response; EXPECT_TRUE(ParseFromByteBuffer(&recv_buf_, &response)); EXPECT_EQ(request_.message(), response.message()); diff --git a/test/cpp/end2end/client_crash_test_server.cc b/test/cpp/end2end/client_crash_test_server.cc index d92f9c5cff3..2fc44dc0ee6 100644 --- a/test/cpp/end2end/client_crash_test_server.cc +++ b/test/cpp/end2end/client_crash_test_server.cc @@ -46,7 +46,7 @@ namespace testing { class ServiceImpl final : public ::grpc::testing::EchoTestService::Service { Status BidiStream( - ServerContext* context, + ServerContext* /*context*/, ServerReaderWriter* stream) override { EchoRequest request; EchoResponse response; diff --git a/test/cpp/end2end/client_interceptors_end2end_test.cc b/test/cpp/end2end/client_interceptors_end2end_test.cc index cf151d7626e..7ee95a60c1f 100644 --- a/test/cpp/end2end/client_interceptors_end2end_test.cc +++ b/test/cpp/end2end/client_interceptors_end2end_test.cc @@ -507,7 +507,7 @@ class BidiStreamingRpcHijackingInterceptorFactory // single RPC should be made on the channel before calling the Verify methods. class LoggingInterceptor : public experimental::Interceptor { public: - LoggingInterceptor(experimental::ClientRpcInfo* info) { + LoggingInterceptor(experimental::ClientRpcInfo* /*info*/) { pre_send_initial_metadata_ = false; pre_send_message_count_ = 0; pre_send_close_ = false; diff --git a/test/cpp/end2end/delegating_channel_test.cc b/test/cpp/end2end/delegating_channel_test.cc index 4b8ff6f353e..e3e975bbfc8 100644 --- a/test/cpp/end2end/delegating_channel_test.cc +++ b/test/cpp/end2end/delegating_channel_test.cc @@ -48,7 +48,7 @@ class TestChannel : public experimental::DelegatingChannel { TestChannel(const std::shared_ptr& delegate_channel) : experimental::DelegatingChannel(delegate_channel) {} // Always returns GRPC_CHANNEL_READY - grpc_connectivity_state GetState(bool try_to_connect) override { + grpc_connectivity_state GetState(bool /*try_to_connect*/) override { return GRPC_CHANNEL_READY; } }; diff --git a/test/cpp/end2end/end2end_test.cc b/test/cpp/end2end/end2end_test.cc index c61c78b5201..34cb5e4cc50 100644 --- a/test/cpp/end2end/end2end_test.cc +++ b/test/cpp/end2end/end2end_test.cc @@ -212,7 +212,7 @@ class Proxy : public ::grpc::testing::EchoTestService::Service { class TestServiceImplDupPkg : public ::grpc::testing::duplicate::EchoTestService::Service { public: - Status Echo(ServerContext* context, const EchoRequest* request, + Status Echo(ServerContext* /*context*/, const EchoRequest* /*request*/, EchoResponse* response) override { response->set_message("no package"); return Status::OK; diff --git a/test/cpp/end2end/exception_test.cc b/test/cpp/end2end/exception_test.cc index 0d2c00263b7..cd29eb8a10b 100644 --- a/test/cpp/end2end/exception_test.cc +++ b/test/cpp/end2end/exception_test.cc @@ -39,13 +39,13 @@ const char* kErrorMessage = "This service caused an exception"; #if GRPC_ALLOW_EXCEPTIONS class ExceptingServiceImpl : public ::grpc::testing::EchoTestService::Service { public: - Status Echo(ServerContext* server_context, const EchoRequest* request, - EchoResponse* response) override { + Status Echo(ServerContext* /*server_context*/, const EchoRequest* /*request*/, + EchoResponse* /*response*/) override { throw - 1; } - Status RequestStream(ServerContext* context, - ServerReader* reader, - EchoResponse* response) override { + Status RequestStream(ServerContext* /*context*/, + ServerReader* /*reader*/, + EchoResponse* /*response*/) override { throw ServiceException(); } diff --git a/test/cpp/end2end/filter_end2end_test.cc b/test/cpp/end2end/filter_end2end_test.cc index a224d6596a3..91300268e78 100644 --- a/test/cpp/end2end/filter_end2end_test.cc +++ b/test/cpp/end2end/filter_end2end_test.cc @@ -99,8 +99,8 @@ int GetCallCounterValue() { class ChannelDataImpl : public ChannelData { public: - grpc_error* Init(grpc_channel_element* elem, - grpc_channel_element_args* args) { + grpc_error* Init(grpc_channel_element* /*elem*/, + grpc_channel_element_args* /*args*/) { IncrementConnectionCounter(); return GRPC_ERROR_NONE; } diff --git a/test/cpp/end2end/hybrid_end2end_test.cc b/test/cpp/end2end/hybrid_end2end_test.cc index d0175399eaf..9eacb032f7b 100644 --- a/test/cpp/end2end/hybrid_end2end_test.cc +++ b/test/cpp/end2end/hybrid_end2end_test.cc @@ -80,7 +80,7 @@ void HandleEcho(Service* service, ServerCompletionQueue* cq, bool dup_service) { // that the req/resp are ByteBuffers template void HandleRawEcho(Service* service, ServerCompletionQueue* cq, - bool dup_service) { + bool /*dup_service*/) { ServerContext srv_ctx; GenericServerAsyncResponseWriter response_writer(&srv_ctx); ByteBuffer recv_buffer; @@ -219,7 +219,7 @@ void HandleGenericCall(AsyncGenericService* service, class TestServiceImplDupPkg : public ::grpc::testing::duplicate::EchoTestService::Service { public: - Status Echo(ServerContext* context, const EchoRequest* request, + Status Echo(ServerContext* /*context*/, const EchoRequest* request, EchoResponse* response) override { response->set_message(request->message() + "_dup"); return Status::OK; @@ -566,7 +566,7 @@ class StreamedUnaryDupPkg TestServiceImplDupPkg> { public: Status StreamedEcho( - ServerContext* context, + ServerContext* /*context*/, ServerUnaryStreamer* stream) override { EchoRequest req; EchoResponse resp; @@ -604,7 +604,7 @@ class FullyStreamedUnaryDupPkg : public duplicate::EchoTestService::StreamedUnaryService { public: Status StreamedEcho( - ServerContext* context, + ServerContext* /*context*/, ServerUnaryStreamer* stream) override { EchoRequest req; EchoResponse resp; @@ -643,7 +643,7 @@ class SplitResponseStreamDupPkg WithSplitStreamingMethod_ResponseStream { public: Status StreamedResponseStream( - ServerContext* context, + ServerContext* /*context*/, ServerSplitStreamer* stream) override { EchoRequest req; EchoResponse resp; @@ -683,7 +683,7 @@ class FullySplitStreamedDupPkg : public duplicate::EchoTestService::SplitStreamedService { public: Status StreamedResponseStream( - ServerContext* context, + ServerContext* /*context*/, ServerSplitStreamer* stream) override { EchoRequest req; EchoResponse resp; @@ -722,7 +722,7 @@ TEST_F(HybridEnd2endTest, class FullyStreamedDupPkg : public duplicate::EchoTestService::StreamedService { public: Status StreamedEcho( - ServerContext* context, + ServerContext* /*context*/, ServerUnaryStreamer* stream) override { EchoRequest req; EchoResponse resp; @@ -735,7 +735,7 @@ class FullyStreamedDupPkg : public duplicate::EchoTestService::StreamedService { return Status::OK; } Status StreamedResponseStream( - ServerContext* context, + ServerContext* /*context*/, ServerSplitStreamer* stream) override { EchoRequest req; EchoResponse resp; diff --git a/test/cpp/end2end/interceptors_util.h b/test/cpp/end2end/interceptors_util.h index 1cd1448a6fa..6027c9b3dcf 100644 --- a/test/cpp/end2end/interceptors_util.h +++ b/test/cpp/end2end/interceptors_util.h @@ -72,12 +72,12 @@ class DummyInterceptorFactory public experimental::ServerInterceptorFactoryInterface { public: virtual experimental::Interceptor* CreateClientInterceptor( - experimental::ClientRpcInfo* info) override { + experimental::ClientRpcInfo* /*info*/) override { return new DummyInterceptor(); } virtual experimental::Interceptor* CreateServerInterceptor( - experimental::ServerRpcInfo* info) override { + experimental::ServerRpcInfo* /*info*/) override { return new DummyInterceptor(); } }; @@ -88,12 +88,12 @@ class NullInterceptorFactory public experimental::ServerInterceptorFactoryInterface { public: virtual experimental::Interceptor* CreateClientInterceptor( - experimental::ClientRpcInfo* info) override { + experimental::ClientRpcInfo* /*info*/) override { return nullptr; } virtual experimental::Interceptor* CreateServerInterceptor( - experimental::ServerRpcInfo* info) override { + experimental::ServerRpcInfo* /*info*/) override { return nullptr; } };