]> git.sur5r.net Git - minitube/blobdiff - src/channelsuggest.cpp
New upstream version 2.9
[minitube] / src / channelsuggest.cpp
index 20b564ac54249f5d4d00cb579bcde85a3e8059d0..eff93b4de205b58c9dc88a21342993884dd8fba3 100644 (file)
@@ -19,12 +19,8 @@ along with Minitube.  If not, see <http://www.gnu.org/licenses/>.
 $END_LICENSE */
 
 #include "channelsuggest.h"
-#include "networkaccess.h"
-#include "compatibility/qurlqueryhelper.h"
-
-namespace The {
-    NetworkAccess* http();
-}
+#include "http.h"
+#include "httputils.h"
 
 ChannelSuggest::ChannelSuggest(QObject *parent) : Suggester(parent) {
 
@@ -32,19 +28,21 @@ ChannelSuggest::ChannelSuggest(QObject *parent) : Suggester(parent) {
 
 void ChannelSuggest::suggest(const QString &query) {
     QUrl url("https://www.youtube.com/results");
-    {
-        QUrlQueryHelper urlHelper(url);
-        urlHelper.addQueryItem("search_type", "search_users");
-        urlHelper.addQueryItem("search_query", query);
-    }
+    QUrlQuery q;
+    q.addQueryItem("search_type", "search_users");
+    q.addQueryItem("search_query", query);
+    url.setQuery(q);
 
-    QObject *reply = The::http()->get(url);
+    QObject *reply = HttpUtils::yt().get(url);
     connect(reply, SIGNAL(data(QByteArray)), SLOT(handleNetworkData(QByteArray)));
 }
 
 void ChannelSuggest::handleNetworkData(QByteArray data) {
+    const int maxSuggestions = 10;
     QStringList choices;
-    QList<Suggestion*> suggestions;
+    choices.reserve(maxSuggestions);
+    QVector<Suggestion*> suggestions;
+    suggestions.reserve(maxSuggestions);
 
     QString html = QString::fromUtf8(data);
     QRegExp re("/(?:user|channel)/[a-zA-Z0-9]+[^>]+data-ytid=[\"']([^\"']+)[\"'][^>]+>([a-zA-Z0-9 ]+)</a>");
@@ -57,7 +55,7 @@ void ChannelSuggest::handleNetworkData(QByteArray data) {
             QString channelId = re.cap(1);
             suggestions << new Suggestion(choice, "channel", channelId);
             choices << choice;
-            if (choices.size() == 10) break;
+            if (choices.size() == maxSuggestions) break;
         }
         pos += re.matchedLength();
     }