Move cpp_common to base and its own namespace.
Looks like in next CL we need more shared bits (e.g. TaskRunner for
the Ftrace parser).
It makes sense to start splitting "base.h" in actual units before it
becomes too late and too painful.
Also turns out that the policy of just not including base headers
in public API headers isn't enough. Protobuf autogenerated .pb.h
headers in fact pull in a different definition of CHECK.
Hence moving our macros to be PERFETTO_CHECK and so on.
Change-Id: I9d340c6a26b94ee54ba45472c78409a432364100
diff --git a/build/BUILD.gn b/build/BUILD.gn
index 3fa8f70..a4527a5 100644
--- a/build/BUILD.gn
+++ b/build/BUILD.gn
@@ -32,7 +32,6 @@
"-Wno-gnu-zero-variadic-macro-arguments",
"-Wno-padded",
"-Wno-reserved-id-macro",
- "-Wno-unknown-warning-option",
"-Wno-unused-parameter",
"-Wno-weak-vtables",
]
@@ -72,7 +71,10 @@
# Color compiler output, see https://github.com/ninja-build/ninja/wiki/FAQ
if (is_clang) {
- cflags += [ "-fcolor-diagnostics" ]
+ cflags += [
+ "-fcolor-diagnostics",
+ "-Wno-unknown-warning-option",
+ ]
}
if (current_cpu == "arm") {
