cppbind: simplify with 'auto'

This commit is contained in:
Thomas Geymayer
2018-01-21 14:39:24 +01:00
parent af5833cbc5
commit db3725880e
+14 -21
View File
@@ -408,17 +408,14 @@ namespace nasal
// Replace any getter that is not available in the current class. // Replace any getter that is not available in the current class.
// TODO check if this is the correct behavior of function overriding // TODO check if this is the correct behavior of function overriding
for( typename BaseGhost::MemberMap::const_iterator member = for(auto const& base_member: base->_members)
base->_members.begin();
member != base->_members.end();
++member )
{ {
if( _members.find(member->first) == _members.end() ) if( _members.find(base_member.first) == _members.end() )
_members[member->first] = member_t _members[base_member.first] = member_t
( (
member->second.getter, base_member.second.getter,
member->second.setter, base_member.second.setter,
member->second.func base_member.second.func
); );
} }
@@ -885,10 +882,9 @@ namespace nasal
} }
// otherwise try the derived classes // otherwise try the derived classes
for( typename DerivedList::reverse_iterator for( auto derived = getSingletonPtr()->_derived_types.rbegin();
derived = getSingletonPtr()->_derived_types.rbegin(); derived != getSingletonPtr()->_derived_types.rend();
derived != getSingletonPtr()->_derived_types.rend(); ++derived )
++derived )
{ {
strong_ref ref = (derived->from_nasal)(c, me); strong_ref ref = (derived->from_nasal)(c, me);
@@ -1070,10 +1066,9 @@ namespace nasal
static_pointer_cast<typename Ghost::raw_type>(base_ref); static_pointer_cast<typename Ghost::raw_type>(base_ref);
// Now check if we can further downcast to one of our derived classes. // Now check if we can further downcast to one of our derived classes.
for( typename DerivedList::reverse_iterator for( auto derived = getSingletonPtr()->_derived_types.rbegin();
derived = getSingletonPtr()->_derived_types.rbegin(); derived != getSingletonPtr()->_derived_types.rend();
derived != getSingletonPtr()->_derived_types.rend(); ++derived )
++derived )
{ {
naRef ghost = (derived->to_nasal)(c, ref, strong); naRef ghost = (derived->to_nasal)(c, ref, strong);
@@ -1353,9 +1348,7 @@ namespace nasal
// return ""; // return "";
// } // }
typename MemberMap::iterator member = auto member = getSingletonPtr()->_members.find(key_str);
getSingletonPtr()->_members.find(key_str);
if( member == getSingletonPtr()->_members.end() ) if( member == getSingletonPtr()->_members.end() )
{ {
fallback_getter_t fallback_get = getSingletonPtr()->_fallback_getter; fallback_getter_t fallback_get = getSingletonPtr()->_fallback_getter;
@@ -1405,7 +1398,7 @@ namespace nasal
const std::string key = nasal::from_nasal<std::string>(c, field); const std::string key = nasal::from_nasal<std::string>(c, field);
const MemberMap& members = getSingletonPtr()->_members; const MemberMap& members = getSingletonPtr()->_members;
typename MemberMap::const_iterator member = members.find(key); auto member = members.find(key);
if( member == members.end() ) if( member == members.end() )
{ {
fallback_setter_t fallback_set = getSingletonPtr()->_fallback_setter; fallback_setter_t fallback_set = getSingletonPtr()->_fallback_setter;