From 819f755b74b217acfee017ec9aa7e8d0ea3a706a Mon Sep 17 00:00:00 2001 From: Chen Wang Date: Thu, 22 Jan 2015 14:52:49 -0800 Subject: [PATCH] fixed some typos --- examples/tips/client.cc | 7 +++---- examples/tips/client.h | 2 +- examples/tips/client_main.cc | 32 ++++++++++---------------------- examples/tips/client_test.cc | 4 ---- examples/tips/empty.proto | 6 ------ examples/tips/label.proto | 3 +-- 6 files changed, 15 insertions(+), 39 deletions(-) diff --git a/examples/tips/client.cc b/examples/tips/client.cc index bfbd59c9d56..a902a21c3c6 100644 --- a/examples/tips/client.cc +++ b/examples/tips/client.cc @@ -35,7 +35,6 @@ #include "examples/tips/client.h" -using grpc::ChannelInterface; using grpc::ClientContext; using tech::pubsub::Topic; using tech::pubsub::PublisherService; @@ -44,7 +43,7 @@ namespace grpc { namespace examples { namespace tips { -Client::Client(std::shared_ptr channel) +Client::Client(std::shared_ptr channel) : stub_(PublisherService::NewStub(channel)) { } @@ -52,11 +51,11 @@ Status Client::CreateTopic(grpc::string topic) { Topic request; Topic response; request.set_name(topic); - ClientContext context; + grpc::ClientContext context; return stub_->CreateTopic(&context, request, &response); } -} // namesapce tips +} // namespace tips } // namespace examples } // namespace grpc diff --git a/examples/tips/client.h b/examples/tips/client.h index 83e79118235..152a2a396dd 100644 --- a/examples/tips/client.h +++ b/examples/tips/client.h @@ -48,6 +48,6 @@ class Client { std::unique_ptr stub_; }; -} // namesapce tips +} // namespace tips } // namespace examples } // namespace grpc diff --git a/examples/tips/client_main.cc b/examples/tips/client_main.cc index 0324b4b8b08..17567b6f17a 100644 --- a/examples/tips/client_main.cc +++ b/examples/tips/client_main.cc @@ -31,40 +31,26 @@ * */ -#include -#include -#include -#include - #include #include #include -#include #include -#include #include #include -#include + +#include "examples/tips/client.h" #include "test/cpp/util/create_test_channel.h" -DEFINE_bool(enable_ssl, false, "Whether to use ssl/tls."); -DEFINE_bool(use_prod_roots, false, "True to use SSL roots for production GFE"); +DEFINE_bool(enable_ssl, true, "Whether to use ssl/tls."); +DEFINE_bool(use_prod_roots, true, "True to use SSL roots for production GFE"); DEFINE_int32(server_port, 0, "Server port."); DEFINE_string(server_host, "127.0.0.1", "Server host to connect to"); DEFINE_string(server_host_override, "foo.test.google.com", "Override the server host which is sent in HTTP header"); -using grpc::ChannelInterface; -using grpc::ClientContext; -using grpc::CreateTestChannel; - -void CreateTopic(std::shared_ptr channel, grpc::string topic); - int main(int argc, char** argv) { grpc_init(); - google::ParseCommandLineFlags(&argc, &argv, true); - gpr_log(GPR_INFO, "Start TIPS client"); GPR_ASSERT(FLAGS_server_port); @@ -73,11 +59,13 @@ int main(int argc, char** argv) { snprintf(host_port, host_port_buf_size, "%s:%d", FLAGS_server_host.c_str(), FLAGS_server_port); - std::shared_ptr channel( - CreateTestChannel(host_port, FLAGS_server_host_override, FLAGS_enable_ssl, - FLAGS_use_prod_roots)); + std::shared_ptr channel( + grpc::CreateTestChannel(host_port, FLAGS_server_host_override, + FLAGS_enable_ssl, FLAGS_use_prod_roots)); - CreateTopic(channel, "test"); + grpc::examples::tips::Client client(channel); + grpc::Status s = client.CreateTopic("test"); + GPR_ASSERT(s.IsOk()); channel.reset(); grpc_shutdown(); diff --git a/examples/tips/client_test.cc b/examples/tips/client_test.cc index d40f0c0d8b4..69238f2c6fc 100644 --- a/examples/tips/client_test.cc +++ b/examples/tips/client_test.cc @@ -47,10 +47,6 @@ using grpc::ChannelInterface; -namespace { - -} // - namespace grpc { namespace testing { namespace { diff --git a/examples/tips/empty.proto b/examples/tips/empty.proto index a698ab6f8a9..adf66b5e614 100644 --- a/examples/tips/empty.proto +++ b/examples/tips/empty.proto @@ -10,10 +10,4 @@ package proto2; // rpc Bar (proto2.Empty) returns (proto2.Empty) { }; // }; // -// BEGIN GOOGLE-INTERNAL -// The difference between this one and net/rpc/empty-message.proto is that -// 1) The generated message here is in proto2 C++ API. -// 2) The proto2.Empty has minimum dependencies -// (no message_set or net/rpc dependencies) -// END GOOGLE-INTERNAL message Empty {} diff --git a/examples/tips/label.proto b/examples/tips/label.proto index e79e1db77c2..e93ac9dea30 100644 --- a/examples/tips/label.proto +++ b/examples/tips/label.proto @@ -1,7 +1,6 @@ // Labels provide a way to associate user-defined metadata with various // objects. Labels may be used to organize objects into non-hierarchical -// groups; think metadata tags attached to mp3s. For more details, see -// http://go/exosphere:labels +// groups; think metadata tags attached to mp3s. syntax = "proto2";