diff --git a/tools/lib/logreader.py b/tools/lib/logreader.py index e354dc8b49..763ef35cd6 100755 --- a/tools/lib/logreader.py +++ b/tools/lib/logreader.py @@ -336,9 +336,7 @@ if __name__ == "__main__": parser = ArgumentParser(description="Process a log file and print identifiers or full messages.") parser.add_argument("log_path", help="Path to the log file") - parser.add_argument( - "--identifiers-only", action="store_true", help="Print only log identifiers instead of full messages" - ) + parser.add_argument("--identifiers-only", action="store_true", help="Print only log identifiers") args = parser.parse_args() lr = LogReader(args.log_path, sort_by_time=True) diff --git a/tools/replay/main.cc b/tools/replay/main.cc index 2346ef8faf..3137d6d465 100644 --- a/tools/replay/main.cc +++ b/tools/replay/main.cc @@ -97,13 +97,9 @@ bool parseArgs(int argc, char *argv[], ReplayConfig &config) { case 'p': config.prefix = optarg; break; case 0: { std::string name = cli_options[option_index].name; - if (name == "demo") { - config.route = DEMO_ROUTE; - } else if (name == "auto") { - config.auto_source = true; - } else { - config.flags |= flag_map.at(name); - } + if (name == "demo") config.route = DEMO_ROUTE; + else if (name == "auto") config.auto_source = true; + else config.flags |= flag_map.at(name); break; } case 'h': std::cout << helpText; return false; diff --git a/tools/replay/route.cc b/tools/replay/route.cc index 08cf470f83..40f4f030a2 100644 --- a/tools/replay/route.cc +++ b/tools/replay/route.cc @@ -77,8 +77,7 @@ bool Route::loadFromCommaApi() { bool Route::loadFromAutoSource() { auto cmd = util::string_format("python ../lib/logreader.py \"%s\" --identifiers-only", route_string_.c_str()); - auto output = util::check_output(cmd); - auto log_files = split(output, '\n'); + auto log_files = split(util::check_output(cmd), '\n'); for (int i = 0; i < log_files.size(); ++i) { addFileToSegment(i, log_files[i]); } @@ -86,7 +85,6 @@ bool Route::loadFromAutoSource() { route_.end_segment = log_files.size() - 1; route_.dongle_id = route_string_; route_.str = route_string_; - route_.timestamp = ""; return !segments_.empty(); }