Funkwhale/Clementine-audio-player-Mac-Linux-Windows
1
0
Fork 0
mirror of https://github.com/clementine-player/Clementine synced 2025-01-10 17:13:39 +01:00
Code Issues Releases Wiki Activity

Changed my mind about automatically passing QFutures.

Browse Source
This commit is contained in:
John Maguire 2015-11-27 12:03:47 +00:00
parent ec98a68c3d
commit 2d61fe6c87
3 changed files with 8 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
ext/libclementine-common/core/closure.h
View File

@ -197,16 +197,6 @@ _detail::ClosureBase* NewClosure(QFuture<T> future, QObject* receiver,
QFutureWatcher<T>* watcher = new QFutureWatcher<T>;
watcher->setFuture(future);
QObject::connect(watcher, SIGNAL(finished()), watcher, SLOT(deleteLater()));
return NewClosure(watcher, SIGNAL(finished()), receiver, slot, future,
args...);
}
template <typename... Args>
_detail::ClosureBase* NewClosure(QFuture<void> future, QObject* receiver,
const char* slot, const Args&... args) {
QFutureWatcher<void>* watcher = new QFutureWatcher<void>;
watcher->setFuture(future);
QObject::connect(watcher, SIGNAL(finished()), watcher, SLOT(deleteLater()));
return NewClosure(watcher, SIGNAL(finished()), receiver, slot, args...);
}

9
src/engines/gstengine.cpp
View File

@ -455,8 +455,9 @@ bool GstEngine::Play(quint64 offset_nanosec) {
QFuture<GstStateChangeReturn> future =
current_pipeline_->SetState(GST_STATE_PLAYING);
NewClosure(future, this, SLOT(PlayDone(QFuture<GstStateChangeReturn>, quint64, int)), offset_nanosec,
current_pipeline_->id());
NewClosure(future, this,
SLOT(PlayDone(QFuture<GstStateChangeReturn>, quint64, int)),
future, offset_nanosec, current_pipeline_->id());
if (is_fading_out_to_pause_) {
current_pipeline_->SetState(GST_STATE_PAUSED);
@ -848,7 +849,9 @@ int GstEngine::AddBackgroundStream(shared_ptr<GstEnginePipeline> pipeline) {
background_streams_[stream_id] = pipeline;
QFuture<GstStateChangeReturn> future = pipeline->SetState(GST_STATE_PLAYING);
NewClosure(future, this, SLOT(BackgroundStreamPlayDone(QFuture<GstStateChangeReturn>, int)), stream_id);
NewClosure(future, this,
SLOT(BackgroundStreamPlayDone(QFuture<GstStateChangeReturn>, int)),
future, stream_id);
return stream_id;
}

3
src/globalsearch/searchprovider.cpp
View File

@ -82,7 +82,8 @@ void BlockingSearchProvider::SearchAsync(int id, const QString& query) {
QFuture<ResultList> future =
QtConcurrent::run(this, &BlockingSearchProvider::Search, id, query);
NewClosure(future, this,
SLOT(BlockingSearchFinished(QFuture<ResultList>, int)), id);
SLOT(BlockingSearchFinished(QFuture<ResultList>, int)), future,
id);
}
void BlockingSearchProvider::BlockingSearchFinished(QFuture<ResultList> future,