From 02731743c0f4fec68f701cdc211a2fe2995d76bb Mon Sep 17 00:00:00 2001 From: Marshall Lochbaum Date: Sat, 23 Jul 2016 11:07:01 -0400 Subject: [PATCH] Use qtutils.ensure_valid instead of testing isValid in get_by_qurl --- qutebrowser/browser/urlmarks.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/qutebrowser/browser/urlmarks.py b/qutebrowser/browser/urlmarks.py index f8b011439..caabf72fa 100644 --- a/qutebrowser/browser/urlmarks.py +++ b/qutebrowser/browser/urlmarks.py @@ -32,7 +32,8 @@ import collections from PyQt5.QtCore import pyqtSignal, QUrl, QObject -from qutebrowser.utils import message, usertypes, urlutils, standarddir, objreg +from qutebrowser.utils import (message, usertypes, qtutils, urlutils, + standarddir, objreg) from qutebrowser.commands import cmdexc, cmdutils from qutebrowser.misc import lineparser @@ -210,8 +211,7 @@ class QuickmarkManager(UrlMarkManager): Takes O(n) time, where n is the number of quickmarks. Use a name instead where possible. """ - if not url.isValid(): - raise ValueError("Invalid URL: {}".format(url.errorString())) + qtutils.ensure_valid(url) urlstr = url.toString(QUrl.RemovePassword | QUrl.FullyEncoded) try: