Make CodeLoader::installVescPackage() always erase both qml and lisp

If one or the other is missing in the package, we still want to erase
the memory, otherwise the code of the old package will stay there and
keep running.

If a package that contains neither qml nor lisp is loaded, it erases
both. That should be the expected outcome.

Reorganizes the logic a bit to hopefully make it more streamlined.

codeloader.cpp

@@ -787,25 +787,31 @@ bool CodeLoader::installVescPackage(VescPackage pkg)
     bool res = true;
     QByteArray qml;
 
-    if (res && !pkg.qmlFile.isEmpty()) {
+    if (!pkg.qmlFile.isEmpty()) {
         qml = qmlCompress(pkg.qmlFile);
         res = qmlErase(qml.size() + 100);
-    }
 
-    if (res && !pkg.qmlFile.isEmpty()) {
-        res = qmlUpload(qml, pkg.qmlIsFullscreen);
+        if (res) {
+            res = qmlUpload(qml, pkg.qmlIsFullscreen);
+        }
+    } else {
+        res = qmlErase(16);
     }
 
-    if (res && !pkg.lispData.isEmpty()) {
-        res = lispErase(pkg.lispData.size() + 100);
-    }
+    if (res) {
+        if (!pkg.lispData.isEmpty()) {
+            res = lispErase(pkg.lispData.size() + 100);
 
-    if (res && !pkg.lispData.isEmpty()) {
-        res = lispUpload(VByteArray(pkg.lispData));
-    }
+            if (res) {
+                res = lispUpload(VByteArray(pkg.lispData));
 
-    if (res && !pkg.lispData.isEmpty()) {
-        mVesc->commands()->lispSetRunning(1);
+                if (res) {
+                    mVesc->commands()->lispSetRunning(1);
+                }
+            }
+        } else {
+            res = lispErase(16);
+        }
     }
 
     Utility::sleepWithEventLoop(500);