Merge pull request #214 from biometrics/recursive_property

Implement a method for setting properties after objects have been created

Merge pull request #214 from biometrics/recursive_property
Implement a method for setting properties after objects have been created
Josh Klontz
2 parents 6cdd5961 d15634d6
Showing 15 changed files with 230 additions and 171 deletions
app/br/br.cpp
openbr/core/core.cpp
openbr/core/eval.cpp
openbr/openbr_plugin.cpp
openbr/openbr_plugin.h
openbr/plugins/draw.cpp
openbr/plugins/gallery.cpp
openbr/plugins/gui.cpp
openbr/plugins/independent.cpp
openbr/plugins/meta.cpp
openbr/plugins/misc.cpp
openbr/plugins/openbr_internal.h
openbr/plugins/pp5.cpp
openbr/plugins/slidingwindow.cpp
openbr/plugins/stream.cpp
@@ -178,7 +178,7 @@ public:
             } else if (!strcmp(fun, "objects")) {
                 check(parc <= 2, "Incorrect parameter count for 'objects'.");
                 int size = br_objects(NULL, 0, parc >= 1 ? parv[0] : ".*", parc >= 2 ? parv[1] : ".*");
-                char * temp = new char[size];
+                char *temp = new char[size];
                 br_objects(temp, size, parc >= 1 ? parv[0] : ".*", parc >= 2 ? parv[1] : ".*");
                 printf("%s\n", temp);
                 delete [] temp;
@@ -49,7 +49,7 @@ struct AlgorithmCore
         QScopedPointer<Transform> trainingWrapper(Transform::make("DirectStream(readMode=DistributeFrames)", NULL));
-        CompositeTransform * downcast = dynamic_cast<CompositeTransform *>(trainingWrapper.data());
+        CompositeTransform *downcast = dynamic_cast<CompositeTransform *>(trainingWrapper.data());
         if (downcast == NULL)
             qFatal("downcast failed?");
         downcast->transforms.append(this->transform.data());
@@ -144,7 +144,7 @@ struct AlgorithmCore
             fileExclusion = true;
         }
-        Gallery * temp = Gallery::make(input);
+        Gallery *temp = Gallery::make(input);
         qint64 total = temp->totalSize();
         Globals->currentStep = 0;
