comparison runner/main.cpp @ 120:dcecd6997214 test-reorg

Abandon if we fail to add a transform; don't continue with the remaining transforms (that's very confusing behaviour)
author Chris Cannam
date Wed, 08 Oct 2014 15:26:48 +0100
parents 5be4995f4029
children 685e5cf3cb1d
comparison
equal deleted inserted replaced
119:7a31201dc42d 120:dcecd6997214
764 764
765 for (set<string>::const_iterator i = requestedTransformFiles.begin(); 765 for (set<string>::const_iterator i = requestedTransformFiles.begin();
766 i != requestedTransformFiles.end(); ++i) { 766 i != requestedTransformFiles.end(); ++i) {
767 if (manager.addFeatureExtractorFromFile(i->c_str(), writers)) { 767 if (manager.addFeatureExtractorFromFile(i->c_str(), writers)) {
768 haveFeatureExtractor = true; 768 haveFeatureExtractor = true;
769 } else {
770 cerr << "ERROR: Failed to add feature extractor from transform file \"" << *i << "\"" << endl;
771 good = false;
769 } 772 }
770 } 773 }
771 774
772 for (set<string>::const_iterator i = requestedDefaultTransforms.begin(); 775 for (set<string>::const_iterator i = requestedDefaultTransforms.begin();
773 i != requestedDefaultTransforms.end(); ++i) { 776 i != requestedDefaultTransforms.end(); ++i) {
774 if (manager.addDefaultFeatureExtractor(i->c_str(), writers)) { 777 if (manager.addDefaultFeatureExtractor(i->c_str(), writers)) {
775 haveFeatureExtractor = true; 778 haveFeatureExtractor = true;
779 } else {
780 cerr << "ERROR: Failed to add default feature extractor for transform \"" << *i << "\"" << endl;
781 good = false;
776 } 782 }
777 } 783 }
778 784
779 if (!haveFeatureExtractor) { 785 if (!haveFeatureExtractor) {
780 cerr << myname.toStdString() << ": no feature extractors added" << endl; 786 cerr << myname.toStdString() << ": no feature extractors added" << endl;