diff options
| author | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2024-09-16 15:31:15 +0200 |
|---|---|---|
| committer | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2024-09-17 14:43:21 +0200 |
| commit | 59a0ce6d9df4465f2a7e6cbeb78a339f30574ae6 (patch) | |
| tree | 8c42280acfc5603bcf1f4734cf29a05d0263286e /src/buildtool/execution_engine/dag/dag.cpp | |
| parent | a1b45eef0a50da931a2c46fe842631d27ca4be56 (diff) | |
| download | justbuild-59a0ce6d9df4465f2a7e6cbeb78a339f30574ae6.tar.gz | |
Small code improvements based on lint warnings
- add more noexcept requirements and enforce existing
- fixing inconsistencies related to function arguments
- remove redundant static keywords
- silencing excessive lint reporting in test cases
While there, make more getters const ref.
Diffstat (limited to 'src/buildtool/execution_engine/dag/dag.cpp')
| -rw-r--r-- | src/buildtool/execution_engine/dag/dag.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/buildtool/execution_engine/dag/dag.cpp b/src/buildtool/execution_engine/dag/dag.cpp index 17f4b3af..cb82ec55 100644 --- a/src/buildtool/execution_engine/dag/dag.cpp +++ b/src/buildtool/execution_engine/dag/dag.cpp @@ -148,7 +148,7 @@ auto DependencyGraph::AddAction(ActionDescription const& description) -> bool { auto DependencyGraph::AddAction(Action const& a) noexcept -> DependencyGraph::ActionNodeIdentifier { - auto id = a.Id(); + auto const& id = a.Id(); auto const action_it = action_ids_.find(id); if (action_it != action_ids_.end()) { return action_it->second; |