From b4f0096f96de5e8c0ba747efc03543784da05293 Mon Sep 17 00:00:00 2001 From: nowrep Date: Thu, 5 Dec 2013 19:52:13 +0100 Subject: [PATCH] [Code] Add/Edit some comments --- src/lib/navigation/completer/locationcompletermodel.cpp | 2 ++ src/lib/webview/webview.cpp | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/lib/navigation/completer/locationcompletermodel.cpp b/src/lib/navigation/completer/locationcompletermodel.cpp index b5d74995a..564f59812 100644 --- a/src/lib/navigation/completer/locationcompletermodel.cpp +++ b/src/lib/navigation/completer/locationcompletermodel.cpp @@ -58,6 +58,8 @@ void LocationCompleterModel::refreshCompletions(const QString &string) QList urlList; QList itemList; + // TODO: Those 2 SQL queries can be merged with UNION + if (showType == HistoryAndBookmarks || showType == Bookmarks) { QSqlQuery query = createQuery(string, "history.count DESC", urlList, limit, true); query.exec(); diff --git a/src/lib/webview/webview.cpp b/src/lib/webview/webview.cpp index 0f26ba967..a18e72aba 100644 --- a/src/lib/webview/webview.cpp +++ b/src/lib/webview/webview.cpp @@ -239,7 +239,7 @@ QWebElement WebView::activeElement() const bool WebView::isUrlValid(const QUrl &url) { - // Valid url must have scheme and actually contains something (so scheme:// is invalid) + // Valid url must have scheme and actually contains something (therefore scheme:// is invalid) return url.isValid() && !url.scheme().isEmpty() && (!url.host().isEmpty() || !url.path().isEmpty()); }