@@ -156,13 +156,13 @@ struct AlgorithmCore
         if (!multiProcess) {
             basePipe.reset(Transform::make(pipeDesc,NULL));
-            CompositeTransform * downcast = dynamic_cast<CompositeTransform *>(basePipe.data());
+            CompositeTransform *downcast = dynamic_cast<CompositeTransform *>(basePipe.data());
             if (downcast == NULL) qFatal("downcast failed?");
             downcast->transforms.prepend(this->transform.data());
             if (fileExclusion) {
-                Transform * temp = Transform::make("FileExclusion(" + gallery.flat() + ")", downcast);
+                Transform *temp = Transform::make("FileExclusion(" + gallery.flat() + ")", downcast);
                 downcast->transforms.prepend(temp);
             }
@@ -180,7 +180,7 @@ struct AlgorithmCore
         // Next, we make a Stream (with placeholder transform)
         QString streamDesc = "Stream(readMode=StreamGallery)";
         QScopedPointer<Transform> baseStream(Transform::make(streamDesc, NULL));
-        WrapperTransform * wrapper = dynamic_cast<WrapperTransform *> (baseStream.data());
+        WrapperTransform *wrapper = dynamic_cast<WrapperTransform *> (baseStream.data());
         // replace that placeholder with the pipe we built
         wrapper->transform = basePipe.data();
@@ -373,7 +373,7 @@ struct AlgorithmCore
         File colGallery = targetGallery;
         qint64 rowSize;
-        Gallery * temp;
+        Gallery *temp;
         if (transposeMode)
         {
             rowGallery = targetGallery;
@@ -456,7 +456,7 @@ struct AlgorithmCore
             {
                 compareRegionDesc = compareRegionDesc;
                 compareRegion.reset(Transform::make(compareRegionDesc,NULL));
-                CompositeTransform * downcast = dynamic_cast<CompositeTransform *> (compareRegion.data());
+                CompositeTransform *downcast = dynamic_cast<CompositeTransform *> (compareRegion.data());
                 if (downcast == NULL)
                     qFatal("Pipe downcast failed in compare");
@@ -496,7 +496,7 @@ struct AlgorithmCore
         // Assign the comparison transform we previously built, and the output transform  we just built to
         // two stages of a pipe.
-        CompositeTransform * downcast = dynamic_cast<CompositeTransform *> (join.data());
+        CompositeTransform *downcast = dynamic_cast<CompositeTransform *> (join.data());
         downcast->transforms.append(compareRegion.data());
         downcast->transforms.append(outputTform.data());
@@ -508,7 +508,7 @@ struct AlgorithmCore
         // and pass the transforms it reads through the base algorithm.
         QString streamDesc = "Stream(readMode=StreamGallery)";
         QScopedPointer<Transform> streamBase(Transform::make(streamDesc, NULL));
-        WrapperTransform * streamWrapper = dynamic_cast<WrapperTransform *> (streamBase.data());
+        WrapperTransform *streamWrapper = dynamic_cast<WrapperTransform *> (streamBase.data());
         streamWrapper->transform = join.data();
         // The transform we will use is now complete.
@@ -687,7 +687,7 @@ void br::Convert(const File &amp;fileType, const File &amp;inputFile, const File &amp;output
         if (targetFiles.size() != m.cols)
         {
-            MatrixOutput   * mOut = dynamic_cast<MatrixOutput *>(o.data());
+            MatrixOutput *mOut = dynamic_cast<MatrixOutput *>(o.data());
             if (mOut)
                 mOut->data.create(queryFiles.size(), 1, CV_32FC1);
         }
@@ -730,7 +730,7 @@ QSharedPointer&lt;br::Transform&gt; br::Transform::fromAlgorithm(const QString &amp;algori
     else {
         QSharedPointer<Transform> orig_tform = AlgorithmManager::getAlgorithm(algorithm)->transform;
         QSharedPointer<Transform> newRoot = QSharedPointer<Transform>(Transform::make("Stream(readMode=DistributeFrames)", NULL));
-        WrapperTransform * downcast = dynamic_cast<WrapperTransform *> (newRoot.data());
+        WrapperTransform *downcast = dynamic_cast<WrapperTransform *> (newRoot.data());
         downcast->transform = orig_tform.data();
         downcast->init();
         return newRoot;
@@ -68,7 +68,7 @@ static float getTAR(const QList&lt;OperatingPoint&gt; &amp;operatingPoints, float FAR)
 // Decide whether to construct a normal mask matrix, or a pairwise mask by comparing the dimensions of
 // scores with the size of the target and query lists
-static cv::Mat constructMatchingMask(const cv::Mat & scores, const FileList & target, const FileList & query, int partition=0)
+static cv::Mat constructMatchingMask(const cv::Mat &scores, const FileList &target, const FileList &query, int partition=0)
 {
     // If the dimensions of the score matrix match the sizes of the target and query lists, construct a normal mask matrix
     if (target.size() == scores.cols && query.size() == scores.rows)
@@ -288,7 +288,7 @@ struct GenImpCounts
     qint64 impCount;
 };
-float InplaceEval(const QString & simmat, const QString & target, const QString & query, const QString & csv)
+float InplaceEval(const QString &simmat, const QString &target, const QString &query, const QString &csv)
 {
     qDebug("Evaluating %s%s%s",
             qPrintable(simmat),
@@ -517,7 +517,7 @@ TemplateList TemplateList::relabel(const TemplateList &amp;tl, const QString &amp;propNa
     return result;
 }
-QList<int> TemplateList::indexProperty(const QString & propName, QHash<QString, int> * valueMap,QHash<int, QVariant> * reverseLookup) const
+QList<int> TemplateList::indexProperty(const QString &propName, QHash<QString, int> * valueMap,QHash<int, QVariant> * reverseLookup) const
 {
     QHash<QString, int> dummyForwards;
     QHash<int, QVariant> dummyBackwards;
@@ -528,13 +528,13 @@ QList&lt;int&gt; TemplateList::indexProperty(const QString &amp; propName, QHash&lt;QString, 
     return indexProperty(propName, *valueMap, *reverseLookup);
 }
-QList<int> TemplateList::indexProperty(const QString & propName, QHash<QString, int> & valueMap, QHash<int, QVariant> & reverseLookup) const
+QList<int> TemplateList::indexProperty(const QString &propName, QHash<QString, int> &valueMap, QHash<int, QVariant> &reverseLookup) const
 {
     valueMap.clear();
     reverseLookup.clear();
     const QList<QVariant> originalLabels = File::values(*this, propName);
-    foreach (const QVariant & label, originalLabels) {
+    foreach (const QVariant &label, originalLabels) {
         QString labelString = label.toString();
         if (!valueMap.contains(labelString)) {
             reverseLookup.insert(valueMap.size(), label);
@@ -717,6 +717,16 @@ void Object::load(QDataStream &amp;stream)
     init();
 }
+bool Object::setPropertyRecursive(const QString &name, QVariant value)
+{
+    if (this->metaObject()->indexOfProperty(qPrintable(name)) == -1)
+        return false;
+
+    setProperty(name, value);
+    init();
+    return true;
+}
+
 void Object::setProperty(const QString &name, QVariant value)
 {
     QString type;
@@ -1300,8 +1310,8 @@ QList&lt;Transform *&gt; Transform::getChildren() const
 {
     QList<Transform *> output;
     for (int i=0; i < metaObject()->propertyCount(); i++) {
-        const char * prop_name = metaObject()->property(i).name();
-        const QVariant & variant = this->property(prop_name);
+        const char *prop_name = metaObject()->property(i).name();
+        const QVariant &variant = this->property(prop_name);
         if (variant.canConvert<Transform *>())
             output.append(variant.value<Transform *>());
@@ -1311,11 +1321,11 @@ QList&lt;Transform *&gt; Transform::getChildren() const
     return output;
 }
-TemplateEvent * Transform::getEvent(const QString & name)
+TemplateEvent *Transform::getEvent(const QString &name)
 {
-    foreach(Transform * child, getChildren())
+    foreach(Transform *child, getChildren())
     {
-        TemplateEvent * probe = child->getEvent(name);
+        TemplateEvent *probe = child->getEvent(name);
         if (probe)
             return probe;
     }
@@ -1337,7 +1347,7 @@ void Transform::train(const TemplateList &amp;data)
 void Transform::train(const QList<TemplateList> &data)
 {
     TemplateList combined;
-    foreach(const TemplateList & set, data) {
+    foreach(const TemplateList &set, data) {
         combined.append(set);
     }
     train(combined);
@@ -461,9 +461,9 @@ struct TemplateList : public QList&lt;Template&gt;
     /*!< \brief Ensure labels are in the range [0,numClasses-1]. */
     BR_EXPORT static TemplateList relabel(const TemplateList &tl, const QString &propName, bool preserveIntegers);
-    QList<int> indexProperty(const QString & propName, QHash<QString, int> * valueMap=NULL,QHash<int, QVariant> * reverseLookup = NULL) const;
-    QList<int> indexProperty(const QString & propName, QHash<QString, int> & valueMap, QHash<int, QVariant> & reverseLookup) const;
-    QList<int> applyIndex(const QString & propName, const QHash<QString, int> & valueMap) const;
+    QList<int> indexProperty(const QString &propName, QHash<QString, int> * valueMap=NULL,QHash<int, QVariant> * reverseLookup = NULL) const;
+    QList<int> indexProperty(const QString &propName, QHash<QString, int> &valueMap, QHash<int, QVariant> &reverseLookup) const;
+    QList<int> applyIndex(const QString &propName, const QHash<QString, int> &valueMap) const;
     /*!
      * \brief Returns the total number of bytes in all the templates.
@@ -539,7 +539,7 @@ struct TemplateList : public QList&lt;Template&gt;
      * \brief Returns the number of occurences for each label in the list.
      */
     template<typename T>
-    QMap<T,int> countValues(const QString & propName, bool excludeFailures = false) const
+    QMap<T,int> countValues(const QString &propName, bool excludeFailures = false) const
     {
         QMap<T, int> labelCounts;
         foreach (const File &file, files())
@@ -597,6 +597,8 @@ public:
     QString argument(int index) const; /*!< \brief A string value for the argument at the specified index. */
     QString description() const; /*!< \brief Returns a string description of the object. */
     void setProperty(const QString &name, QVariant value); /*!< \brief Overload of QObject::setProperty to handle OpenBR data types. */
+    virtual bool setPropertyRecursive(const QString &name, QVariant value); /*!< \brief Recursive version of setProperty, try to set the property on this object, or its children, returns true if successful. */
+
     static QStringList parse(const QString &string, char split = ','); /*!< \brief Splits the string while respecting lexical scoping of <tt>()</tt>, <tt>[]</tt>, <tt>\<\></tt>, and <tt>{}</tt>. */
 private:
@@ -1126,13 +1128,13 @@ class TemplateEvent : public QObject
     Q_OBJECT
 public:
-    void pulseSignal(const Template & output) const
+    void pulseSignal(const Template &output) const
     {
         emit theSignal(output);
     }
 signals:
-    void theSignal(const Template & output) const;
+    void theSignal(const Template &output) const;
 };
 /*!
@@ -1219,7 +1221,7 @@ public:
      * and the transform can emit a final set if templates if it wants. Time-invariant transforms
      * don't have to do anything.
      */
-    virtual void finalize(TemplateList & output) { output = TemplateList(); }
+    virtual void finalize(TemplateList &output) { output = TemplateList(); }
     /*!
      * \brief Does the transform require the non-const version of project? Can vary for aggregation type transforms
@@ -1256,14 +1258,14 @@ public:
      * and copy enough of their state that projectUpdate can safely be called on the original
      * instance, and the copy concurrently.
      */
-    virtual Transform * smartCopy(bool & newTransform) { newTransform=false; return this;}
+    virtual Transform *smartCopy(bool &newTransform) { newTransform=false; return this;}
-    virtual Transform * smartCopy() {bool junk; return smartCopy(junk);}
+    virtual Transform *smartCopy() {bool junk; return smartCopy(junk);}
     /*!
      * \brief Recursively retrieve a named event, returns NULL if an event is not found.
      */
-    virtual TemplateEvent * getEvent(const QString & name);
+    virtual TemplateEvent *getEvent(const QString &name);
     /*!
      * \brief Get a list of child transforms of this transform, child transforms are considered to be
@@ -163,7 +163,7 @@ class DrawPropertiesPointTransform : public UntrainableTransform
         const Scalar textColor(255, 255, 0);
         std::string outString = "";
-        foreach (const QString & propName, propNames)
+        foreach (const QString &propName, propNames)
         {
             QVariant prop = dst.file.value(propName);
@@ -725,7 +725,7 @@ class memGallery : public Gallery
 BR_REGISTER(Gallery, memGallery)
-FileList FileList::fromGallery(const File & file, bool cache)
+FileList FileList::fromGallery(const File &file, bool cache)
 {
     File targetMeta = file;
     targetMeta.name = targetMeta.path() + targetMeta.baseName() + "_meta" + targetMeta.hash() + ".mem";
@@ -1059,7 +1059,7 @@ class xmlGallery : public FileGallery
                     // a presentation!
                     if (signatureToken == QXmlStreamReader::StartElement && reader.name() == "presentation") {
                         templates.append(Template(File("",currentSignatureName)));
-                        foreach (const QXmlStreamAttribute & attribute, reader.attributes()) {
+                        foreach (const QXmlStreamAttribute &attribute, reader.attributes()) {
                             // file-name is stored directly on file, not as a key/value pair
                             if (attribute.name() == "file-name")
                                 templates.last().file.name = attribute.value().toString();
@@ -46,7 +46,7 @@ template&lt;typename T&gt;
 class ActualCreation : public NominalCreation
 {
 public:
-    T * basis;
+    T *basis;
     void creation()
     {
@@ -83,7 +83,7 @@ public:
     // the template with the NominalCreation interface, and call worker->creation
     // in the slot.
     template<typename T>
-    T * getItem()
+    T *getItem()
     {
         // If this is called by the main thread, we can just create the object
         // it's important to check, otherwise we will have problems trying to
@@ -103,12 +103,12 @@ public:
         emit needCreation();
         // collect the results, and return.
-        T * output = actualWorker->basis;
+        T *output = actualWorker->basis;
         delete actualWorker;
         return output;
     }
-    NominalCreation * worker;
+    NominalCreation *worker;
 signals:
     void needCreation();
@@ -135,7 +135,7 @@ protected:
 public:
-    DisplayWindow(QWidget * parent = NULL) : QLabel(parent)
+    DisplayWindow(QWidget *parent = NULL) : QLabel(parent)
     {
         setFixedSize(200,200);
         QApplication::instance()->installEventFilter(this);
@@ -146,7 +146,7 @@ public:
     }
 public slots:
-    void showImage(const QPixmap & input)
+    void showImage(const QPixmap &input)
     {
         pixmap = input;
@@ -161,7 +161,7 @@ public slots:
         setFixedSize(temp);
     }
-    bool eventFilter(QObject * obj, QEvent * event)
+    bool eventFilter(QObject *obj, QEvent *event)
     {
         if (event->type() == QEvent::KeyPress)
         {
@@ -292,7 +292,7 @@ public:
         } else {
             if (event->type() == QEvent::KeyPress)
             {
-                QKeyEvent * kevent = (QKeyEvent *) event;
+                QKeyEvent *kevent = (QKeyEvent *) event;
                 if (kevent->key() == Qt::Key_Enter || kevent->key() == Qt::Key_Return) {
                     event->accept();
                     return true;
@@ -315,7 +315,7 @@ public:
         return QList<QPointF>();
     }
-    void setKeys(const QStringList & keys)
+    void setKeys(const QStringList &keys)
     {
         promptKeys = keys;
     }
@@ -370,13 +370,13 @@ private:
 class PromptWindow : public DisplayWindow
 {
-    bool eventFilter(QObject * obj, QEvent * event)
+    bool eventFilter(QObject *obj, QEvent *event)
     {
         if (event->type() == QEvent::KeyPress)
         {
             event->accept();
-            QKeyEvent * key_event = dynamic_cast<QKeyEvent *> (event);
+            QKeyEvent *key_event = dynamic_cast<QKeyEvent *> (event);
             if (key_event == NULL) {
                 qDebug("failed to donwcast key event");
                 return true;
@@ -419,7 +419,7 @@ class GUIWindow : public QMainWindow
 public:
-    GUIWindow(QWidget * parent = NULL) : QMainWindow(parent)
+    GUIWindow(QWidget *parent = NULL) : QMainWindow(parent)
     {
         centralWidget = new QWidget();
         layout = new QHBoxLayout();
@@ -440,7 +440,7 @@ public:
     }
 public slots:
-    void showImage(const QPixmap & input)
+    void showImage(const QPixmap &input)
     {
         hide();
@@ -545,11 +545,11 @@ public:
         if (src.empty())
             return;
-        foreach (const Template & t, src) {
+        foreach (const Template &t, src) {
             // build label
             QString newTitle;
-            foreach (const QString & s, keys) {
+            foreach (const QString &s, keys) {
                 if (s.compare("name", Qt::CaseInsensitive) == 0) {
                     newTitle = newTitle + s + ": " + t.file.fileName() + " ";
                 } else if (t.file.contains(s)) {
@@ -578,7 +578,7 @@ public:
         }
     }
-    void finalize(TemplateList & output)
+    void finalize(TemplateList &output)
     {
         (void) output;
         emit hideWindow();
@@ -614,13 +614,13 @@ public:
 protected:
     MainThreadCreator creator;
-    DisplayWindow * window;
+    DisplayWindow *window;
     QImage qImageBuffer;
-    QPixmap * displayBuffer;
+    QPixmap *displayBuffer;
 signals:
-    void updateImage(const QPixmap & input);
-    void changeTitle(const QString & input);
+    void updateImage(const QPixmap &input);
+    void changeTitle(const QString &input);
     void hideWindow();
     void destroyWindow();
 };
@@ -772,7 +772,7 @@ public:
             }
         }
     }
-    RectMarkingWindow * trueWindow;
+    RectMarkingWindow *trueWindow;
     void init()
     {
         if (!Globals->useGui)
@@ -812,7 +812,7 @@ public:
     void project(const TemplateList &src, TemplateList &dst) const
     {
-        Transform * non_const = (ElicitTransform *) this;
+        Transform *non_const = (ElicitTransform *) this;
         non_const->projectUpdate(src,dst);
     }
@@ -910,7 +910,7 @@ public:
             }
         }
     }
-    PromptWindow * p_window;
+    PromptWindow *p_window;
     void init()
@@ -964,7 +964,7 @@ public:
         last_time = timer.elapsed();
     }
-    void finalize(TemplateList & output)
+    void finalize(TemplateList &output)
     {
         (void) output;
     }
@@ -1024,7 +1024,7 @@ public:
         }
     }
-    void finalize(TemplateList & output)
+    void finalize(TemplateList &output)
     {
         (void) output;
     }
@@ -9,7 +9,7 @@ using namespace cv;
 namespace br
 {
-static TemplateList Downsample(const TemplateList &templates, int classes, int instances, float fraction, const QString & inputVariable, const QStringList &gallery, const QStringList &subjects)
+static TemplateList Downsample(const TemplateList &templates, int classes, int instances, float fraction, const QString &inputVariable, const QStringList &gallery, const QStringList &subjects)
 {
     // Return early when no downsampling is required
     if ((classes == std::numeric_limits<int>::max()) &&
@@ -29,7 +29,7 @@ static TemplateList Downsample(const TemplateList &amp;templates, int classes, int i
     QMap<QString,int> counts = templates.countValues<QString>(inputVariable, instances != std::numeric_limits<int>::max());
     if ((instances != std::numeric_limits<int>::max()) && (classes != std::numeric_limits<int>::max()))
-        foreach (const QString & label, counts.keys())
+        foreach (const QString &label, counts.keys())
             if (counts[label] < instances)
                 counts.remove(label);
@@ -94,7 +94,7 @@ class DownsampleTrainingTransform : public Transform
     BR_PROPERTY(QStringList, subjects, QStringList())
-    void project(const Template & src, Template & dst) const
+    void project(const Template &src, Template &dst) const
     {
        transform->project(src,dst);      
     }
@@ -126,6 +126,21 @@ class IndependentTransform : public MetaTransform
     QList<Transform*> transforms;
+
+    bool setPropertyRecursive(const QString &name, QVariant value)
+    {
+        if (br::Object::setPropertyRecursive(name, value))
+            return true;
+
+        if (!transform->setPropertyRecursive(name, value))
+            return false;
+
+        for (int i=0;i < transforms.size();i++)
+            transforms[i]->setPropertyRecursive(name, value);
+
+        return true;
+    }
+
     void init()
     {
         transforms.clear();
@@ -149,7 +149,7 @@ class PipeTransform : public CompositeTransform
     // For time varying transforms, parallel execution over individual templates
     // won't work.
-    void projectUpdate(const TemplateList & src, TemplateList & dst)
+    void projectUpdate(const TemplateList &src, TemplateList &dst)
     {
         dst = src;
         foreach (Transform *f, transforms)
@@ -158,7 +158,7 @@ class PipeTransform : public CompositeTransform
         }
     }
-    virtual void finalize(TemplateList & output)
+    virtual void finalize(TemplateList &output)
     {
         output.clear();
         // For each transform,
@@ -185,7 +185,7 @@ class PipeTransform : public CompositeTransform
         QList<Transform *> flattened;
         for (int i=0;i < transforms.size(); i++)
         {
-            PipeTransform * probe = dynamic_cast<PipeTransform *> (transforms[i]);
+            PipeTransform *probe = dynamic_cast<PipeTransform *> (transforms[i]);
             if (!probe) {
                 flattened.append(transforms[i]);
                 continue;
@@ -212,7 +212,7 @@ protected:
     }
    // Single template const project, pass the template through each sub-transform, one after the other
-   virtual void _project(const Template & src, Template & dst) const
+   virtual void _project(const Template &src, Template &dst) const
    {
        dst = src;
        foreach (const Transform *f, transforms) {
@@ -247,7 +247,7 @@ class ExpandTransform : public UntrainableMetaTransform
         dst = Expanded(src);
     }
-    virtual void project(const Template & src, Template & dst) const
+    virtual void project(const Template &src, Template &dst) const
     {
         dst = src;
         qDebug("Called Expand project(Template,Template), nothing will happen");
@@ -272,11 +272,11 @@ class ContractTransform : public UntrainableMetaTransform
         if (src.empty()) return;
         Template out;
-        foreach (const Template & t, src) {
+        foreach (const Template &t, src) {
             out.merge(t);
         }
         out.file.clearRects();
-        foreach (const Template & t, src) {
+        foreach (const Template &t, src) {
             if (!t.file.rects().empty())
                 out.file.appendRects(t.file.rects());
         }
@@ -284,7 +284,7 @@ class ContractTransform : public UntrainableMetaTransform
         dst.append(out);
     }
-    virtual void project(const Template & src, Template & dst) const
+    virtual void project(const Template &src, Template &dst) const
     {
         qFatal("this has gone bad");
         (void) src; (void) dst;
@@ -316,7 +316,7 @@ class ForkTransform : public CompositeTransform
     }
     // same as _project, but calls projectUpdate on sub-transforms
-    void projectupdate(const Template & src, Template & dst)
+    void projectupdate(const Template &src, Template &dst)
     {
         foreach (Transform *f, transforms) {
             try {
@@ -331,7 +331,7 @@ class ForkTransform : public CompositeTransform
         }
     }
-    void projectUpdate(const TemplateList & src, TemplateList & dst)
+    void projectUpdate(const TemplateList &src, TemplateList &dst)
     {
         dst.reserve(src.size());
         for (int i=0; i<src.size(); i++) dst.append(Template(src[i].file));
@@ -345,7 +345,7 @@ class ForkTransform : public CompositeTransform
     // this is probably going to go bad, fork transform probably won't work well in a variable
     // input/output scenario
-    virtual void finalize(TemplateList & output)
+    virtual void finalize(TemplateList &output)
     {
         output.clear();
         // For each transform,
@@ -493,6 +493,12 @@ class LoadStoreTransform : public MetaTransform
 public:
     LoadStoreTransform() : transform(NULL) {}
+    bool setPropertyRecursive(const QString &name, QVariant value)
+    {
+        if (br::Object::setPropertyRecursive(name, value))
+            return true;
+        return transform->setPropertyRecursive(name, value);
+    }
 private:
     void init()
     {
@@ -543,7 +549,7 @@ private:
         transform->projectUpdate(src, dst);
     }
-    void finalize(TemplateList & output)
+    void finalize(TemplateList &output)
     {
         transform->finalize(output);
     }
@@ -635,7 +641,7 @@ class DistributeTemplateTransform : public MetaTransform
 public:
-    Transform * smartCopy(bool & newTransform)
+    Transform *smartCopy(bool &newTransform)
     {
         if (!transform->timeVarying()) {
             newTransform = false;
@@ -643,7 +649,7 @@ public:
         }
         newTransform = true;
-        DistributeTemplateTransform * output = new DistributeTemplateTransform;
+        DistributeTemplateTransform *output = new DistributeTemplateTransform;
         bool newChild = false;
         output->transform = transform->smartCopy(newChild);
         if (newChild)
@@ -660,8 +666,8 @@ public:
         }
         QList<TemplateList> separated;
-        foreach (const TemplateList & list, data) {
-            foreach(const Template & t, list) {
+        foreach (const TemplateList &list, data) {
+            foreach(const Template &t, list) {
                 separated.append(TemplateList());
                 separated.last().append(t);
             }
@@ -525,7 +525,7 @@ class IncrementalOutputTransform : public TimeVaryingTransform
         dst = src;
         int idx =0;
-        foreach(const Template & t, src) {
+        foreach(const Template &t, src) {
             if (t.empty())
                 continue;
@@ -546,7 +546,7 @@ class IncrementalOutputTransform : public TimeVaryingTransform
     }
     // Drop the current gallery.
-    void finalize(TemplateList & data)
+    void finalize(TemplateList &data)
     {
         (void) data;
         galleryUp = false;
@@ -573,7 +573,7 @@ class EventTransform : public UntrainableMetaTransform
         event.pulseSignal(dst);
     }
-    TemplateEvent * getEvent(const QString & name)
+    TemplateEvent *getEvent(const QString &name)
     {
         return name == eventName ? &event : NULL;
     }
@@ -646,7 +646,7 @@ class ProgressCounterTransform : public TimeVaryingTransform
         (void) data;
     }
-    void finalize(TemplateList & data)
+    void finalize(TemplateList &data)
     {
         (void) data;
         float p = br_progress();
@@ -692,7 +692,7 @@ class OutputTransform : public TimeVaryingTransform
             return;
         // we received a template, which is the next row/column in order
-        foreach(const Template & t, dst) {
+        foreach(const Template &t, dst) {
             for (int i=0; i < t.m().cols; i++)
             {
                 output->setRelative(t.m().at<float>(0, i), currentRow, currentCol);
@@ -819,7 +819,7 @@ class FileExclusionTransform : public UntrainableMetaTransform
     void project(const TemplateList &src, TemplateList &dst) const
     {
-        foreach(const Template & srcTemp, src)
+        foreach(const Template &srcTemp, src)
         {
             if (!excluded.contains(srcTemp.file))
                 dst.append(srcTemp);
@@ -48,8 +48,8 @@ protected:
 class TransformCopier : public ResourceMaker<Transform>
 {
 public:
-    Transform * basis;
-    TransformCopier(Transform * _basis)
+    Transform *basis;
+    TransformCopier(Transform *_basis)
     {
         basis = _basis;
     }
@@ -66,7 +66,7 @@ class TimeInvariantWrapperTransform : public MetaTransform
 public:
     Resource<Transform> transformSource;
-    TimeInvariantWrapperTransform(Transform * basis) : transformSource(new TransformCopier(basis))
+    TimeInvariantWrapperTransform(Transform *basis) : transformSource(new TransformCopier(basis))
     {
         if (!basis)
             qFatal("TimeInvariantWrapper created with NULL transform");
@@ -76,14 +76,14 @@ public:
     virtual void project(const Template &src, Template &dst) const
     {
-        Transform * aTransform = transformSource.acquire();
+        Transform *aTransform = transformSource.acquire();
         aTransform->projectUpdate(src,dst);
         transformSource.release(aTransform);
     }
     void project(const TemplateList &src, TemplateList &dst) const
     {
-        Transform * aTransform = transformSource.acquire();
+        Transform *aTransform = transformSource.acquire();
         aTransform->projectUpdate(src,dst);
         transformSource.release(aTransform);
     }
@@ -94,7 +94,7 @@ public:
     }
 private:
-    Transform * baseTransform;
+    Transform *baseTransform;
 };
 /*!
@@ -120,7 +120,7 @@ public:
     // Get a compile failure if this isn't here to go along with the other
     // projectUpdate, no idea why
-    virtual void projectUpdate(const Template & src, Template & dst)
+    virtual void projectUpdate(const Template &src, Template &dst)
     {
         (void) src; (void) dst;
         qFatal("do something useful");
@@ -128,7 +128,7 @@ public:
     virtual void projectUpdate(const TemplateList &src, TemplateList &dst)
     {
-        foreach (const Template & src_part, src) {
+        foreach (const Template &src_part, src) {
             Template out;
             projectUpdate(src_part, out);
             dst.append(out);
@@ -139,7 +139,7 @@ public:
      *\brief For transforms that don't do any training, this default implementation
      * which creates a new copy of the Transform from its description string is sufficient.
      */
-    virtual Transform * smartCopy(bool & newTransform)
+    virtual Transform *smartCopy(bool &newTransform)
     {
         newTransform = true;
         return this->clone();
@@ -180,17 +180,17 @@ public:
     {
         transform->projectUpdate(src,dst);
     }
-    void projectUpdate(const TemplateList & src, TemplateList & dst)
+    void projectUpdate(const TemplateList &src, TemplateList &dst)
     {
         transform->projectUpdate(src,dst);
     }
-    void train(const QList<TemplateList> & data)
+    void train(const QList<TemplateList> &data)
     {
         transform->train(data);
     }
-    virtual void finalize(TemplateList & output)
+    virtual void finalize(TemplateList &output)
     {
         transform->finalize(output);
     }
@@ -201,6 +201,17 @@ public:
             this->trainable = transform->trainable;
     }
+    bool setPropertyRecursive(const QString &name, QVariant value)
+    {
+        if (br::Object::setPropertyRecursive(name, value))
+            return true;
+
+        if (transform->setPropertyRecursive(name, value)) {
+            init();
+            return true;
+        }
+        return false;
+    }
 };
 /*!
@@ -251,7 +262,7 @@ public:
      * it creates a new copy of its own class, and gives that copy the child transforms
      * returned by calling smartCopy on this transforms children
      */
-    Transform * smartCopy(bool & newTransform)
+    Transform *smartCopy(bool &newTransform)
     {
         if (!timeVarying()) {
             newTransform = false;
@@ -273,7 +284,7 @@ public:
         name += ")";
         name.replace("br::","");
-        CompositeTransform * output = dynamic_cast<CompositeTransform *>(Transform::make(name, NULL));
+        CompositeTransform *output = dynamic_cast<CompositeTransform *>(Transform::make(name, NULL));
         if (output == NULL)
             qFatal("Dynamic cast failed!");
@@ -281,7 +292,7 @@ public:
         foreach(Transform* t, transforms )
         {
             bool newItem = false;
-            Transform * maybe_copy = t->smartCopy(newItem);
+            Transform *maybe_copy = t->smartCopy(newItem);
             if (newItem)
                 maybe_copy->setParent(output);
             output->transforms.append(maybe_copy);
@@ -293,11 +304,26 @@ public:
         return output;
     }
+    bool setPropertyRecursive(const QString &name, QVariant value)
+    {
+        if (br::Object::setPropertyRecursive(name, value))
+            return true;
+
+        for (int i=0; i < this->transforms.size();i++) {
+            if (transforms[i]->setPropertyRecursive(name, value)) {
+                init();
+                return true;
+            }
+        }
+        return false;
+    }
+
+
 protected:
     bool isTimeVarying;
-    virtual void _project(const Template & src, Template & dst) const = 0;
-    virtual void _project(const TemplateList & src, TemplateList & dst) const = 0;
+    virtual void _project(const Template &src, Template &dst) const = 0;
+    virtual void _project(const TemplateList &src, TemplateList &dst) const = 0;
     CompositeTransform() : TimeVaryingTransform(false) {}
 };
@@ -309,7 +335,7 @@ struct WorkerProcess
 {
     QString transform;
     QString baseName;
-    EnrollmentWorker * processInterface;
+    EnrollmentWorker *processInterface;
     void mainLoop();
 };
@@ -324,7 +350,7 @@ public:
     virtual void projectMetadata(const File &src, File &dst) const = 0;
-    void project(const Template & src, Template & dst) const
+    void project(const Template &src, Template &dst) const
     {
         dst = src;
         projectMetadata(src.file, dst.file);
@@ -220,7 +220,7 @@ class PP5EnrollTransform : public UntrainableMetaTransform
     BR_PROPERTY(bool, detectOnly, false)
     Resource<PP5Context> contexts;
-    void project(const Template & src, Template & dst) const
+    void project(const Template &src, Template &dst) const
     {
         if (Globals->enrollAll)
             qFatal("single template project doesn't support enrollAll");
@@ -232,7 +232,7 @@ class PP5EnrollTransform : public UntrainableMetaTransform
         dst = dstList.first();
     }
-    void project(const TemplateList &srcList, TemplateList & dstList) const
+    void project(const TemplateList &srcList, TemplateList &dstList) const
     {
         // Nothing to do here
         if (srcList.empty())
@@ -428,9 +428,9 @@ private:
         // each input dimension. Each input dimension corresponds to
         // one of the input rect region. Thus, each eigenvector represents
         // a set of overlaping regions.
-        float  * midX = new float[nRegions];
-        float * midY = new float[nRegions];
-        float * avgWidth = new float[nRegions];
+        float *midX = new float[nRegions];
+        float *midY = new float[nRegions];
+        float *avgWidth = new float[nRegions];
         float *avgHeight = new float[nRegions];
         float *confs = new float[nRegions];
         int *cnts = new int[nRegions];
@@ -45,10 +45,10 @@ public:
     SharedBuffer() {}
     virtual ~SharedBuffer() {}
-    virtual void addItem(FrameData * input)=0;
+    virtual void addItem(FrameData *input)=0;
     virtual void reset()=0;
-    virtual FrameData * tryGetItem()=0;
+    virtual FrameData *tryGetItem()=0;
     virtual int size()=0;
 };
@@ -63,14 +63,14 @@ public:
         next_target = 0;
     }
-    void addItem(FrameData * input)
+    void addItem(FrameData *input)
     {
         QMutexLocker bufferLock(&bufferGuard);
         buffer.insert(input->sequenceNumber, input);
     }
-    FrameData * tryGetItem()
+    FrameData *tryGetItem()
     {
         QMutexLocker bufferLock(&bufferGuard);
@@ -86,7 +86,7 @@ public:
         next_target = next_target + 1;
-        FrameData * output = result.value();
+        FrameData *output = result.value();
         buffer.erase(result);
         return output;
     }
@@ -134,19 +134,19 @@ public:
     }
     // called from the producer thread
-    void addItem(FrameData * input)
+    void addItem(FrameData *input)
     {
         QReadLocker readLock(&bufferGuard);
         inputBuffer->append(input);
     }
-    FrameData * tryGetItem()
+    FrameData *tryGetItem()
     {
         QReadLocker readLock(&bufferGuard);
         // There is something for us to get
         if (!outputBuffer->empty()) {
-            FrameData * output = outputBuffer->first();
+            FrameData *output = outputBuffer->first();
             outputBuffer->removeFirst();
             return output;
         }
@@ -165,7 +165,7 @@ public:
         std::swap(inputBuffer, outputBuffer);
         // Return a frame
-        FrameData * output = outputBuffer->first();
+        FrameData *output = outputBuffer->first();
         outputBuffer->removeFirst();
         return output;
     }
@@ -201,10 +201,10 @@ class TemplateProcessor
 {
 public:
     virtual ~TemplateProcessor() {}
-    virtual bool open(Template & input)=0;
+    virtual bool open(Template &input)=0;
     virtual bool isOpen()=0;
     virtual void close()=0;
-    virtual bool getNextTemplate(Template & output)=0;
+    virtual bool getNextTemplate(Template &output)=0;
 protected:
     Template basis;
     string getAbsolutePath(QString filename)
@@ -259,7 +259,7 @@ public:
     void close() { video.release(); }
-    bool getNextTemplate(Template & output)
+    bool getNextTemplate(Template &output)
     {
         if (!isOpen()) {
             qDebug("video source is not open");
@@ -320,7 +320,7 @@ struct StreamGallery : public TemplateProcessor
         lastBlock = true;
     }
-    bool getNextTemplate(Template & output)
+    bool getNextTemplate(Template &output)
     {
         // If we still have data available, we return one of those
         if ((nextIdx >= currentData.size()) && !lastBlock) {
@@ -373,7 +373,7 @@ public:
         basis.clear();
     }
-    bool getNextTemplate(Template & output)
+    bool getNextTemplate(Template &output)
     {
         if (!data_ok)
             return false;
@@ -563,7 +563,7 @@ private:
     // apparently the text in seq files is 16 bit characters (UTF-16?)
     // since we don't really need the last byte, snad since it gets interpreted as
     // a terminating char, let's just grab the first byte for storage
-    void readText(int bytes, char * buffer)
+    void readText(int bytes, char *buffer)
     {
         seqFile.read(buffer, bytes);
         for (int i=0; i<bytes; i+=2) {
@@ -601,7 +601,7 @@ public:
     {
         while (true)
         {
-            FrameData * frame = allFrames.tryGetItem();
+            FrameData *frame = allFrames.tryGetItem();
             if (frame == NULL)
                 break;
             delete frame;
@@ -623,7 +623,7 @@ public:
         return this->templates.size();
     }
-    bool open(const TemplateList & input, br::Idiocy::StreamModes _mode)
+    bool open(const TemplateList &input, br::Idiocy::StreamModes _mode)
     {
         // Set up variables specific to us
         current_template_idx = 0;
@@ -655,7 +655,7 @@ public:
     // Returns a NULL FrameData if too many frames are out, or the
     // data source is broken. Sets last_frame to true iff the FrameData
     // returned is the last valid frame, and the data source is now broken.
-    FrameData * tryGetFrame(bool & last_frame)
+    FrameData *tryGetFrame(bool &last_frame)
     {
         last_frame = false;
@@ -665,7 +665,7 @@ public:
         // Try to get a FrameData from the pool, if we can't it means too many
         // frames are already out, and we will return NULL to indicate failure
-        FrameData * aFrame = allFrames.tryGetItem();
+        FrameData *aFrame = allFrames.tryGetItem();
         if (aFrame == NULL)
             return NULL;
@@ -691,7 +691,7 @@ public:
     // Return a frame to the pool, returns true if the frame returned was the last
     // frame issued, false otherwise
-    bool returnFrame(FrameData * inputFrame)
+    bool returnFrame(FrameData *inputFrame)
     {
         int frameNumber = inputFrame->sequenceNumber;
@@ -786,7 +786,7 @@ protected:
         return true;
     }
-    bool getNextFrame(FrameData & output)
+    bool getNextFrame(FrameData &output)
     {
         bool got_frame = false;
@@ -832,7 +832,7 @@ protected:
     TemplateList templates;
     // processor for the current template
-    TemplateProcessor * frameSource;
+    TemplateProcessor *frameSource;
     int next_sequence_number;
     int final_frame;
@@ -859,7 +859,7 @@ public:
     QList<ProcessingStage *> * stages;
     int start_idx;
-    FrameData * startItem;
+    FrameData *startItem;
 };
 class ProcessingStage
@@ -873,9 +873,9 @@ public:
     }
     virtual ~ProcessingStage() {}
-    virtual FrameData* run(FrameData * input, bool & should_continue, bool & final)=0;
+    virtual FrameData* run(FrameData *input, bool &should_continue, bool &final)=0;
-    virtual bool tryAcquireNextStage(FrameData *& input, bool & final)=0;
+    virtual bool tryAcquireNextStage(FrameData *& input, bool &final)=0;
     int stage_id;
@@ -886,11 +886,11 @@ public:
 protected:
     int thread_count;
-    SharedBuffer * inputBuffer;
-    ProcessingStage * nextStage;
+    SharedBuffer *inputBuffer;
+    ProcessingStage *nextStage;
     QList<ProcessingStage *> * stages;
-    QThreadPool * threads;
-    Transform * transform;
+    QThreadPool *threads;
+    Transform *transform;
 };
@@ -901,7 +901,7 @@ public:
     // Not much to worry about here, we will project the input
     // and try to continue to the next stage.
-    FrameData * run(FrameData * input, bool & should_continue, bool & final)
+    FrameData *run(FrameData *input, bool &should_continue, bool &final)
     {
         if (input == NULL) {
             qFatal("null input to multi-thread stage");
@@ -916,7 +916,7 @@ public:
     // Called from a different thread than run. Nothing to worry about
     // we offer no restrictions on when loops may enter this stage.
-    virtual bool tryAcquireNextStage(FrameData *& input, bool & final)
+    virtual bool tryAcquireNextStage(FrameData *& input, bool &final)
     {
         (void) input;
         final = false;
@@ -974,7 +974,7 @@ public:
     QReadWriteLock statusLock;
     Status currentStatus;
-    FrameData * run(FrameData * input, bool & should_continue, bool & final)
+    FrameData *run(FrameData *input, bool &should_continue, bool &final)
     {
         if (input == NULL)
             qFatal("NULL input to stage %d", this->stage_id);
@@ -995,7 +995,7 @@ public:
         // Is there anything on our input buffer? If so we should start a thread with that.
         QWriteLocker lock(&statusLock);
-        FrameData * newItem = inputBuffer->tryGetItem();
+        FrameData *newItem = inputBuffer->tryGetItem();
         if (!newItem)
         {
             this->currentStatus = STOPPING;
@@ -1008,9 +1008,9 @@ public:
         return input;
     }
-    void startThread(br::FrameData * newItem)
+    void startThread(br::FrameData *newItem)
     {
-        BasicLoop * next = new BasicLoop();
+        BasicLoop *next = new BasicLoop();
         next->stages = stages;
         next->start_idx = this->stage_id;
         next->startItem = newItem;
@@ -1025,7 +1025,7 @@ public:
     // Calledfrom a different thread than run.
-    bool tryAcquireNextStage(FrameData *& input, bool & final)
+    bool tryAcquireNextStage(FrameData *& input, bool &final)
     {
         final = false;
         inputBuffer->addItem(input);
@@ -1078,7 +1078,7 @@ public:
         sets.append(src);
     }
-    void train(const TemplateList & data)
+    void train(const TemplateList &data)
     {
         (void) data;
     }
@@ -1099,7 +1099,7 @@ public:
         SingleThreadStage::reset();
     }
-    FrameData * run(FrameData * input, bool & should_continue, bool & final)
+    FrameData *run(FrameData *input, bool &should_continue, bool &final)
     {
         if (input == NULL)
             qFatal("NULL frame in input stage");
@@ -1112,7 +1112,7 @@ public:
         // frame if a frame is currently available.
         QWriteLocker lock(&statusLock);
         bool last_frame = false;
-        FrameData * newFrame = dataSource.tryGetFrame(last_frame);
+        FrameData *newFrame = dataSource.tryGetFrame(last_frame);
         // Were we able to get a frame?
         if (newFrame) startThread(newFrame);
@@ -1127,7 +1127,7 @@ public:
     }
     // The last stage, trying to access the first stage
-    bool tryAcquireNextStage(FrameData *& input, bool & final)
+    bool tryAcquireNextStage(FrameData *& input, bool &final)
     {
         // Return the frame, was it the last one?
         final = dataSource.returnFrame(input);
@@ -1180,7 +1180,7 @@ public:
 void BasicLoop::run()
 {
     int current_idx = start_idx;
-    FrameData * target_item = startItem;
+    FrameData *target_item = startItem;
     bool should_continue = true;
     bool the_end = false;
     forever
@@ -1212,7 +1212,7 @@ public:
     friend class StreamTransfrom;
-    void subProject(QList<TemplateList> & data, int end_idx)
+    void subProject(QList<TemplateList> &data, int end_idx)
     {
         if (end_idx == 0)
             return;
@@ -1239,15 +1239,15 @@ public:
         transforms = backup;
     }
-    void train(const QList<TemplateList> & data)
+    void train(const QList<TemplateList> &data)
     {
         if (!trainable) {
             qWarning("Attempted to train untrainable transform, nothing will happen.");
             return;
         }
         QList<TemplateList> separated;
-        foreach (const TemplateList & list, data) {
-            foreach(const Template & t, list) {
+        foreach (const TemplateList &list, data) {
+            foreach(const Template &t, list) {
                 separated.append(TemplateList());
                 separated.last().append(t);
             }
@@ -1292,7 +1292,7 @@ public:
     }
-    virtual void finalize(TemplateList & output)
+    virtual void finalize(TemplateList &output)
     {
         (void) output;
         // Nothing in particular to do here, stream calls finalize
@@ -1301,7 +1301,7 @@ public:
     // start processing, consider all templates in src a continuous
     // 'video'
-    void projectUpdate(const TemplateList & src, TemplateList & dst)
+    void projectUpdate(const TemplateList &src, TemplateList &dst)
     {
         dst = src;
         if (src.empty())
@@ -1319,7 +1319,7 @@ public:
         // We have to get a frame before starting the thread
         bool last_frame = false;
-        FrameData * firstFrame = readStage->dataSource.tryGetFrame(last_frame);
+        FrameData *firstFrame = readStage->dataSource.tryGetFrame(last_frame);
         if (firstFrame == NULL)
             qFatal("Failed to read first frame of video");
@@ -1355,14 +1355,14 @@ public:
         // dst is set to all output received by the final stage, along
         // with anything output via the calls to finalize.
-        foreach(const TemplateList & list, collector->sets) {
+        foreach(const TemplateList &list, collector->sets) {
             dst.append(list);
         }
         collector->sets.clear();
         dst.append(final_output);
-        foreach(ProcessingStage * stage, processingStages) {
+        foreach(ProcessingStage *stage, processingStages) {
             stage->reset();
         }
     }
@@ -1474,8 +1474,8 @@ public:
 protected:
     QList<bool> stage_variance;
-    ReadStage * readStage;
-    SingleThreadStage * collectionStage;
+    ReadStage *readStage;
+    SingleThreadStage *collectionStage;
     QSharedPointer<CollectSets> collector;
     QList<ProcessingStage *> processingStages;
@@ -1497,14 +1497,14 @@ protected:
     // number of jobs.
     static QHash<QObject *, QThreadPool *> pools;
     static QMutex poolsAccess;
-    QThreadPool * threads;
+    QThreadPool *threads;
     void _project(const Template &src, Template &dst) const
     {
         (void) src; (void) dst;
         qFatal("nope");
     }
-    void _project(const TemplateList & src, TemplateList & dst) const
+    void _project(const TemplateList &src, TemplateList &dst) const
     {
         (void) src; (void) dst;
         qFatal("nope");
@@ -1542,17 +1542,17 @@ public:
     {
         basis.projectUpdate(src,dst);
     }
-    void projectUpdate(const TemplateList & src, TemplateList & dst)
+    void projectUpdate(const TemplateList &src, TemplateList &dst)
     {
         basis.projectUpdate(src,dst);
     }
-    void train(const QList<TemplateList> & data)
+    void train(const QList<TemplateList> &data)
     {
         basis.train(data);
     }
-    virtual void finalize(TemplateList & output)
+    virtual void finalize(TemplateList &output)
     {
         (void) output;
         // Nothing in particular to do here, stream calls finalize
@@ -1573,7 +1573,7 @@ public:
         basis.readMode = this->readMode;
         // We need at least a CompositeTransform * to acess transform's children.
-        CompositeTransform * downcast = dynamic_cast<CompositeTransform *> (transform);
+        CompositeTransform *downcast = dynamic_cast<CompositeTransform *> (transform);
         // If this isn't even a composite transform, or it's not a pipe, just set up
         // basis with 1 stage.
@@ -1627,7 +1627,7 @@ public:
             }
             //otherwise we build a pipe
             else {
-                CompositeTransform * pipe = dynamic_cast<CompositeTransform *>(Transform::make("Pipe([])", this));
+                CompositeTransform *pipe = dynamic_cast<CompositeTransform *>(Transform::make("Pipe([])", this));
                 pipe->transforms = sets[i];
                 pipe->init();
                 transform_set.append(pipe);
@@ -1638,10 +1638,10 @@ public:
         basis.init();
     }
-    Transform * smartCopy(bool & newTransform)
+    Transform *smartCopy(bool &newTransform)
     {
         // We just want the DirectStream to begin with, so just return a copy of that.
-        DirectStreamTransform * res = (DirectStreamTransform *) basis.smartCopy(newTransform);
+        DirectStreamTransform *res = (DirectStreamTransform *) basis.smartCopy(newTransform);
         res->activeFrames = this->activeFrames;
         return res;
     }