diff options
author | A Thousand Ships <96648715+AThousandShips@users.noreply.github.com> | 2024-01-19 13:21:39 +0100 |
---|---|---|
committer | A Thousand Ships <96648715+AThousandShips@users.noreply.github.com> | 2024-02-09 12:50:15 +0100 |
commit | 684752e75bdeb58727c2d9b0ff0265d7fcd47de0 (patch) | |
tree | 4fc57e9d0738021b8b31699a6339275347e38ec0 /core/variant/array.cpp | |
parent | 94dbf69f5d6b7d2fd9561692df2e71557607fddc (diff) | |
download | redot-engine-684752e75bdeb58727c2d9b0ff0265d7fcd47de0.tar.gz |
Replace error checks against `size` with `is_empty`
Diffstat (limited to 'core/variant/array.cpp')
-rw-r--r-- | core/variant/array.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/core/variant/array.cpp b/core/variant/array.cpp index ab0315ae34..5d6fbb8bed 100644 --- a/core/variant/array.cpp +++ b/core/variant/array.cpp @@ -316,17 +316,17 @@ void Array::erase(const Variant &p_value) { } Variant Array::front() const { - ERR_FAIL_COND_V_MSG(_p->array.size() == 0, Variant(), "Can't take value from empty array."); + ERR_FAIL_COND_V_MSG(_p->array.is_empty(), Variant(), "Can't take value from empty array."); return operator[](0); } Variant Array::back() const { - ERR_FAIL_COND_V_MSG(_p->array.size() == 0, Variant(), "Can't take value from empty array."); + ERR_FAIL_COND_V_MSG(_p->array.is_empty(), Variant(), "Can't take value from empty array."); return operator[](_p->array.size() - 1); } Variant Array::pick_random() const { - ERR_FAIL_COND_V_MSG(_p->array.size() == 0, Variant(), "Can't take value from empty array."); + ERR_FAIL_COND_V_MSG(_p->array.is_empty(), Variant(), "Can't take value from empty array."); return operator[](Math::rand() % _p->array.size()); } |