diff options
author | Rémi Verschelde <rverschelde@gmail.com> | 2017-07-05 10:55:11 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-07-05 10:55:11 +0200 |
commit | 6f63a01302355077af2459c96a17e299c32b2960 (patch) | |
tree | c0c6ed6e20f8dc26b436e9c193a962dbd09ffcb7 /core/vector.h | |
parent | 5a48b428fd349411456b785a5cb9a6ee0d5b1506 (diff) | |
parent | 211c4518903d82068c061943064824ac5595fd38 (diff) |
Merge pull request #8943 from RandomShaper/fix-error-handling
Implement well-defined handling of unrecoverable errors
Diffstat (limited to 'core/vector.h')
-rw-r--r-- | core/vector.h | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/core/vector.h b/core/vector.h index fe1c1b05dd..5eed8dce96 100644 --- a/core/vector.h +++ b/core/vector.h @@ -134,10 +134,7 @@ public: inline T &operator[](int p_index) { - if (p_index < 0 || p_index >= size()) { - T &aux = *((T *)0); //nullreturn - ERR_FAIL_COND_V(p_index < 0 || p_index >= size(), aux); - } + CRASH_BAD_INDEX(p_index, size()); _copy_on_write(); // wants to write, so copy on write. @@ -146,10 +143,8 @@ public: inline const T &operator[](int p_index) const { - if (p_index < 0 || p_index >= size()) { - const T &aux = *((T *)0); //nullreturn - ERR_FAIL_COND_V(p_index < 0 || p_index >= size(), aux); - } + CRASH_BAD_INDEX(p_index, size()); + // no cow needed, since it's reading return _get_data()[p_index]; } |