diff options
author | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2024-03-15 12:42:42 +0100 |
---|---|---|
committer | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2024-03-18 10:57:51 +0100 |
commit | a8516db696890571f454b7a396b71ef567acb17f (patch) | |
tree | 1ab248c17209191901bcc8a93d8514699e59fd51 /src/buildtool/graph_traverser/graph_traverser.hpp | |
parent | c328856cb218a8df8d04b021b3ef12ad8f9729b4 (diff) | |
download | justbuild-a8516db696890571f454b7a396b71ef567acb17f.tar.gz |
graph_traverser: Remove unnecessary get call
...as it is redundant here and exception prone.
Diffstat (limited to 'src/buildtool/graph_traverser/graph_traverser.hpp')
-rw-r--r-- | src/buildtool/graph_traverser/graph_traverser.hpp | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/buildtool/graph_traverser/graph_traverser.hpp b/src/buildtool/graph_traverser/graph_traverser.hpp index 83876072..c301177a 100644 --- a/src/buildtool/graph_traverser/graph_traverser.hpp +++ b/src/buildtool/graph_traverser/graph_traverser.hpp @@ -139,7 +139,7 @@ class GraphTraverser { if (not artifacts) { return std::nullopt; } - auto const [rel_paths, artifact_nodes, extra_nodes] = *artifacts; + auto [rel_paths, artifact_nodes, extra_nodes] = *artifacts; auto const object_infos = CollectObjectInfos(artifact_nodes); auto extra_infos = CollectObjectInfos(extra_nodes); @@ -168,10 +168,9 @@ class GraphTraverser { artifact_nodes, runfile_descriptions); MaybePrintToStdout(rel_paths, artifact_nodes); - return BuildResult{ - .output_paths = std::move(std::get<0>(*artifacts)), - .extra_infos = std::move(infos), - .failed_artifacts = failed_artifacts}; + return BuildResult{.output_paths = std::move(rel_paths), + .extra_infos = std::move(infos), + .failed_artifacts = failed_artifacts}; } if (clargs_.stage->remember) { |