summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlberto Sartori <alberto.sartori@huawei.com>2022-03-24 17:12:48 +0100
committerAlberto Sartori <alberto.sartori@huawei.com>2022-03-25 09:25:05 +0100
commit7d70ea28d8ab335928343fb10ad136c9efe4a69e (patch)
treeff22ceb4833028fd5b18e9f87c7f7968bc4cdccd
parent2262244e6d1f07659195e523d3fa279775d44c57 (diff)
downloadjustbuild-7d70ea28d8ab335928343fb10ad136c9efe4a69e.tar.gz
don't check bounds twice
-rw-r--r--src/buildtool/build_engine/expression/expression.cpp4
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)};