Update names of ResourceRequest enums

This commit is contained in:
Ryan Huffman 2015-08-27 22:23:06 -07:00
parent 8f8e0f9624
commit 2a4b25dd76
8 changed files with 35 additions and 35 deletions

View file

@ -29,14 +29,14 @@ void AssetResourceRequest::doSend() {
connect(request, &AssetRequest::progress, this, &AssetResourceRequest::progress); connect(request, &AssetRequest::progress, this, &AssetResourceRequest::progress);
QObject::connect(request, &AssetRequest::finished, [this](AssetRequest* req) mutable { QObject::connect(request, &AssetRequest::finished, [this](AssetRequest* req) mutable {
if (_state != IN_PROGRESS) return; if (_state != InProgress) return;
_state = FINISHED; _state = Finished;
if (true) { if (true) {
_data = req->getData(); _data = req->getData();
_result = ResourceRequest::SUCCESS; _result = ResourceRequest::Success;
emit finished(); emit finished();
} else { } else {
_result = ResourceRequest::ERROR; _result = ResourceRequest::Error;
emit finished(); emit finished();
} }
}); });

View file

@ -16,13 +16,13 @@
void FileResourceRequest::doSend() { void FileResourceRequest::doSend() {
QString filename = _url.toLocalFile(); QString filename = _url.toLocalFile();
QFile file(filename); QFile file(filename);
_state = FINISHED; _state = Finished;
if (file.open(QFile::ReadOnly)) { if (file.open(QFile::ReadOnly)) {
_data = file.readAll(); _data = file.readAll();
_result = ResourceRequest::SUCCESS; _result = ResourceRequest::Success;
emit finished(); emit finished();
} else { } else {
_result = ResourceRequest::ERROR; _result = ResourceRequest::Error;
emit finished(); emit finished();
} }
} }

View file

@ -51,22 +51,22 @@ void HTTPResourceRequest::doSend() {
} }
void HTTPResourceRequest::onRequestFinished() { void HTTPResourceRequest::onRequestFinished() {
Q_ASSERT(_state == IN_PROGRESS); Q_ASSERT(_state == InProgress);
Q_ASSERT(_reply); Q_ASSERT(_reply);
_state = FINISHED; _state = Finished;
auto error = _reply->error(); auto error = _reply->error();
if (error == QNetworkReply::NoError) { if (error == QNetworkReply::NoError) {
_data = _reply->readAll(); _data = _reply->readAll();
_loadedFromCache = _reply->attribute(QNetworkRequest::SourceIsFromCacheAttribute).toBool(); _loadedFromCache = _reply->attribute(QNetworkRequest::SourceIsFromCacheAttribute).toBool();
_result = ResourceRequest::SUCCESS; _result = ResourceRequest::Success;
emit finished(); emit finished();
} else if (error == QNetworkReply::TimeoutError) { } else if (error == QNetworkReply::TimeoutError) {
_result = ResourceRequest::TIMEOUT; _result = ResourceRequest::Timeout;
emit finished(); emit finished();
} else { } else {
_result = ResourceRequest::ERROR; _result = ResourceRequest::Error;
emit finished(); emit finished();
} }
@ -75,7 +75,7 @@ void HTTPResourceRequest::onRequestFinished() {
} }
void HTTPResourceRequest::onDownloadProgress(qint64 bytesReceived, qint64 bytesTotal) { void HTTPResourceRequest::onDownloadProgress(qint64 bytesReceived, qint64 bytesTotal) {
if (_state == IN_PROGRESS) { if (_state == InProgress) {
// We've received data, so reset the timer // We've received data, so reset the timer
_sendTimer.start(); _sendTimer.start();
} }
@ -84,13 +84,13 @@ void HTTPResourceRequest::onDownloadProgress(qint64 bytesReceived, qint64 bytesT
} }
void HTTPResourceRequest::onTimeout() { void HTTPResourceRequest::onTimeout() {
Q_ASSERT(_state != UNSENT); Q_ASSERT(_state != Unsent);
if (_state == IN_PROGRESS) { if (_state == InProgress) {
qCDebug(networking) << "Timed out loading " << _url; qCDebug(networking) << "Timed out loading " << _url;
_reply->abort(); _reply->abort();
_state = FINISHED; _state = Finished;
_result = TIMEOUT; _result = Timeout;
emit finished(); emit finished();
} }
} }

View file

@ -368,7 +368,7 @@ void Resource::handleReplyFinished() {
Q_ASSERT(_request); Q_ASSERT(_request);
auto result = _request->getResult(); auto result = _request->getResult();
if (result == ResourceRequest::SUCCESS) { if (result == ResourceRequest::Success) {
_data = _request->getData(); _data = _request->getData();
qDebug() << "Request finished for " << _url << ", " << _activeUrl; qDebug() << "Request finished for " << _url << ", " << _activeUrl;
@ -386,7 +386,7 @@ void Resource::handleReplyFinished() {
_request->deleteLater(); _request->deleteLater();
_request = nullptr; _request = nullptr;
if (result == ResourceRequest::Result::TIMEOUT) { if (result == ResourceRequest::Result::Timeout) {
qDebug() << "Timed out loading" << _url << "received" << _bytesReceived << "total" << _bytesTotal; qDebug() << "Timed out loading" << _url << "received" << _bytesReceived << "total" << _bytesTotal;
} else { } else {
qDebug() << "Error loading " << _url; qDebug() << "Error loading " << _url;
@ -394,8 +394,8 @@ void Resource::handleReplyFinished() {
bool retry = false; bool retry = false;
switch (result) { switch (result) {
case ResourceRequest::Result::TIMEOUT: case ResourceRequest::Result::Timeout:
case ResourceRequest::Result::ERROR: { case ResourceRequest::Result::Error: {
// retry with increasing delays // retry with increasing delays
const int MAX_ATTEMPTS = 8; const int MAX_ATTEMPTS = 8;
const int BASE_DELAY_MS = 1000; const int BASE_DELAY_MS = 1000;
@ -411,7 +411,7 @@ void Resource::handleReplyFinished() {
break; break;
} }
auto error = result == ResourceRequest::TIMEOUT ? QNetworkReply::TimeoutError : QNetworkReply::UnknownNetworkError; auto error = result == ResourceRequest::Timeout ? QNetworkReply::TimeoutError : QNetworkReply::UnknownNetworkError;
if (!retry) { if (!retry) {
emit failed(error); emit failed(error);

View file

@ -17,8 +17,8 @@ ResourceRequest::ResourceRequest(QObject* parent, const QUrl& url) :
} }
void ResourceRequest::send() { void ResourceRequest::send() {
Q_ASSERT(_state == UNSENT); Q_ASSERT(_state == Unsent);
_state = IN_PROGRESS; _state = InProgress;
doSend(); doSend();
} }

View file

@ -21,16 +21,16 @@ public:
ResourceRequest(QObject* parent, const QUrl& url); ResourceRequest(QObject* parent, const QUrl& url);
enum State { enum State {
UNSENT = 0, Unsent = 0,
IN_PROGRESS, InProgress,
FINISHED Finished
}; };
enum Result { enum Result {
SUCCESS, Success,
ERROR, Error,
TIMEOUT, Timeout,
NOT_FOUND NotFound
}; };
void send(); void send();
@ -50,7 +50,7 @@ protected:
virtual void doSend() = 0; virtual void doSend() = 0;
QUrl _url; QUrl _url;
State _state { UNSENT }; State _state { Unsent };
Result _result; Result _result;
QByteArray _data; QByteArray _data;
bool _cacheEnabled { true }; bool _cacheEnabled { true };

View file

@ -40,7 +40,7 @@ void BatchLoader::start() {
continue; continue;
} }
connect(request, &ResourceRequest::finished, this, [=]() { connect(request, &ResourceRequest::finished, this, [=]() {
if (request->getResult() == ResourceRequest::SUCCESS) { if (request->getResult() == ResourceRequest::Success) {
_data.insert(url, request->getData()); _data.insert(url, request->getData());
} else { } else {
_data.insert(url, QString()); _data.insert(url, QString());

View file

@ -63,7 +63,7 @@ void ScriptCache::scriptDownloaded() {
QList<ScriptUser*> scriptUsers = _scriptUsers.values(url); QList<ScriptUser*> scriptUsers = _scriptUsers.values(url);
_scriptUsers.remove(url); _scriptUsers.remove(url);
if (req->getResult() == ResourceRequest::SUCCESS) { if (req->getResult() == ResourceRequest::Success) {
_scriptCache[url] = req->getData(); _scriptCache[url] = req->getData();
qCDebug(scriptengine) << "Done downloading script at:" << url.toString(); qCDebug(scriptengine) << "Done downloading script at:" << url.toString();