summaryrefslogtreecommitdiffstats
path: root/logic/net/CacheDownload.cpp
diff options
context:
space:
mode:
authorPetr Mrázek <peterix@gmail.com>2014-01-18 22:11:33 +0100
committerPetr Mrázek <peterix@gmail.com>2014-01-18 22:11:33 +0100
commit3fabb11f4c59baffb14db00d338d9efe342e277e (patch)
treec33d242761d31879f12821daab4d4f714a47b698 /logic/net/CacheDownload.cpp
parent8650aa81f06d9e229764b200f1dca135412c2ec1 (diff)
downloadMultiMC-3fabb11f4c59baffb14db00d338d9efe342e277e.tar
MultiMC-3fabb11f4c59baffb14db00d338d9efe342e277e.tar.gz
MultiMC-3fabb11f4c59baffb14db00d338d9efe342e277e.tar.lz
MultiMC-3fabb11f4c59baffb14db00d338d9efe342e277e.tar.xz
MultiMC-3fabb11f4c59baffb14db00d338d9efe342e277e.zip
Marginally improve OneSix offline mode launch
While reconstructing assets, skip files that don't exist. Report missing OneSix native libraries.
Diffstat (limited to 'logic/net/CacheDownload.cpp')
-rw-r--r--logic/net/CacheDownload.cpp21
1 files changed, 13 insertions, 8 deletions
diff --git a/logic/net/CacheDownload.cpp b/logic/net/CacheDownload.cpp
index 0022c361..d2a9bdee 100644
--- a/logic/net/CacheDownload.cpp
+++ b/logic/net/CacheDownload.cpp
@@ -40,7 +40,9 @@ void CacheDownload::start()
emit succeeded(m_index_within_job);
return;
}
- m_output_file.setFileName(m_target_path);
+ // create a new save file
+ m_output_file.reset(new QSaveFile(m_target_path));
+
// if there already is a file and md5 checking is in effect and it can be opened
if (!ensureFilePathExists(m_target_path))
{
@@ -49,7 +51,7 @@ void CacheDownload::start()
emit failed(m_index_within_job);
return;
}
- if (!m_output_file.open(QIODevice::WriteOnly))
+ if (!m_output_file->open(QIODevice::WriteOnly))
{
QLOG_ERROR() << "Could not open " + m_target_path + " for writing";
m_status = Job_Failed;
@@ -94,7 +96,7 @@ void CacheDownload::downloadProgress(qint64 bytesReceived, qint64 bytesTotal)
void CacheDownload::downloadError(QNetworkReply::NetworkError error)
{
// error happened during download.
- QLOG_ERROR() << "Failed" << m_url.toString() << "with reason" << error;
+ QLOG_ERROR() << "Failed " << m_url.toString() << " with reason " << error;
m_status = Job_Failed;
}
void CacheDownload::downloadFinished()
@@ -102,17 +104,17 @@ void CacheDownload::downloadFinished()
// if the download succeeded
if (m_status == Job_Failed)
{
- m_output_file.cancelWriting();
+ m_output_file->cancelWriting();
m_reply.reset();
- m_status = Job_Failed;
emit failed(m_index_within_job);
return;
}
+ // if we wrote any data to the save file, we try to commit the data to the real file.
if (wroteAnyData)
{
// nothing went wrong...
- if (m_output_file.commit())
+ if (m_output_file->commit())
{
m_status = Job_Finished;
m_entry->md5sum = md5sum.result().toHex().constData();
@@ -120,7 +122,7 @@ void CacheDownload::downloadFinished()
else
{
QLOG_ERROR() << "Failed to commit changes to " << m_target_path;
- m_output_file.cancelWriting();
+ m_output_file->cancelWriting();
m_reply.reset();
m_status = Job_Failed;
emit failed(m_index_within_job);
@@ -132,6 +134,9 @@ void CacheDownload::downloadFinished()
m_status = Job_Finished;
}
+ // then get rid of the save file
+ m_output_file.reset();
+
QFileInfo output_file_info(m_target_path);
m_entry->etag = m_reply->rawHeader("ETag").constData();
@@ -153,7 +158,7 @@ void CacheDownload::downloadReadyRead()
{
QByteArray ba = m_reply->readAll();
md5sum.addData(ba);
- if (m_output_file.write(ba) != ba.size())
+ if (m_output_file->write(ba) != ba.size())
{
QLOG_ERROR() << "Failed writing into " + m_target_path;
m_status = Job_Failed;