Fixed clearing notifications

libyui · Jul 9, 2018 · e1d546d · e1d546d
1 parent 5ebf1c1
commit e1d546d
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/YQDialog.cc b/src/YQDialog.cc
@@ -133,6 +133,7 @@ void YQDialog::clearHttpNotifiers() {
 	yuiMilestone() << "Clearing notifiers..." << std::endl;
 	BOOST_FOREACH(QSocketNotifier *_notifier, _http_notifiers)
 	{
+		yuiMilestone() << "Removing notifier for fd " << _notifier->socket() << std::endl;
 		_notifier->setEnabled(false);
 		delete _notifier;
 	}
@@ -806,20 +807,23 @@ void YQDialog::createHttpNotifiers()
 			QSocketNotifier *_notifier = new QSocketNotifier( s, QSocketNotifier::Read );
 			QObject::connect( _notifier,	&pclass(_notifier)::activated,
 				this, &pclass(this)::httpData);
+			_http_notifiers.push_back(_notifier);
 		}
 
 		BOOST_FOREACH(int s, sockets.write())
 		{
 			QSocketNotifier *_notifier = new QSocketNotifier( s, QSocketNotifier::Write );
 			QObject::connect( _notifier,	&pclass(_notifier)::activated,
 				this, &pclass(this)::httpData);
+			_http_notifiers.push_back(_notifier);
 		}
 
 		BOOST_FOREACH(int s, sockets.exception())
 		{
 			QSocketNotifier *_notifier = new QSocketNotifier( s, QSocketNotifier::Exception );
 			QObject::connect( _notifier,	&pclass(_notifier)::activated,
 				this, &pclass(this)::httpData);
+			_http_notifiers.push_back(_notifier);
 		}
 	}
 }
@@ -851,8 +855,6 @@ YQDialog::waitForEventInternal( int timeout_millisec )
 #endif
 	_eventLoop->exec();
 
-		clearHttpNotifiers();
-
 #if VERBOSE_EVENT_LOOP
 	yuiDebug() << "Event loop finished for " << this << std::endl;
 #endif