diff options
| author | Maksim Denisov <denisov.maksim@huawei.com> | 2024-09-26 14:20:16 +0200 |
|---|---|---|
| committer | Maksim Denisov <denisov.maksim@huawei.com> | 2024-09-26 16:54:16 +0200 |
| commit | 5baab75fd2ae62b6f6407991922fe234f9e73c88 (patch) | |
| tree | bc2cb07d625dc6866b0be9a1e347e51467df1703 /src/buildtool/build_engine/expression/expression.hpp | |
| parent | 845744929e40dbdc81ed9c7df0152d58bbb28be6 (diff) | |
| download | justbuild-5baab75fd2ae62b6f6407991922fe234f9e73c88.tar.gz | |
Fix redundant std::optional conversions
...proposed by clang-tidy.
Enable bugprone-optional-value-conversion check.
Diffstat (limited to 'src/buildtool/build_engine/expression/expression.hpp')
| -rw-r--r-- | src/buildtool/build_engine/expression/expression.hpp | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/buildtool/build_engine/expression/expression.hpp b/src/buildtool/build_engine/expression/expression.hpp index cbfb2f24..e5788e38 100644 --- a/src/buildtool/build_engine/expression/expression.hpp +++ b/src/buildtool/build_engine/expression/expression.hpp @@ -152,11 +152,7 @@ class Expression { [[nodiscard]] auto At( std::string const& key) && -> std::optional<ExpressionPtr> { - auto value = std::move(*this).Map().Find(key); - if (value) { - return std::move(*value); - } - return std::nullopt; + return std::move(*this).Map().Find(key); } template <class T> |