mirror of
https://invent.kde.org/network/falkon.git
synced 2024-12-20 02:36:34 +01:00
[Code] Add/Edit some comments
This commit is contained in:
parent
0d5c86f8cd
commit
b4f0096f96
@ -58,6 +58,8 @@ void LocationCompleterModel::refreshCompletions(const QString &string)
|
|||||||
QList<QUrl> urlList;
|
QList<QUrl> urlList;
|
||||||
QList<QStandardItem*> itemList;
|
QList<QStandardItem*> itemList;
|
||||||
|
|
||||||
|
// TODO: Those 2 SQL queries can be merged with UNION
|
||||||
|
|
||||||
if (showType == HistoryAndBookmarks || showType == Bookmarks) {
|
if (showType == HistoryAndBookmarks || showType == Bookmarks) {
|
||||||
QSqlQuery query = createQuery(string, "history.count DESC", urlList, limit, true);
|
QSqlQuery query = createQuery(string, "history.count DESC", urlList, limit, true);
|
||||||
query.exec();
|
query.exec();
|
||||||
|
@ -239,7 +239,7 @@ QWebElement WebView::activeElement() const
|
|||||||
|
|
||||||
bool WebView::isUrlValid(const QUrl &url)
|
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());
|
return url.isValid() && !url.scheme().isEmpty() && (!url.host().isEmpty() || !url.path().isEmpty());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user