0.9.4.1 - Fix windows #define and alarmmodel assert.

docs/RELEASES

@@ -1,3 +1,8 @@
+0.9.4.1 - Fixed bug where #defines were not working with newest qt/gcc.
+          (thanks Hibasnev!)
+
+          Fixed bug where Windows build would assert setting alarms.
+
 0.9.4 - Fixed crash when jump bridges fail to load.  Also suppressed
         the error message, since most systems won't have jump bridges.
 

src/alarmmodel.cpp

@@ -231,9 +231,13 @@ void AlarmModel::setAlarms(QList<Alarm> newAlarms)
     if(newAlarms.count() <= 0)
         return;
 
-    beginRemoveRows(QModelIndex(), 0, alarms.count()-1);
-    alarms.clear();
-    endRemoveRows();
+    if(alarms.count() > 0)
+    {
+        beginRemoveRows(QModelIndex(), 0, alarms.count()-1);
+        alarms.clear();
+        endRemoveRows();
+    }
+
     beginInsertRows(QModelIndex(), 0, newAlarms.count()-1);
     alarms = newAlarms;
     endInsertRows();

src/logcatcher.cpp

@@ -132,8 +132,8 @@ void LogCatcher::findCurrentLogs(const QString& dirName)
                 // are in them.
 
                 QString channelName = logNameRegEx.cap(1);
-                if(m_options->getIntelChannels().contains(channelName))
-                //if(!localChannels.contains(channelName))
+                //if(m_options->getIntelChannels().contains(channelName))
+                if(!localChannels.contains(channelName))
                 {
                     QMutableListIterator<QFileInfo> i(infoList);
                     while (i.hasNext()) {

src/logcatcher.h

@@ -21,18 +21,6 @@
 #ifndef LOGCATCHER_H
 #define LOGCATCHER_H
 
-/* On Linux, QFileSystemWatcher works fine.  On Windows, it works 50% (dirs only) thanks to
- * the way Windows doesn't update the file modified time on flush().  See the bug report at
- * https://bugreports.qt.io/browse/QTBUG-41119 for more details.
- *
- * So... On Windows we currently need to use a timer-based hack to check the individual
- * files.
- */
-
-#ifdef Q_OS_WIN32 // Q_OS_LINUX
-#define USE_FALLBACK_POLLER
-#endif
-
 /* For some reason, actually using the above define in this same file ends up with the
  * QTimer not firing?!?!  At least it does when testing with gcc on Linux.  That's why I
  * do not use the flag below to optimize what's built on Linux.
@@ -47,6 +35,18 @@
 
 #include "options.h"
 
+/* On Linux, QFileSystemWatcher works fine.  On Windows, it works 50% (dirs only) thanks to
+ * the way Windows doesn't update the file modified time on flush().  See the bug report at
+ * https://bugreports.qt.io/browse/QTBUG-41119 for more details.
+ *
+ * So... On Windows we currently need to use a timer-based hack to check the individual
+ * files.
+ */
+
+#ifdef Q_OS_WIN32 // Q_OS_LINUX
+#define USE_FALLBACK_POLLER
+#endif
+
 using namespace std;