various fixes:

+ fix typos
+ fix const correctness
+ whitespace improvements
This commit is contained in:
wmayer
2017-06-19 17:18:16 +02:00
parent fe8e69f122
commit 90653787a6
14 changed files with 33 additions and 34 deletions

View File

@@ -88,7 +88,7 @@ std::vector< DocumentObject* > GroupExtension::addObjects(std::vector< DocumentO
auto geogrp = GeoFeatureGroupExtension::getGroupOfObject(getExtendedObject());
auto objgrp = GeoFeatureGroupExtension::getGroupOfObject(obj);
if( geogrp != objgrp ) {
//what to doo depends on if we are in geofeature group or not
//what to do depends on if we are in geofeature group or not
if(geogrp)
geogrp->getExtensionByType<GeoFeatureGroupExtension>()->addObject(obj);
else
@@ -133,7 +133,6 @@ std::vector< DocumentObject* > GroupExtension::removeObjects(std::vector< Docume
return removed;
}
void GroupExtension::removeObjectsFromDocument()
{
const std::vector<DocumentObject*> & grp = Group.getValues();
@@ -233,7 +232,7 @@ int GroupExtension::countObjectsOfType(const Base::Type& typeId) const
DocumentObject* GroupExtension::getGroupOfObject(const DocumentObject* obj)
{
//note that we return here only Groups, but nothing derived from it, e.g. no GeoFeatureGroups.
//That is important as there are clear differences between groups/geofeature groups (e.g. a object
//That is important as there are clear differences between groups/geofeature groups (e.g. an object
//can be in only one group, and only one geofeaturegroup, however, it can be in both at the same time)
auto list = obj->getInList();
for (auto obj : list) {