BlackMATov/meta.hpp
1
0
Fork 0
mirror of https://github.com/BlackMATov/meta.hpp.git synced 2025-12-14 11:40:35 +07:00
Code Issues Packages Projects Releases Wiki Activity

fix some potential problem in the type_registry::for_each_type function

Browse Source
This commit is contained in:
BlackMATov
2023-12-29 23:31:22 +07:00
parent f248035c30
commit e2b295784b
4 changed files with 11 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
ROADMAP.md
View File

@@ -6,6 +6,7 @@
- fix all includes to work with the library more flexible
- test and support shared libraries
- add debug type names
- auto register destructor without class_<T>
## Version 1.0

8
develop/singles/headers/meta.hpp/meta_all.hpp
View File

@@ -4050,8 +4050,10 @@ namespace meta_hpp::detail
void for_each_type(F&& f) const {
const locker lock;
for ( const any_type& type : types_ ) {
std::invoke(f, type);
// we use an index based for loop to avoid the iterator invalidation issues
// that can happen when adding a new type inside the loop
for ( std::size_t i{}; i < types_.size(); ++i ) {
std::invoke(f, types_[i]);
}
}
@@ -4259,7 +4261,7 @@ namespace meta_hpp::detail
void for_each_scope(F&& f) const {
const locker lock;
for ( auto&& [name, scope] : scopes_ ) {
for ( auto&& [_, scope] : scopes_ ) {
std::invoke(f, scope);
}
}

2
headers/meta.hpp/meta_detail/state_registry.hpp
View File

@@ -41,7 +41,7 @@ namespace meta_hpp::detail
void for_each_scope(F&& f) const {
const locker lock;
for ( auto&& [name, scope] : scopes_ ) {
for ( auto&& [_, scope] : scopes_ ) {
std::invoke(f, scope);
}
}

6
headers/meta.hpp/meta_detail/type_registry.hpp
View File

@@ -41,8 +41,10 @@ namespace meta_hpp::detail
void for_each_type(F&& f) const {
const locker lock;
for ( const any_type& type : types_ ) {
std::invoke(f, type);
// we use an index based for loop to avoid the iterator invalidation issues
// that can happen when adding a new type inside the loop
for ( std::size_t i{}; i < types_.size(); ++i ) {
std::invoke(f, types_[i]);
}
}