XML-RPC клиент для rTorrent

[fuCtor]

на днях появилась идея сделать небольшую утилитку для мониторинга и управления rTorrent-ом. Нашел библиотеку(код) реализации XML-RPC клиента: http://code.google.com/p/qxmlrpc/
Данный клиент работает в асинхронном режиме. Поэтому сделал обертку над всеми методами и тд. В итоге получилось что каждый класс по таймеру (у каждого свой таймер) запрашивает обновления о состояниях, и обновляет внутренние переменные. Вот тут и возникла проблема. При наличии в списке большого количества торентов отправляется большое количество запросов, примерно 8-9 на каждый. В результате чего процессор загружается до 60% на некоторое время. Для приложения это не заметно т.к. весь этот процесс вынесен в отдельный поток.

Как можно оптимизировать данную задачу? Пока решил тем что увеличил интервал обновления данных.
А также любая конструктивная критика приветствуется =).

Исходники прилагаются.

UPD: обнаружил утечку памяти... приличную утечку  ... но еще не устранил ...
UPD2:
Утечку нашел в файле xmlrpc/client.cpp:167
QBuffer *outBuffer = new QBuffer;

Только как исправить

[vaprele07]

QBuffer *outBuffer = new QBuffer(this);

[fuCtor]

Так вроде тогда объект удалится когда удалится и родитель. Но родитель (xmlrpc::Client) создается при запуске и удаляется после закрытия, а в промежутке между этим объекты все равно будут копиться, или я что-то не до понял в механизмах Qt?

[kuzulis]

1. или вместо

Код:

    QBuffer *outBuffer = new QBuffer;

написать :

Код:

    QBuffer outBuffer;

2. или после

Код:

    d->serverResponses[id] = outBuffer;

написать:

Код:

delete(outBuffer);
outBuffer = 0;

не?

[spirit]

непонятно зачем вот это?

Код:

delete(outBuffer);
outBuffer = 0;

если вы выше объект в стеке создаете.

Код:

QBuffer outBuffer;

[BRE]

не?

Не.

Камрады, посмотрите весь код. Этот буфер используется в requestFinished. Вот там и нужно его освобождать.
Внимание обращаем на те места где используется d->serverResponses.take(id)

P.S. Да, это только беглый взгляд на этот код, возможны недосмотры. 

[vaprele07]

может стоит QByteArray хранить вместо буфера?

[kuzulis]

2 spirit  , внимательнее надо быть !

[fuCtor]

Вот итоговый слот, утечки нет наконец-то =)

Код

C++ (Qt)
void Client::requestFinished(int id, bool error)
{
    if ( !d->serverResponses.count(id) ) {
        return;
    }
 
#ifdef XMLRPC_DEBUG
    qDebug() << "request" <<  d->methodNames[id] <<  "finished, id=" << id << ", isError:" << error;
#endif
 
    if ( error ) {
        //if ( d->serverResponses.count(id) )
 
        delete d->serverResponses.take(id);
 
        emit failed(id, -32300, d->http->errorString() );
        return;
    }
 
    if ( d->serverResponses.count(id) ) {
		QBuffer * inBuffer = d->serverResponses.take(id);
        QByteArray buf = inBuffer->buffer();
 
        //qDebug() << "xml-rpc server response:\n" << QString(buf);
 
        Response response;
 
        QString errorMessage;
        if ( response.setContent( buf, &errorMessage ) ) {
            Q_ASSERT( !response.isNull() );
 
            if ( response.isFault() ) {
                qDebug() << "request failed:" << response.faultCode() << response.faultString();
                emit failed(id, response.faultCode(), response.faultString() );
            } else {
#ifdef XMLRPC_DEBUG
                qDebug() << response.returnValue().pprint();
#endif
                emit done( id, response.returnValue() );
            }
 
        } else {
 
#ifdef XMLRPC_DEBUG
            qDebug() << "incorrect xmlrpc response:" << errorMessage;
            qDebug() << QString(buf);
#endif
            emit failed(id, -32600, "Server error: Invalid xml-rpc. \nNot conforming to spec.");
        }
		delete inBuffer;
    }
 
}

Надо будет автору кода отправить изменения.

А основная часть вопроса остается актуальной, можно ли как-нибудь оптимизировать обновление данных?
UPD: поправил при случаи ошибки.

[Rcus]

Все же есть. если произойдет ошибка то буфер не удалится

[kuzulis]

Так нужно или не нужно писать патч?

ЗЫ: мне эта билиотека тоже оч нужна будет! Поэтому и интересуюсь!

ЗЫЗЫ: киньте кто- нить на почту scapig@yandex.ru и на scapig2040@rambler.ru архив с этой библой, т.к у нас на работе админ палит то что кто качает... да и скачать из Windows не имея svn не хочется (по одному файлу)

[spirit]

имхо, нужно, а то как-то с утечкой несерьезно 

[kuzulis]

Ну так исправленную версию и скиньте мне на почту пожалуйста!

[fuCtor]

Отправил, смотрите почту.

[kuzulis]

не, ничего нету... сдублируйте если не трудно и на :  scapig2040@rambler.ru

PS: злостный админ фильтрует