Skip to content
This repository was archived by the owner on Jul 31, 2023. It is now read-only.

Commit 4da30f7

Browse files
authored
Move RAPIDJSON_HAS_STDSTRING define to Zipkin exporter code. (#331)
This is cleaner than having it in the build rule.
1 parent 824c78a commit 4da30f7

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

WORKSPACE

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,6 @@ cc_library(
123123
"include/rapidjson/error/*.h",
124124
]),
125125
includes = ["include/"],
126-
defines = ["RAPIDJSON_HAS_STDSTRING=1",],
127126
visibility = ["//visibility:public"],
128127
)
129128
""",

opencensus/exporters/trace/zipkin/internal/zipkin_exporter.cc

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,8 @@
1919
#include <iostream>
2020

2121
#include <curl/curl.h>
22+
#undef RAPIDJSON_HAS_STDSTRING
23+
#define RAPIDJSON_HAS_STDSTRING 1
2224
#include <rapidjson/prettywriter.h>
2325
#include <rapidjson/stringbuffer.h>
2426
#include "absl/memory/memory.h"

0 commit comments

Comments
 (0)