diff options
Diffstat (limited to 'api/logic/minecraft')
| -rw-r--r-- | api/logic/minecraft/ModList.cpp | 7 | ||||
| -rw-r--r-- | api/logic/minecraft/ParseUtils_test.cpp | 2 | ||||
| -rw-r--r-- | api/logic/minecraft/forge/ForgeXzDownload.cpp | 6 | ||||
| -rw-r--r-- | api/logic/minecraft/onesix/OneSixProfileStrategy.cpp | 2 |
4 files changed, 7 insertions, 10 deletions
diff --git a/api/logic/minecraft/ModList.cpp b/api/logic/minecraft/ModList.cpp index 3f28f7d4..02b09eef 100644 --- a/api/logic/minecraft/ModList.cpp +++ b/api/logic/minecraft/ModList.cpp @@ -341,7 +341,6 @@ bool ModList::dropMimeData(const QMimeData* data, Qt::DropAction action, int, in if (data->hasUrls()) { bool was_watching = is_watching; - bool added = false; if (was_watching) { stopWatching(); @@ -355,10 +354,8 @@ bool ModList::dropMimeData(const QMimeData* data, Qt::DropAction action, int, in continue; } // TODO: implement not only copy, but also move - if (installMod(url.toLocalFile())) - { - added = true; - } + // FIXME: handle errors here + installMod(url.toLocalFile()); } if (was_watching) { diff --git a/api/logic/minecraft/ParseUtils_test.cpp b/api/logic/minecraft/ParseUtils_test.cpp index 79bed1d4..1ce96248 100644 --- a/api/logic/minecraft/ParseUtils_test.cpp +++ b/api/logic/minecraft/ParseUtils_test.cpp @@ -22,7 +22,7 @@ slots: "2016-02-10T15:06:41+00:00", "2016-02-04T15:28:02-05:33" }; - for(int i = 0; i < (sizeof(timestamps) / sizeof(const char *)); i++) + for(unsigned i = 0; i < (sizeof(timestamps) / sizeof(const char *)); i++) { QTest::newRow(timestamps[i]) << QString(timestamps[i]); } diff --git a/api/logic/minecraft/forge/ForgeXzDownload.cpp b/api/logic/minecraft/forge/ForgeXzDownload.cpp index 4d94dd12..593aa24f 100644 --- a/api/logic/minecraft/forge/ForgeXzDownload.cpp +++ b/api/logic/minecraft/forge/ForgeXzDownload.cpp @@ -210,7 +210,8 @@ void ForgeXzDownload::decompressAndInstall() if (b.out_pos == sizeof(out)) { - if (pack200_file.write((char *)out, b.out_pos) != b.out_pos) + auto wresult = pack200_file.write((char *)out, b.out_pos); + if (wresult < 0 || size_t(wresult) != b.out_pos) { // msg = "Write error\n"; xz_dec_end(s); @@ -230,7 +231,8 @@ void ForgeXzDownload::decompressAndInstall() continue; } - if (pack200_file.write((char *)out, b.out_pos) != b.out_pos) + auto wresult = pack200_file.write((char *)out, b.out_pos); + if (wresult < 0 || size_t(wresult) != b.out_pos) { // write error pack200_file.close(); diff --git a/api/logic/minecraft/onesix/OneSixProfileStrategy.cpp b/api/logic/minecraft/onesix/OneSixProfileStrategy.cpp index 0b83c2e1..ef2a7294 100644 --- a/api/logic/minecraft/onesix/OneSixProfileStrategy.cpp +++ b/api/logic/minecraft/onesix/OneSixProfileStrategy.cpp @@ -158,7 +158,6 @@ void OneSixProfileStrategy::loadUserPatches() // now add all the patches by user sort order ProfileUtils::PatchOrder userOrder; ProfileUtils::readOverrideOrders(FS::PathCombine(m_instance->instanceRoot(), "order.json"), userOrder); - bool orderIsDirty = false; for (auto uid : userOrder) { // ignore builtins @@ -169,7 +168,6 @@ void OneSixProfileStrategy::loadUserPatches() // ordering has a patch that is gone? if(!loadedPatches.contains(uid)) { - orderIsDirty = true; continue; } profile->appendPatch(loadedPatches.take(uid)); |