From ce097235f2013909d50046db13fa97fd202b1bfb Mon Sep 17 00:00:00 2001
From: TheAssassin <theassassin@assassinate-you.net>
Date: Fri, 23 Nov 2018 18:05:56 +0100
Subject: [PATCH] Note for self: -O1+ strips out expressions surrounded by
 assert()

---
 src/core/desktopfile/desktopfile.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/core/desktopfile/desktopfile.cpp b/src/core/desktopfile/desktopfile.cpp
index d66c6e6..87d4ce2 100644
--- a/src/core/desktopfile/desktopfile.cpp
+++ b/src/core/desktopfile/desktopfile.cpp
@@ -173,12 +173,14 @@ namespace linuxdeploy {
                         DesktopFileEntry entry;
 
                         // this should never return false
-                        assert(getEntry(section, key, entry));
+                        auto entryExists = getEntry(section, key, entry);
+                        assert(entryExists);
 
                         ldLog() << LD_WARNING << "Key exists, not modified:" << key << "(current value:" << entry.value() << LD_NO_SPACE << ")" << std::endl;
                         rv = false;
                     } else {
-                        assert(!setEntry(section, std::move(DesktopFileEntry(key, value))));
+                        auto entrySet = setEntry(section, DesktopFileEntry(key, value));
+                        assert(entrySet);
                     }
                 };