Sanity checks and return value

pull/19316/head
Yash Tibrewal 6 years ago
parent 94e7edad99
commit 90f1c32b85
  1. 4
      include/grpcpp/impl/codegen/delegating_channel.h
  2. 2
      test/cpp/end2end/delegating_channel_test.cc

@ -30,7 +30,7 @@ class DelegatingChannel : public ::grpc::ChannelInterface {
: delegate_channel_(delegate_channel) {}
grpc_connectivity_state GetState(bool try_to_connect) override {
delegate_channel()->GetState(try_to_connect);
return delegate_channel()->GetState(try_to_connect);
}
std::shared_ptr<::grpc::ChannelInterface> delegate_channel() {
@ -84,4 +84,4 @@ class DelegatingChannel : public ::grpc::ChannelInterface {
} // namespace internal
} // namespace grpc
#endif // GRPCPP_IMPL_CODEGEN_DELEGATING_CHANNEL_H
#endif // GRPCPP_IMPL_CODEGEN_DELEGATING_CHANNEL_H

@ -46,7 +46,7 @@ namespace {
class TestChannel : public internal::DelegatingChannel {
public:
TestChannel(std::shared_ptr<ChannelInterface> delegate_channel)
: internal::DelegatingChannel(delegate_channel) {}
: internal::DelegatingChannel(std::move(delegate_channel)) {}
// Always returns GRPC_CHANNEL_READY
grpc_connectivity_state GetState(bool try_to_connect) override {
return GRPC_CHANNEL_READY;

Loading…
Cancel
Save