From a025eb58ab3fbd9236adaac7e3d5d84962209bac Mon Sep 17 00:00:00 2001 From: Mikael Olsson Date: Mon, 19 Feb 2024 10:34:06 +0100 Subject: Fix unnecessary string copying in inference runner To avoid copying strings unnecessarily in the inference runner the functions have been changed to take the strings as const reference rather than copying them. Change-Id: I173e37c7f7b08ada5d421bb9008a4bf36284f6c5 Signed-off-by: Mikael Olsson --- utils/inference_runner/inference_runner.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/utils/inference_runner/inference_runner.cpp b/utils/inference_runner/inference_runner.cpp index 531bd4e..34fb07c 100644 --- a/utils/inference_runner/inference_runner.cpp +++ b/utils/inference_runner/inference_runner.cpp @@ -33,7 +33,7 @@ using namespace EthosU; namespace { int64_t defaultTimeout = 60000000000; -void help(const string exe) { +void help(const string &exe) { cerr << "Usage: " << exe << " [ARGS]\n"; cerr << "\n"; cerr << "Arguments:\n"; @@ -50,14 +50,14 @@ void help(const string exe) { cerr << endl; } -void rangeCheck(const int i, const int argc, const string arg) { +void rangeCheck(const int i, const int argc, const string &arg) { if (i >= argc) { cerr << "Error: Missing argument to '" << arg << "'" << endl; exit(1); } } -pair, size_t> getNetworkData(const string filename) { +pair, size_t> getNetworkData(const string &filename) { ifstream stream(filename, ios::binary); if (!stream.is_open()) { cerr << "Error: Failed to open '" << filename << "'" << endl; -- cgit v1.2.1