Skip to content
This repository was archived by the owner on Jan 27, 2025. It is now read-only.

Commit 6da2a26

Browse files
authored
Merge pull request #3 from liblaf/perf
perf(base): use reference for const parameter
2 parents c1eab9c + b15a9be commit 6da2a26

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/wrapper/wrapper_base.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ std::vector<std::filesystem::path> WrapperBase::get_inputs(
5353

5454
std::vector<std::string> WrapperBase::get_options(
5555
const std::vector<std::string>& args,
56-
const std::vector<std::filesystem::path> inputs) {
56+
const std::vector<std::filesystem::path>& inputs) {
5757
std::vector<std::string> options;
5858
for (int i = 1; i < args.size(); ++i) {
5959
const auto& arg = args[i];

src/wrapper/wrapper_base.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ class WrapperBase : public Executor {
3232
const std::vector<Phase>& phases);
3333
std::vector<std::string> get_options(
3434
const std::vector<std::string>& args,
35-
const std::vector<std::filesystem::path> inputs);
35+
const std::vector<std::filesystem::path>& inputs);
3636
std::vector<std::string> join_args(
3737
const std::vector<std::string>& options,
3838
const std::vector<std::filesystem::path>& inputs);

0 commit comments

Comments
 (0)