From 749056ff906435cf990f993d39af251276232478 Mon Sep 17 00:00:00 2001 From: Florian Bruhin Date: Wed, 23 May 2018 09:44:44 +0200 Subject: Use functools instead of a lambda for QTimer It reads nicer, and this is also speculative fix for #3896 as PyQt5 is hopefully better at disconnecting partial-objects from dead objects than it is with lambdas. (cherry picked from commit 7162f15348219d3736a8d17785d9b6e2dff49f8f) --- qutebrowser/browser/webengine/webenginetab.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/qutebrowser/browser/webengine/webenginetab.py b/qutebrowser/browser/webengine/webenginetab.py index a7b1446e1..a7ec618ed 100644 --- a/qutebrowser/browser/webengine/webenginetab.py +++ b/qutebrowser/browser/webengine/webenginetab.py @@ -1026,8 +1026,9 @@ class WebEngineTab(browsertab.AbstractTab): log.config.debug( "Loading {} again because of config change".format( self._reload_url.toDisplayString())) - QTimer.singleShot(100, lambda url=self._reload_url: - self.openurl(url, predict=False)) + QTimer.singleShot(100, functools.partial(self.openurl, + self._reload_url, + predict=False)) self._reload_url = None if not qtutils.version_check('5.10', compiled=False): -- cgit v1.2.3-54-g00ecf