commit 06cdb23644fbb137a937869732e25eddfdb2c275
author Mirko Bonadei <mbonadei@webrtc.org> Fri Jul 19 10:02:46 2019 +0200
committer Commit Bot <commit-bot@chromium.org> Fri Jul 19 08:49:13 2019 +0000
tree 9f3a8bbb6307a12bc11414ca7632387ccfbbca95
parent 857ad6272180a36b62e8e024c75d454ca8da1320

Add usage message to rtp_generator.

Bug: webrtc:10616
Change-Id: Ia13b1e829063986be6b3b4a4b0c194dd4fea3511
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/146202
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#28609}

rtc_tools/BUILD.gn

diff --git a/rtc_tools/BUILD.gn b/rtc_tools/BUILD.gn
index f684528..28a0547 100644
--- a/rtc_tools/BUILD.gn
+++ b/rtc_tools/BUILD.gn
@@ -182,6 +182,7 @@
       "../test:video_test_common",
       "//third_party/abseil-cpp/absl/flags:flag",
       "//third_party/abseil-cpp/absl/flags:parse",
+      "//third_party/abseil-cpp/absl/flags:usage",
       "//third_party/abseil-cpp/absl/memory",
       "//third_party/abseil-cpp/absl/strings",
     ]

rtc_tools/rtp_generator/main.cc

diff --git a/rtc_tools/rtp_generator/main.cc b/rtc_tools/rtp_generator/main.cc
index 0e5dee8..df49576 100644
--- a/rtc_tools/rtp_generator/main.cc
+++ b/rtc_tools/rtp_generator/main.cc
@@ -14,22 +14,20 @@
 
 #include "absl/flags/flag.h"
 #include "absl/flags/parse.h"
+#include "absl/flags/usage.h"
 #include "rtc_tools/rtp_generator/rtp_generator.h"
 
 ABSL_FLAG(std::string, input_config, "", "JSON file with config");
 ABSL_FLAG(std::string, output_rtpdump, "", "Where to store the rtpdump");
 
 int main(int argc, char* argv[]) {
+  absl::SetProgramUsageMessage(
+      "Generates custom configured rtpdumps for the purpose of testing.\n"
+      "Example Usage:\n"
+      "./rtp_generator --input_config=sender_config.json\n"
+      "                --output_rtpdump=my.rtpdump\n");
   absl::ParseCommandLine(argc, argv);
 
-  // TODO(bugs.webrtc.org/10616): Add program usage message when Abseil
-  // flags supports it.
-  // const std::string usage =
-  //    "Generates custom configured rtpdumps for the purpose of testing.\n"
-  //    "Example Usage:\n"
-  //    "./rtp_generator --input_config=sender_config.json\n"
-  //    "                --output_rtpdump=my.rtpdump\n";
-
   const std::string config_path = absl::GetFlag(FLAGS_input_config);
   const std::string rtp_dump_path = absl::GetFlag(FLAGS_output_rtpdump);