Reused some br::Object code instead of looping over parameters myself

Scott Klum
1 parent 324204b5
Showing 2 changed files with 4 additions and 14 deletions
openbr/openbr_plugin.cpp
openbr/plugins/openbr_internal.h
@@ -550,20 +550,18 @@ QStringList Object::parameters() const
  
     for (int i = firstAvailablePropertyIdx; i < metaObject()->propertyCount();i++) {
         QMetaProperty property = metaObject()->property(i);
-        if (property.isStored(this)) continue;
         parameters.append(QString("%1 %2 = %3").arg(property.typeName(), property.name(), property.read(this).toString()));
     }
+
     return parameters;
 }
  
 QStringList Object::arguments() const
 {
     QStringList arguments;
-    for (int i=metaObject()->propertyOffset(); i<metaObject()->propertyCount(); i++) {
-        QMetaProperty property = metaObject()->property(i);
-        if (property.isStored(this)) continue;
+    for (int i=metaObject()->propertyOffset(); i<metaObject()->propertyCount(); i++)
         arguments.append(argument(i));
-    }
+
     return arguments;
 }
  
@@ -266,15 +266,7 @@ public:
         name.replace("Transform","");
         name += "([],";
  
-        QStringList parameters;
-
-        // Skip name and transforms (handled below)
-        for (int i = 2; i < metaObject()->propertyCount();i++) {
-            QMetaProperty property = metaObject()->property(i);
-            parameters.append(property.read(this).toString());
-        }
-
-        name += parameters.join(",");
+        name += this->arguments().join(",");
  
         name += ")";
         name.replace("br::","");