1
0
Fork 0
mirror of https://github.com/transmission/transmission synced 2025-02-23 14:40:43 +00:00
transmission/qt/RpcQueue.cc
Mike Gelfand 2b917de65b Refactor RPC requests code for proper queueing (patch by intelfx @ GH-10)
This refactoring is driven by the need to be able to do true queued RPC calls
(where each successive call uses the result of the previous).

Currently, such queueing of requests is done by assigning them special "magic"
tag numbers, which are then intercepted in one big switch() statement and acted
upon. This (aside from making code greatly unclear) effectively makes each such
queue a singleton, because state passing is restricted to global variables.

We refactor RpcClient to assign an unique tag to each remote call, and then
abstract all the call<->response matching with Qt's future/promise mechanism.

Finally, we introduce a "RPC request queue" class (RpcQueue) which is built on
top of QFutureWatcher and C++11's <functional> library. This class maintains
a queue of functions, where each function receives an RPC response, does
necessary processing, performs another call and finally returns its future.
2016-04-19 20:41:59 +00:00

88 lines
1.9 KiB
C++

/*
* This file Copyright (C) 2016 Mnemosyne LLC
*
* It may be used under the GNU GPL versions 2 or 3
* or any future license endorsed by Mnemosyne LLC.
*
* $Id$
*/
#include <cassert>
#include "RpcQueue.h"
RpcQueue::RpcQueue (QObject * parent):
QObject (parent),
myTolerateErrors (false)
{
connect (&myFutureWatcher, SIGNAL (finished ()), SLOT (stepFinished ()));
}
RpcResponseFuture
RpcQueue::future ()
{
return myPromise.future ();
}
void
RpcQueue::stepFinished ()
{
RpcResponse result;
if (myFutureWatcher.future ().isResultReadyAt (0))
{
result = myFutureWatcher.result ();
RpcResponseFuture future = myFutureWatcher.future ();
// we can't handle network errors, abort queue and pass the error upwards
if (result.networkError != QNetworkReply::NoError)
{
assert (!result.success);
myPromise.reportFinished (&result);
deleteLater ();
return;
}
// call user-handler for ordinary errors
if (!result.success && myNextErrorHandler)
{
myNextErrorHandler (future);
}
// run next request, if we have one to run and there was no error (or if we tolerate errors)
if ((result.success || myTolerateErrors) && !myQueue.isEmpty ())
{
runNext (future);
return;
}
}
else
{
assert (!myNextErrorHandler);
assert (myQueue.isEmpty ());
// one way or another, the last step returned nothing.
// assume it is OK and ensure that we're not going to give an empty response object to any of the next steps.
result.success = true;
}
myPromise.reportFinished (&result);
deleteLater ();
}
void
RpcQueue::runNext (const RpcResponseFuture& response)
{
assert (!myQueue.isEmpty ());
auto next = myQueue.dequeue ();
myNextErrorHandler = next.second;
myFutureWatcher.setFuture ((next.first) (response));
}
void
RpcQueue::run ()
{
runNext (RpcResponseFuture ());
}