options/fix bundle refcount handling

We rolled up our own refcount while using shared_ptr at the same time.
Remove all rolled-up logic and rely on shared_ptr's custom deleter to do
the cleanups.

This greatly simplifies the code here. Unfortunately, _fini ordering
makes Qt crash after the app object runs out of scope:

gui/main.cpp: QApplication app(argc, argv);

Both things considered, use _exit(2) on Unix to avoid running static
destructors.

1 files changed, 5 insertions, 1 deletions

diff --git a/gui/main.cpp b/gui/main.cpp
index 1972e7a3..d9aba3ce 100644
--- a/gui/main.cpp
+++ b/gui/main.cpp
@@ -5,6 +5,8 @@
 #   include <QCoreApplication>
 #   include <QFile>
 #   include <QString>
+#else
+#   include <unistd.h>
 #endif
 
 #include "migration/migration.hpp"
@@ -175,8 +177,10 @@ main(int argc, char** argv)
 #if defined(_MSC_VER)
     qDebug() << "exit: terminating";
     TerminateProcess(GetCurrentProcess(), 0);
+#else
+    // we have some atexit issues when not leaking bundles
+    //_exit(0);
 #endif
-
     qDebug() << "exit: main()";
 
     return 0;