From 1ff11f6d044bdaf46f822ce3ad8fee2980496b70 Mon Sep 17 00:00:00 2001 From: Yang Gao Date: Wed, 14 Jan 2015 11:45:32 -0800 Subject: [PATCH 1/2] Add a space in front of template argument list. Changing possible <:: to < :: --- src/compiler/cpp_generator.cc | 56 +++++++++++++++++------------------ 1 file changed, 28 insertions(+), 28 deletions(-) diff --git a/src/compiler/cpp_generator.cc b/src/compiler/cpp_generator.cc index 11160498068..41adb81c6b7 100644 --- a/src/compiler/cpp_generator.cc +++ b/src/compiler/cpp_generator.cc @@ -143,16 +143,16 @@ void PrintHeaderClientMethod(google::protobuf::io::Printer* printer, } else if (ClientOnlyStreaming(method)) { printer->Print( *vars, - "::grpc::ClientWriter<$Request$>* $Method$(" + "::grpc::ClientWriter< $Request$>* $Method$(" "::grpc::ClientContext* context, $Response$* response);\n\n"); } else if (ServerOnlyStreaming(method)) { printer->Print( *vars, - "::grpc::ClientReader<$Response$>* $Method$(" + "::grpc::ClientReader< $Response$>* $Method$(" "::grpc::ClientContext* context, const $Request$* request);\n\n"); } else if (BidiStreaming(method)) { printer->Print(*vars, - "::grpc::ClientReaderWriter<$Request$, $Response$>* " + "::grpc::ClientReaderWriter< $Request$, $Response$>* " "$Method$(::grpc::ClientContext* context);\n\n"); } } @@ -174,18 +174,18 @@ void PrintHeaderServerMethod(google::protobuf::io::Printer* printer, printer->Print(*vars, "virtual ::grpc::Status $Method$(" "::grpc::ServerContext* context, " - "::grpc::ServerReader<$Request$>* reader, " + "::grpc::ServerReader< $Request$>* reader, " "$Response$* response);\n"); } else if (ServerOnlyStreaming(method)) { printer->Print(*vars, "virtual ::grpc::Status $Method$(" "::grpc::ServerContext* context, const $Request$* request, " - "::grpc::ServerWriter<$Response$>* writer);\n"); + "::grpc::ServerWriter< $Response$>* writer);\n"); } else if (BidiStreaming(method)) { printer->Print(*vars, "virtual ::grpc::Status $Method$(" "::grpc::ServerContext* context, " - "::grpc::ServerReaderWriter<$Response$, $Request$>* stream);" + "::grpc::ServerReaderWriter< $Response$, $Request$>* stream);" "\n"); } } @@ -211,7 +211,7 @@ void PrintHeaderService(google::protobuf::io::Printer* printer, printer->Outdent(); printer->Print("};\n"); printer->Print( - "static Stub* NewStub(const std::shared_ptr<::grpc::ChannelInterface>& " + "static Stub* NewStub(const std::shared_ptr< ::grpc::ChannelInterface>& " "channel);\n"); printer->Print("\n"); @@ -270,10 +270,10 @@ void PrintSourceClientMethod(google::protobuf::io::Printer* printer, "}\n\n"); } else if (ClientOnlyStreaming(method)) { printer->Print(*vars, - "::grpc::ClientWriter<$Request$>* $Service$::Stub::$Method$(" + "::grpc::ClientWriter< $Request$>* $Service$::Stub::$Method$(" "::grpc::ClientContext* context, $Response$* response) {\n"); printer->Print(*vars, - " return new ::grpc::ClientWriter<$Request$>(" + " return new ::grpc::ClientWriter< $Request$>(" "channel()->CreateStream(" "::grpc::RpcMethod(\"/$Package$$Service$/$Method$\", " "::grpc::RpcMethod::RpcType::CLIENT_STREAMING), " @@ -282,10 +282,10 @@ void PrintSourceClientMethod(google::protobuf::io::Printer* printer, } else if (ServerOnlyStreaming(method)) { printer->Print( *vars, - "::grpc::ClientReader<$Response$>* $Service$::Stub::$Method$(" + "::grpc::ClientReader< $Response$>* $Service$::Stub::$Method$(" "::grpc::ClientContext* context, const $Request$* request) {\n"); printer->Print(*vars, - " return new ::grpc::ClientReader<$Response$>(" + " return new ::grpc::ClientReader< $Response$>(" "channel()->CreateStream(" "::grpc::RpcMethod(\"/$Package$$Service$/$Method$\", " "::grpc::RpcMethod::RpcType::SERVER_STREAMING), " @@ -294,11 +294,11 @@ void PrintSourceClientMethod(google::protobuf::io::Printer* printer, } else if (BidiStreaming(method)) { printer->Print( *vars, - "::grpc::ClientReaderWriter<$Request$, $Response$>* " + "::grpc::ClientReaderWriter< $Request$, $Response$>* " "$Service$::Stub::$Method$(::grpc::ClientContext* context) {\n"); printer->Print( *vars, - " return new ::grpc::ClientReaderWriter<$Request$, $Response$>(" + " return new ::grpc::ClientReaderWriter< $Request$, $Response$>(" "channel()->CreateStream(" "::grpc::RpcMethod(\"/$Package$$Service$/$Method$\", " "::grpc::RpcMethod::RpcType::BIDI_STREAMING), " @@ -328,7 +328,7 @@ void PrintSourceServerMethod(google::protobuf::io::Printer* printer, printer->Print(*vars, "::grpc::Status $Service$::Service::$Method$(" "::grpc::ServerContext* context, " - "::grpc::ServerReader<$Request$>* reader, " + "::grpc::ServerReader< $Request$>* reader, " "$Response$* response) {\n"); printer->Print( " return ::grpc::Status(" @@ -339,7 +339,7 @@ void PrintSourceServerMethod(google::protobuf::io::Printer* printer, "::grpc::Status $Service$::Service::$Method$(" "::grpc::ServerContext* context, " "const $Request$* request, " - "::grpc::ServerWriter<$Response$>* writer) {\n"); + "::grpc::ServerWriter< $Response$>* writer) {\n"); printer->Print( " return ::grpc::Status(" "::grpc::StatusCode::UNIMPLEMENTED);\n"); @@ -348,7 +348,7 @@ void PrintSourceServerMethod(google::protobuf::io::Printer* printer, printer->Print(*vars, "::grpc::Status $Service$::Service::$Method$(" "::grpc::ServerContext* context, " - "::grpc::ServerReaderWriter<$Response$, $Request$>* " + "::grpc::ServerReaderWriter< $Response$, $Request$>* " "stream) {\n"); printer->Print( " return ::grpc::Status(" @@ -363,7 +363,7 @@ void PrintSourceService(google::protobuf::io::Printer* printer, (*vars)["Service"] = service->name(); printer->Print(*vars, "$Service$::Stub* $Service$::NewStub(" - "const std::shared_ptr<::grpc::ChannelInterface>& channel) {\n" + "const std::shared_ptr< ::grpc::ChannelInterface>& channel) {\n" " $Service$::Stub* stub = new $Service$::Stub();\n" " stub->set_channel(channel);\n" " return stub;\n" @@ -400,9 +400,9 @@ void PrintSourceService(google::protobuf::io::Printer* printer, "service_->AddMethod(new ::grpc::RpcServiceMethod(\n" " \"/$Package$$Service$/$Method$\",\n" " ::grpc::RpcMethod::NORMAL_RPC,\n" - " new ::grpc::RpcMethodHandler<$Service$::Service, $Request$, " + " new ::grpc::RpcMethodHandler< $Service$::Service, $Request$, " "$Response$>(\n" - " std::function<::grpc::Status($Service$::Service*, " + " std::function< ::grpc::Status($Service$::Service*, " "::grpc::ServerContext*, const $Request$*, $Response$*)>(" "&$Service$::Service::$Method$), this),\n" " new $Request$, new $Response$));\n"); @@ -412,11 +412,11 @@ void PrintSourceService(google::protobuf::io::Printer* printer, "service_->AddMethod(new ::grpc::RpcServiceMethod(\n" " \"/$Package$$Service$/$Method$\",\n" " ::grpc::RpcMethod::CLIENT_STREAMING,\n" - " new ::grpc::ClientStreamingHandler<" + " new ::grpc::ClientStreamingHandler< " "$Service$::Service, $Request$, $Response$>(\n" - " std::function<::grpc::Status($Service$::Service*, " + " std::function< ::grpc::Status($Service$::Service*, " "::grpc::ServerContext*, " - "::grpc::ServerReader<$Request$>*, $Response$*)>(" + "::grpc::ServerReader< $Request$>*, $Response$*)>(" "&$Service$::Service::$Method$), this),\n" " new $Request$, new $Response$));\n"); } else if (ServerOnlyStreaming(method)) { @@ -425,11 +425,11 @@ void PrintSourceService(google::protobuf::io::Printer* printer, "service_->AddMethod(new ::grpc::RpcServiceMethod(\n" " \"/$Package$$Service$/$Method$\",\n" " ::grpc::RpcMethod::SERVER_STREAMING,\n" - " new ::grpc::ServerStreamingHandler<" + " new ::grpc::ServerStreamingHandler< " "$Service$::Service, $Request$, $Response$>(\n" - " std::function<::grpc::Status($Service$::Service*, " + " std::function< ::grpc::Status($Service$::Service*, " "::grpc::ServerContext*, " - "const $Request$*, ::grpc::ServerWriter<$Response$>*)>(" + "const $Request$*, ::grpc::ServerWriter< $Response$>*)>(" "&$Service$::Service::$Method$), this),\n" " new $Request$, new $Response$));\n"); } else if (BidiStreaming(method)) { @@ -438,11 +438,11 @@ void PrintSourceService(google::protobuf::io::Printer* printer, "service_->AddMethod(new ::grpc::RpcServiceMethod(\n" " \"/$Package$$Service$/$Method$\",\n" " ::grpc::RpcMethod::BIDI_STREAMING,\n" - " new ::grpc::BidiStreamingHandler<" + " new ::grpc::BidiStreamingHandler< " "$Service$::Service, $Request$, $Response$>(\n" - " std::function<::grpc::Status($Service$::Service*, " + " std::function< ::grpc::Status($Service$::Service*, " "::grpc::ServerContext*, " - "::grpc::ServerReaderWriter<$Response$, $Request$>*)>(" + "::grpc::ServerReaderWriter< $Response$, $Request$>*)>(" "&$Service$::Service::$Method$), this),\n" " new $Request$, new $Response$));\n"); } From 5680ff4687deaafc84a96d8f7c50f503c2581e71 Mon Sep 17 00:00:00 2001 From: Yang Gao Date: Wed, 14 Jan 2015 12:14:21 -0800 Subject: [PATCH 2/2] Run clang-format-all.sh --- src/compiler/cpp_generator.cc | 33 ++++++++++++++++++--------------- src/core/support/log_posix.c | 1 - test/core/util/test_config.c | 8 ++------ 3 files changed, 20 insertions(+), 22 deletions(-) diff --git a/src/compiler/cpp_generator.cc b/src/compiler/cpp_generator.cc index 41adb81c6b7..94e56d73a6e 100644 --- a/src/compiler/cpp_generator.cc +++ b/src/compiler/cpp_generator.cc @@ -182,11 +182,12 @@ void PrintHeaderServerMethod(google::protobuf::io::Printer* printer, "::grpc::ServerContext* context, const $Request$* request, " "::grpc::ServerWriter< $Response$>* writer);\n"); } else if (BidiStreaming(method)) { - printer->Print(*vars, - "virtual ::grpc::Status $Method$(" - "::grpc::ServerContext* context, " - "::grpc::ServerReaderWriter< $Response$, $Request$>* stream);" - "\n"); + printer->Print( + *vars, + "virtual ::grpc::Status $Method$(" + "::grpc::ServerContext* context, " + "::grpc::ServerReaderWriter< $Response$, $Request$>* stream);" + "\n"); } } @@ -269,9 +270,10 @@ void PrintSourceClientMethod(google::protobuf::io::Printer* printer, "context, request, response);\n" "}\n\n"); } else if (ClientOnlyStreaming(method)) { - printer->Print(*vars, - "::grpc::ClientWriter< $Request$>* $Service$::Stub::$Method$(" - "::grpc::ClientContext* context, $Response$* response) {\n"); + printer->Print( + *vars, + "::grpc::ClientWriter< $Request$>* $Service$::Stub::$Method$(" + "::grpc::ClientContext* context, $Response$* response) {\n"); printer->Print(*vars, " return new ::grpc::ClientWriter< $Request$>(" "channel()->CreateStream(" @@ -361,13 +363,14 @@ void PrintSourceService(google::protobuf::io::Printer* printer, const google::protobuf::ServiceDescriptor* service, map* vars) { (*vars)["Service"] = service->name(); - printer->Print(*vars, - "$Service$::Stub* $Service$::NewStub(" - "const std::shared_ptr< ::grpc::ChannelInterface>& channel) {\n" - " $Service$::Stub* stub = new $Service$::Stub();\n" - " stub->set_channel(channel);\n" - " return stub;\n" - "};\n\n"); + printer->Print( + *vars, + "$Service$::Stub* $Service$::NewStub(" + "const std::shared_ptr< ::grpc::ChannelInterface>& channel) {\n" + " $Service$::Stub* stub = new $Service$::Stub();\n" + " stub->set_channel(channel);\n" + " return stub;\n" + "};\n\n"); for (int i = 0; i < service->method_count(); ++i) { PrintSourceClientMethod(printer, service->method(i), vars); } diff --git a/src/core/support/log_posix.c b/src/core/support/log_posix.c index 55a38b136d0..ee2705a2c20 100644 --- a/src/core/support/log_posix.c +++ b/src/core/support/log_posix.c @@ -31,7 +31,6 @@ * */ - #ifndef _POSIX_C_SOURCE #define _POSIX_C_SOURCE 200112L #endif diff --git a/test/core/util/test_config.c b/test/core/util/test_config.c index 94245d85bcc..44ab35fc5e4 100644 --- a/test/core/util/test_config.c +++ b/test/core/util/test_config.c @@ -39,16 +39,12 @@ #if GPR_GETPID_IN_UNISTD_H #include -static int seed() { - return getpid(); -} +static int seed() { return getpid(); } #endif #if GPR_GETPID_IN_PROCESS_H #include -static int seed(void) { - return _getpid(); -} +static int seed(void) { return _getpid(); } #endif void grpc_test_init(int argc, char **argv) {