Switch reference_less_video_analysis to ABSL_FLAG.
Bug: webrtc:10616
Change-Id: I7b8b1cdae64d8d704056c19806ff8d4708810210
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/143481
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#28364}
diff --git a/rtc_tools/BUILD.gn b/rtc_tools/BUILD.gn
index ffade27..c7022b0 100644
--- a/rtc_tools/BUILD.gn
+++ b/rtc_tools/BUILD.gn
@@ -232,13 +232,15 @@
}
rtc_executable("reference_less_video_analysis") {
+ testonly = true
sources = [
"frame_analyzer/reference_less_video_analysis.cc",
]
deps = [
- ":command_line_parser",
":reference_less_video_analysis_lib",
+ "//third_party/abseil-cpp/absl/flags:flag",
+ "//third_party/abseil-cpp/absl/flags:parse",
]
}
diff --git a/rtc_tools/frame_analyzer/reference_less_video_analysis.cc b/rtc_tools/frame_analyzer/reference_less_video_analysis.cc
index 3d7fd7f..42ee18f 100644
--- a/rtc_tools/frame_analyzer/reference_less_video_analysis.cc
+++ b/rtc_tools/frame_analyzer/reference_less_video_analysis.cc
@@ -9,36 +9,37 @@
*/
#include <stdio.h>
#include <stdlib.h>
-#include <string.h>
+#include <string>
+
+#include "absl/flags/flag.h"
+#include "absl/flags/parse.h"
#include "rtc_tools/frame_analyzer/reference_less_video_analysis_lib.h"
-#include "rtc_tools/simple_command_line_parser.h"
+
+ABSL_FLAG(std::string,
+ video_file,
+ "",
+ "Path of the video file to be analyzed, only y4m file format is "
+ "supported");
int main(int argc, char* argv[]) {
+ absl::ParseCommandLine(argc, argv);
+ // TODO(bugs.webrtc.org/10616): Add program usage message when Abseil
+ // flags supports it.
// This captures the freezing metrics for reference less video analysis.
- std::string program_name = argv[0];
- std::string usage =
- "Outputs the freezing score by comparing current frame "
- "with the previous frame.\nExample usage:\n" +
- program_name +
- " --video_file=video_file.y4m\n"
- "Command line flags:\n"
- " - video_file(string): Path of the video "
- "file to be analyzed. Only y4m file format is supported.\n";
+ // std::string usage =
+ // "Outputs the freezing score by comparing current frame "
+ // "with the previous frame.\nExample usage:\n" +
+ // program_name +
+ // " --video_file=video_file.y4m\n"
+ // "Command line flags:\n"
+ // " - video_file(string): Path of the video "
+ // "file to be analyzed. Only y4m file format is supported.\n";
- webrtc::test::CommandLineParser parser;
-
- // Init the parser and set the usage message.
- parser.Init(argc, argv);
- parser.SetUsageMessage(usage);
-
- parser.SetFlag("video_file", "");
- parser.ProcessFlags();
- if (parser.GetFlag("video_file").empty()) {
- parser.PrintUsageMessage();
- exit(EXIT_SUCCESS);
+ std::string video_file = absl::GetFlag(FLAGS_video_file);
+ if (video_file.empty()) {
+ exit(EXIT_FAILURE);
}
- std::string video_file = parser.GetFlag("video_file");
return run_analysis(video_file);
}