Rename CHECK and FAIL macros to avoid conflict with internal macros when syncing back

This commit is contained in:
Kun Zhang 2015-09-01 12:47:25 -07:00
parent ad6c26e85c
commit db0423cf24
2 changed files with 9 additions and 9 deletions

View File

@ -188,7 +188,7 @@ static void PrintStub(
interface_name += "BlockingServer"; interface_name += "BlockingServer";
break; break;
default: default:
FAIL << "Cannot determine class name for StubType: " << type; GRPC_CODEGEN_FAIL << "Cannot determine class name for StubType: " << type;
} }
bool impl; bool impl;
CallType call_type; CallType call_type;
@ -219,7 +219,7 @@ static void PrintStub(
impl = true; impl = true;
break; break;
default: default:
FAIL << "Cannot determine call type for StubType: " << type; GRPC_CODEGEN_FAIL << "Cannot determine call type for StubType: " << type;
} }
(*vars)["interface_name"] = interface_name; (*vars)["interface_name"] = interface_name;
(*vars)["impl_name"] = impl_name; (*vars)["impl_name"] = impl_name;
@ -300,9 +300,9 @@ static void PrintStub(
switch (call_type) { switch (call_type) {
case BLOCKING_CALL: case BLOCKING_CALL:
// TODO(zhangkun83): decide the blocking server interface // TODO(zhangkun83): decide the blocking server interface
CHECK(type != BLOCKING_SERVER_INTERFACE) GRPC_CODEGEN_CHECK(type != BLOCKING_SERVER_INTERFACE)
<< "Blocking server interface is not available"; << "Blocking server interface is not available";
CHECK(!client_streaming) GRPC_CODEGEN_CHECK(!client_streaming)
<< "Blocking client interface with client streaming is unavailable"; << "Blocking client interface with client streaming is unavailable";
if (server_streaming) { if (server_streaming) {
// Server streaming // Server streaming
@ -333,7 +333,7 @@ static void PrintStub(
} }
break; break;
case FUTURE_CALL: case FUTURE_CALL:
CHECK(!client_streaming && !server_streaming) GRPC_CODEGEN_CHECK(!client_streaming && !server_streaming)
<< "Future interface doesn't support streaming. " << "Future interface doesn't support streaming. "
<< "client_streaming=" << client_streaming << ", " << "client_streaming=" << client_streaming << ", "
<< "server_streaming=" << server_streaming; << "server_streaming=" << server_streaming;
@ -349,7 +349,7 @@ static void PrintStub(
p->Indent(); p->Indent();
switch (call_type) { switch (call_type) {
case BLOCKING_CALL: case BLOCKING_CALL:
CHECK(!client_streaming) GRPC_CODEGEN_CHECK(!client_streaming)
<< "Blocking client streaming interface is not available"; << "Blocking client streaming interface is not available";
if (server_streaming) { if (server_streaming) {
(*vars)["calls_method"] = "blockingServerStreamingCall"; (*vars)["calls_method"] = "blockingServerStreamingCall";
@ -388,7 +388,7 @@ static void PrintStub(
" channel.newCall($method_field_name$, callOptions), $params$);\n"); " channel.newCall($method_field_name$, callOptions), $params$);\n");
break; break;
case FUTURE_CALL: case FUTURE_CALL:
CHECK(!client_streaming && !server_streaming) GRPC_CODEGEN_CHECK(!client_streaming && !server_streaming)
<< "Future interface doesn't support streaming. " << "Future interface doesn't support streaming. "
<< "client_streaming=" << client_streaming << ", " << "client_streaming=" << client_streaming << ", "
<< "server_streaming=" << server_streaming; << "server_streaming=" << server_streaming;

View File

@ -27,12 +27,12 @@ class LogHelper {
// Abort the program after logging the mesage if the given condition is not // Abort the program after logging the mesage if the given condition is not
// true. Otherwise, do nothing. // true. Otherwise, do nothing.
#define CHECK(x) !(x) && LogHelper(&std::cerr, true).get_os() \ #define GRPC_CODEGEN_CHECK(x) !(x) && LogHelper(&std::cerr, true).get_os() \
<< "CHECK FAILED: " << __FILE__ << ":" \ << "CHECK FAILED: " << __FILE__ << ":" \
<< __LINE__ << ": " << __LINE__ << ": "
// Abort the program after logging the mesage. // Abort the program after logging the mesage.
#define FAIL CHECK(false) #define GRPC_CODEGEN_FAIL GRPC_CODEGEN_CHECK(false)
using namespace std; using namespace std;