Skip to content

Commit 3118d41

Browse files
committed
Added iterator support to Array
1 parent a9ac668 commit 3118d41

File tree

2 files changed

+92
-0
lines changed

2 files changed

+92
-0
lines changed

binding_generator.py

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -909,6 +909,47 @@ def generate_builtin_class_header(builtin_api, size, used_classes, fully_used_cl
909909
result.append("\tvoid _ref(const Array &p_from) const;")
910910
result.append("\tconst Variant *ptr() const;")
911911
result.append("\tVariant *ptrw();")
912+
result.append("""
913+
struct Iterator {
914+
_FORCE_INLINE_ Variant &operator*() const;
915+
_FORCE_INLINE_ Variant *operator->() const;
916+
_FORCE_INLINE_ Iterator &operator++();
917+
_FORCE_INLINE_ Iterator &operator--();
918+
919+
_FORCE_INLINE_ bool operator==(const Iterator &b) const { return elem_ptr == b.elem_ptr; }
920+
_FORCE_INLINE_ bool operator!=(const Iterator &b) const { return elem_ptr != b.elem_ptr; }
921+
922+
Iterator(Variant *p_ptr) { elem_ptr = p_ptr; }
923+
Iterator() {}
924+
Iterator(const Iterator &p_it) { elem_ptr = p_it.elem_ptr; }
925+
926+
private:
927+
Variant *elem_ptr = nullptr;
928+
};
929+
930+
struct ConstIterator {
931+
_FORCE_INLINE_ const Variant &operator*() const;
932+
_FORCE_INLINE_ const Variant *operator->() const;
933+
_FORCE_INLINE_ ConstIterator &operator++();
934+
_FORCE_INLINE_ ConstIterator &operator--();
935+
936+
_FORCE_INLINE_ bool operator==(const ConstIterator &b) const { return elem_ptr == b.elem_ptr; }
937+
_FORCE_INLINE_ bool operator!=(const ConstIterator &b) const { return elem_ptr != b.elem_ptr; }
938+
939+
ConstIterator(const Variant *p_ptr) { elem_ptr = p_ptr; }
940+
ConstIterator() {}
941+
ConstIterator(const ConstIterator &p_it) { elem_ptr = p_it.elem_ptr; }
942+
943+
private:
944+
const Variant *elem_ptr = nullptr;
945+
};
946+
947+
_FORCE_INLINE_ Iterator begin();
948+
_FORCE_INLINE_ Iterator end();
949+
950+
_FORCE_INLINE_ ConstIterator begin() const;
951+
_FORCE_INLINE_ ConstIterator end() const;
952+
""")
912953

913954
if class_name == "Dictionary":
914955
result.append("\tconst Variant &operator[](const Variant &p_key) const;")

include/godot_cpp/variant/variant.hpp

Lines changed: 51 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232

3333
#include <godot_cpp/core/defs.hpp>
3434

35+
#include <godot_cpp/variant/array.hpp>
3536
#include <godot_cpp/variant/builtin_types.hpp>
3637
#include <godot_cpp/variant/variant_size.hpp>
3738

@@ -357,6 +358,56 @@ String vformat(const String &p_text, const VarArgs... p_args) {
357358
return p_text % args_array;
358359
}
359360

361+
Variant &Array::Iterator::operator*() const {
362+
return *elem_ptr;
363+
}
364+
365+
Variant *Array::Iterator::operator->() const {
366+
return elem_ptr;
367+
}
368+
369+
Array::Iterator &Array::Iterator::operator++() {
370+
elem_ptr++;
371+
return *this;
372+
}
373+
374+
Array::Iterator &Array::Iterator::operator--() {
375+
elem_ptr--;
376+
return *this;
377+
}
378+
379+
const Variant &Array::ConstIterator::operator*() const {
380+
return *elem_ptr;
381+
}
382+
383+
const Variant *Array::ConstIterator::operator->() const {
384+
return elem_ptr;
385+
}
386+
387+
Array::ConstIterator &Array::ConstIterator::operator++() {
388+
elem_ptr++;
389+
return *this;
390+
}
391+
392+
Array::ConstIterator &Array::ConstIterator::operator--() {
393+
elem_ptr--;
394+
return *this;
395+
}
396+
397+
Array::Iterator Array::begin() {
398+
return Array::Iterator(ptrw());
399+
}
400+
Array::Iterator Array::end() {
401+
return Array::Iterator(ptrw() + size());
402+
}
403+
404+
Array::ConstIterator Array::begin() const {
405+
return Array::ConstIterator(ptr());
406+
}
407+
Array::ConstIterator Array::end() const {
408+
return Array::ConstIterator(ptr() + size());
409+
}
410+
360411
#include <godot_cpp/variant/builtin_vararg_methods.hpp>
361412

362413
#ifdef REAL_T_IS_DOUBLE

0 commit comments

Comments
 (0)