From be7fd4c85b3086c3ec813d53f6eb46da1049e275 Mon Sep 17 00:00:00 2001
From: wang-bin <wbsecg1@gmail.com>
Date: Tue, 4 Nov 2014 18:53:55 +0800
Subject: [PATCH] misc: wrong option parse result. wrong signal param

---
 examples/common/common.cpp                   |  4 ++--
 examples/common/qoptions.cpp                 | 11 +++++------
 examples/player/MainWindow.cpp               |  2 +-
 examples/player/config/DecoderConfigPage.cpp |  2 +-
 tests/extract/main.cpp                       | 10 ++++++++--
 5 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/examples/common/common.cpp b/examples/common/common.cpp
index 92bf065ab..b06d9c74d 100644
--- a/examples/common/common.cpp
+++ b/examples/common/common.cpp
@@ -34,12 +34,12 @@ void load_qm(const QStringList &names)
     foreach(QString qm, qms) {
         QTranslator *ts = new QTranslator(qApp);
         QString path = qApp->applicationDirPath() + "/i18n/" + qm + "_" + QLocale::system().name();
-        qDebug() << "loading qm: " << path;
+        //qDebug() << "loading qm: " << path;
         if (ts->load(path)) {
             qApp->installTranslator(ts);
         } else {
             path = ":/i18n/" + qm + "_" + QLocale::system().name();
-            qDebug() << "loading qm: " << path;
+            //qDebug() << "loading qm: " << path;
             if (ts->load(path))
                 qApp->installTranslator(ts);
             else
diff --git a/examples/common/qoptions.cpp b/examples/common/qoptions.cpp
index a99dd29fb..b581a9556 100644
--- a/examples/common/qoptions.cpp
+++ b/examples/common/qoptions.cpp
@@ -209,7 +209,7 @@ bool QOptions::parse(int argc, const char *const*argv)
         if (it->startsWith("--")) {
             int e = it->indexOf('=');
             for (it_list = mOptions.begin(); it_list != mOptions.end(); ++it_list) {
-                if (it_list->longName().startsWith(it->mid(2,e-2))) {
+                if (it_list->longName() == it->mid(2,e-2)) {
                     if (it_list->type()==QOption::NoToken) {
                         it_list->setValue(true);
                         //qDebug("%d %s", __LINE__, qPrintable(it_list->value().toString()));
@@ -240,11 +240,10 @@ bool QOptions::parse(int argc, const char *const*argv)
             for (it_list = mOptions.begin(); it_list != mOptions.end(); ++it_list) {
                 QString sname = it_list->shortName();
 				int sname_len = sname.length(); //usally is 1
-				//Not endsWith, -oabco
-                if (it->indexOf(sname) == 1) {
+                //TODO: startsWith(-height,-h) Not endsWith, -oabco
+                if (it->midRef(1).compare(sname) == 0) {
                     if (it_list->type() == QOption::NoToken) {
                         it_list->setValue(true);
-                        //qDebug("%d %s", __LINE__, qPrintable(it_list->value().toString()));
 						it = args.erase(it);
 						break;
 					}
@@ -254,10 +253,10 @@ bool QOptions::parse(int argc, const char *const*argv)
                             break;
                         it_list->setValue(*it);
                         //qDebug("%d %s", __LINE__, qPrintable(it_list->value().toString()));
-					} else {
+                    } else {
                         it_list->setValue(it->mid(sname_len+1));
                         //qDebug("%d %s", __LINE__, qPrintable(it_list->value().toString()));
-					}
+                    }
 					it = args.erase(it);
 					break;
 				}
diff --git a/examples/player/MainWindow.cpp b/examples/player/MainWindow.cpp
index 12d817b20..20dca8bdf 100644
--- a/examples/player/MainWindow.cpp
+++ b/examples/player/MainWindow.cpp
@@ -151,7 +151,7 @@ void MainWindow::initPlayer()
     onCaptureConfigChanged();
     onAVFilterConfigChanged();
     connect(&Config::instance(), SIGNAL(captureDirChanged(QString)), SLOT(onCaptureConfigChanged()));
-    connect(&Config::instance(), SIGNAL(captureFormatChanged(QByteArray)), SLOT(onCaptureConfigChanged()));
+    connect(&Config::instance(), SIGNAL(captureFormatChanged(QString)), SLOT(onCaptureConfigChanged()));
     connect(&Config::instance(), SIGNAL(captureQualityChanged(int)), SLOT(onCaptureConfigChanged()));
     connect(&Config::instance(), SIGNAL(avfilterChanged()), SLOT(onAVFilterConfigChanged()));
     connect(mpStopBtn, SIGNAL(clicked()), mpPlayer, SLOT(stop()));
diff --git a/examples/player/config/DecoderConfigPage.cpp b/examples/player/config/DecoderConfigPage.cpp
index 946c69b8f..59e358cc2 100644
--- a/examples/player/config/DecoderConfigPage.cpp
+++ b/examples/player/config/DecoderConfigPage.cpp
@@ -219,7 +219,7 @@ DecoderConfigPage::DecoderConfigPage(QWidget *parent) :
     hb->addWidget(mpUp);
     hb->addWidget(mpDown);
     vb->addLayout(hb);
-    connect(&Config::instance(), SIGNAL(decoderPriorityChanged(QVector<QtAV::VideoDecoderId>)), SLOT(onConfigChanged()));
+    connect(&Config::instance(), SIGNAL(decoderPriorityNamesChanged()), SLOT(onConfigChanged()));
     updateDecodersUi();
 }
 
diff --git a/tests/extract/main.cpp b/tests/extract/main.cpp
index 265163e3c..ce78f1fb4 100644
--- a/tests/extract/main.cpp
+++ b/tests/extract/main.cpp
@@ -62,16 +62,22 @@ int main(int argc, char** argv)
     int t = 0;
     if (idx > 0)
         t = a.arguments().at(idx+1).toInt();
+    int n = 1;
+    idx = a.arguments().indexOf("-n");
+    if (idx > 0)
+        n = a.arguments().at(idx+1).toInt();
+    bool async = a.arguments().contains("-async");
+
 
     VideoFrameExtractor extractor;
-    extractor.setAsync(true);
+    extractor.setAsync(async);
     VideoFrameObserver obs;
     QObject::connect(&extractor, SIGNAL(frameExtracted(QtAV::VideoFrame)), &obs, SLOT(onVideoFrameExtracted(QtAV::VideoFrame)));
     extractor.setSource(file);
 
     QElapsedTimer timer;
     timer.start();
-    for (int i = 0; i < 30; ++i) {
+    for (int i = 0; i < n; ++i) {
         extractor.setPosition(t + 1000*i);
     }
     qDebug("elapsed: %lld", timer.elapsed());