diff --git a/DataControlPlugin.cc b/DataControlPlugin.cc index 5f681b72639a09cca8fba3e97f2e24fe8cc9c88a..7d5d028cc2ddbddee222cfafd71dc8ee4653d736 100644 --- a/DataControlPlugin.cc +++ b/DataControlPlugin.cc @@ -863,7 +863,7 @@ void DataControlPlugin::propagateDownwards(BaseObject* _obj, int _column ){ */ void DataControlPlugin::slotBoundingBoxChange( ) { - for (PluginFunctions::ObjectIterator o_it; o_it != PluginFunctions::objectsEnd(); ++o_it) { + for (auto* o_it : PluginFunctions::objects()) { updateBoundingBox (o_it); } diff --git a/DataControlPluginScripting.cc b/DataControlPluginScripting.cc index 501720e9d3a43c92769e0798f5066ad11022e7a5..6b760eebdcdcd0b1d3c8c50031abb68e1c3731bc 100644 --- a/DataControlPluginScripting.cc +++ b/DataControlPluginScripting.cc @@ -454,8 +454,7 @@ void DataControlPlugin::setAllTarget() { if ( action != 0 && action->data().isValid() ) type = (DataType) action->data().toUInt(); - for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::ALL_OBJECTS, type) ; - o_it != PluginFunctions::objectsEnd(); ++o_it){ + for (auto* o_it : PluginFunctions::objects(PluginFunctions::ALL_OBJECTS, type) ) { o_it->target(true); } } @@ -476,8 +475,7 @@ void DataControlPlugin::setAllSource() { if ( action != 0 && action->data().isValid() ) type = (DataType) action->data().toUInt(); - for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::ALL_OBJECTS, type) ; - o_it != PluginFunctions::objectsEnd(); ++o_it){ + for (auto* o_it : PluginFunctions::objects(PluginFunctions::ALL_OBJECTS, type) ) { o_it->source(true); } } @@ -498,8 +496,7 @@ void DataControlPlugin::clearAllTarget() { if ( action != 0 && action->data().isValid() ) type = (DataType) action->data().toUInt(); - for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::ALL_OBJECTS, type) ; - o_it != PluginFunctions::objectsEnd(); ++o_it){ + for (auto* o_it : PluginFunctions::objects(PluginFunctions::ALL_OBJECTS, type) ) { o_it->target(false); } } @@ -520,8 +517,7 @@ void DataControlPlugin::clearAllSource() { if ( action != 0 && action->data().isValid() ) type = (DataType) action->data().toUInt(); - for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::ALL_OBJECTS, type) ; - o_it != PluginFunctions::objectsEnd(); ++o_it){ + for (auto* o_it : PluginFunctions::objects(PluginFunctions::ALL_OBJECTS, type) ) { o_it->source(false); } } @@ -545,8 +541,7 @@ void DataControlPlugin::hideAll() { if ( action != 0 && action->data().isValid() ) type = (DataType) action->data().toUInt(); - for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::ALL_OBJECTS,type) ; - o_it != PluginFunctions::objectsEnd(); ++o_it){ + for (auto* o_it : PluginFunctions::objects(PluginFunctions::ALL_OBJECTS, type) ) { o_it->hide(); } @@ -571,7 +566,7 @@ void DataControlPlugin::showAll() { if ( action != 0 && action->data().isValid() ) type = (DataType) action->data().toUInt(); - for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::ALL_OBJECTS, type); o_it != PluginFunctions::objectsEnd(); ++o_it){ + for (auto* o_it : PluginFunctions::objects(PluginFunctions::ALL_OBJECTS, type) ) { o_it->show(); } @@ -586,8 +581,9 @@ IdList DataControlPlugin::getTargetObjects(DataType _type) { IdList list; - for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::TARGET_OBJECTS, _type); o_it != PluginFunctions::objectsEnd(); ++o_it) + for (auto* o_it : PluginFunctions::objects(PluginFunctions::TARGET_OBJECTS, _type) ) { list.push_back( o_it->id() ); + } return list; } @@ -601,8 +597,9 @@ IdList DataControlPlugin::getSourceObjects(DataType _type) { IdList list; - for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::SOURCE_OBJECTS, _type); o_it != PluginFunctions::objectsEnd(); ++o_it) + for (auto* o_it : PluginFunctions::objects(PluginFunctions::SOURCE_OBJECTS, _type) ) { list.push_back( o_it->id() ); + } return list; } @@ -614,8 +611,9 @@ IdList DataControlPlugin::getSourceObjects(DataType _type) { */ void DataControlPlugin::printObjectInfoToLog() { - for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::ALL_OBJECTS); o_it != PluginFunctions::objectsEnd(); ++o_it) + for (auto* o_it : PluginFunctions::objects(PluginFunctions::ALL_OBJECTS) ) { emit log(LOGINFO, tr("Object \"%1\" with ID %2 of type %3 ").arg(o_it->name()).arg(o_it->id()).arg(o_it->dataType().name())); + } } //****************************************************************************** @@ -628,8 +626,8 @@ void DataControlPlugin::printObjectInfoToLog() { */ unsigned int DataControlPlugin::groupCount() const { unsigned int count = 0; - for ( PluginFunctions::BaseObjectIterator o_it(PluginFunctions::ALL_OBJECTS,DATA_GROUP); o_it != PluginFunctions::objectsEnd(); ++o_it) - ++count; + for (auto* o_it : PluginFunctions::objects(PluginFunctions::ALL_OBJECTS, DATA_GROUP) ) + ++count; return count; } diff --git a/Popup.cc b/Popup.cc index eca63d54887726a06f4e3963493e3b83ef925c6c..352ea0d11e32d6c6cedc201c9c47efd8e02f22eb 100644 --- a/Popup.cc +++ b/Popup.cc @@ -321,8 +321,7 @@ void DataControlPlugin::slotHeaderCustomContextMenuRequested ( const QPoint & _p //get all used types QVector<DataType> types; - for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::ALL_OBJECTS) ; - o_it != PluginFunctions::objectsEnd(); ++o_it) + for (auto* o_it : PluginFunctions::objects(PluginFunctions::ALL_OBJECTS) ) if ( !types.contains( o_it->dataType() ) ) types.push_back( o_it->dataType() ); @@ -478,8 +477,7 @@ void DataControlPlugin::slotCopyMaterialToTargeted() { BaseObjectData* itemData = dynamic_cast< BaseObjectData* > (item); const ACG::SceneGraph::Material &sourceMaterial = itemData->materialNode()->material(); - for (PluginFunctions::ObjectIterator o_it(PluginFunctions::TARGET_OBJECTS); - o_it != PluginFunctions::objectsEnd(); ++o_it) { + for (auto* o_it : PluginFunctions::objects(PluginFunctions::TARGET_OBJECTS) ) { MaterialNode * const materialNode = o_it->materialNode(); if (materialNode) {