diff options
author | Alberto Sartori <alberto.sartori@huawei.com> | 2022-03-24 17:12:48 +0100 |
---|---|---|
committer | Alberto Sartori <alberto.sartori@huawei.com> | 2022-03-25 09:25:05 +0100 |
commit | 7d70ea28d8ab335928343fb10ad136c9efe4a69e (patch) | |
tree | ff22ceb4833028fd5b18e9f87c7f7968bc4cdccd | |
parent | 2262244e6d1f07659195e523d3fa279775d44c57 (diff) | |
download | justbuild-7d70ea28d8ab335928343fb10ad136c9efe4a69e.tar.gz |
don't check bounds twice
-rw-r--r-- | src/buildtool/build_engine/expression/expression.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/buildtool/build_engine/expression/expression.cpp b/src/buildtool/build_engine/expression/expression.cpp index f9e43463..679ebcbd 100644 --- a/src/buildtool/build_engine/expression/expression.cpp +++ b/src/buildtool/build_engine/expression/expression.cpp @@ -42,7 +42,7 @@ auto Expression::operator[](ExpressionPtr const& key) && -> ExpressionPtr { auto Expression::operator[](size_t pos) const& -> ExpressionPtr const& { if (pos < List().size()) { - return List().at(pos); + return List()[pos]; } throw ExpressionTypeError{ fmt::format("List pos '{}' is out of bounds.", pos)}; @@ -51,7 +51,7 @@ auto Expression::operator[](size_t pos) const& -> ExpressionPtr const& { auto Expression::operator[](size_t pos) && -> ExpressionPtr { auto&& list = std::move(*this).List(); if (pos < list.size()) { - return list.at(pos); + return list[pos]; } throw ExpressionTypeError{ fmt::format("List pos '{}' is out of bounds.", pos)}; |