diff options
author | Muhammad Qureshi <muhammadq@google.com> | 2020-01-10 14:03:13 -0800 |
---|---|---|
committer | Muhammad Qureshi <muhammadq@google.com> | 2020-01-23 11:39:38 -0800 |
commit | b11e00d655e0c855b09ac66fe3d0f4b36c806d7d (patch) | |
tree | caf6cf9bd7a0292bf5f7427736233351ab7d9af0 /tools/stats_log_api_gen/main.cpp | |
parent | 17c204de33821625f84cefc2c6f20d38566b5355 (diff) |
Add flag to stats-log-api-gen for WorkSource.
As part of getting rid of StatsLogInternal and moving every atom to be
logged from a module, this CL addresses 2 issues:
1. Add WorkSource logging to modules.
2. Gate WorkSource logging via a CLI flag to prevent WorkSource methods
from being included in modules that are Mainline modules and can't have
@hide calls.
Bug: 147541828
Test: m
Test: atest android.cts.statsd.atom.UidAtomTests
Change-Id: Icce2f6ed7298be5e5ceebfb05d87e6fa7cd6bdbb
Merged-In: Icce2f6ed7298be5e5ceebfb05d87e6fa7cd6bdbb
Diffstat (limited to 'tools/stats_log_api_gen/main.cpp')
-rw-r--r-- | tools/stats_log_api_gen/main.cpp | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/tools/stats_log_api_gen/main.cpp b/tools/stats_log_api_gen/main.cpp index 00a370484823..6089532e1ca4 100644 --- a/tools/stats_log_api_gen/main.cpp +++ b/tools/stats_log_api_gen/main.cpp @@ -512,6 +512,7 @@ print_usage() fprintf(stderr, " Optional for Java with module.\n"); fprintf(stderr, " Default is \"StatsLogInternal\"\n"); fprintf(stderr, " --supportQ Include support for Android Q.\n"); + fprintf(stderr, " --worksource Include support for logging WorkSource objects.\n"); } /** @@ -534,6 +535,7 @@ run(int argc, char const*const* argv) string javaPackage = DEFAULT_JAVA_PACKAGE; string javaClass = DEFAULT_JAVA_CLASS; bool supportQ = false; + bool supportWorkSource = false; int index = 1; while (index < argc) { @@ -626,6 +628,8 @@ run(int argc, char const*const* argv) atomsInfoCppHeaderImport = argv[index]; } else if (0 == strcmp("--supportQ", argv[index])) { supportQ = true; + } else if (0 == strcmp("--worksource", argv[index])) { + supportWorkSource = true; } index++; @@ -728,19 +732,15 @@ run(int argc, char const*const* argv) fprintf(stderr, "Unable to open file for write: %s\n", javaFilename.c_str()); return 1; } - // If this is for a specific module, the java package must also be provided. - if (moduleName != DEFAULT_MODULE_NAME && javaPackage== DEFAULT_JAVA_PACKAGE) { - fprintf(stderr, "Must supply --javaPackage if supplying a specific module\n"); - return 1; - } #if defined(STATS_SCHEMA_LEGACY) if (moduleName == DEFAULT_MODULE_NAME) { errorCount = android::stats_log_api_gen::write_stats_log_java_q( - out, atoms, attributionDecl); + out, atoms, attributionDecl, supportWorkSource); } else { errorCount = android::stats_log_api_gen::write_stats_log_java_q_for_module( - out, atoms, attributionDecl, moduleName, javaClass, javaPackage); + out, atoms, attributionDecl, moduleName, javaClass, javaPackage, + supportWorkSource); } #else @@ -749,7 +749,8 @@ run(int argc, char const*const* argv) javaPackage = "android.util"; } errorCount = android::stats_log_api_gen::write_stats_log_java( - out, atoms, attributionDecl, moduleName, javaClass, javaPackage, supportQ); + out, atoms, attributionDecl, moduleName, javaClass, javaPackage, supportQ, + supportWorkSource); #endif fclose(out); |