1 files changed, 185 insertions, 144 deletions
diff --git a/src/logbuffer.cpp b/src/logbuffer.cpp
index 5a74d5f5..522ce3c8 100644
--- a/src/logbuffer.cpp
+++ b/src/logbuffer.cpp
@@ -18,212 +18,253 @@ along with Mod Organizer.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 #include "logbuffer.h"
-#include <QDateTime>
+#include <scopeguard.h>
+#include <report.h>
+#include <QMutexLocker>
 #include <QFile>
 #include <QIcon>
-#include <QMutexLocker>
+#include <QDateTime>
 #include <Windows.h>
-#include <report.h>
-#include <scopeguard.h>
 
 using MOBase::reportError;
 
 QScopedPointer<LogBuffer> LogBuffer::s_Instance;
 QMutex LogBuffer::s_Mutex;
 
-LogBuffer::LogBuffer(int messageCount, QtMsgType minMsgType, const QString& outputFileName)
-    : QAbstractItemModel(nullptr), m_OutFileName(outputFileName), m_ShutDown(false), m_MinMsgType(minMsgType),
-      m_NumMessages(0) {
-    m_Messages.resize(messageCount);
+LogBuffer::LogBuffer(int messageCount, QtMsgType minMsgType,
+                     const QString &outputFileName)
+  : QAbstractItemModel(nullptr)
+  , m_OutFileName(outputFileName)
+  , m_ShutDown(false)
+  , m_MinMsgType(minMsgType)
+  , m_NumMessages(0)
+{
+  m_Messages.resize(messageCount);
 }
 
-LogBuffer::~LogBuffer() {
-    qInstallMessageHandler(0);
-    write();
+LogBuffer::~LogBuffer()
+{
+  qInstallMessageHandler(0);
+  write();
 }
 
-void LogBuffer::logMessage(QtMsgType type, const QString& message) {
-    if (type >= m_MinMsgType) {
-        Message msg = {type, QTime::currentTime(), message};
-        if (m_NumMessages < m_Messages.size()) {
-            beginInsertRows(QModelIndex(), static_cast<int>(m_NumMessages), static_cast<int>(m_NumMessages) + 1);
-        }
-        m_Messages.at(m_NumMessages % m_Messages.size()) = msg;
-        if (m_NumMessages < m_Messages.size()) {
-            endInsertRows();
-        } else {
-            emit dataChanged(createIndex(0, 0), createIndex(static_cast<int>(m_Messages.size()), 0));
-        }
-        ++m_NumMessages;
-        if (type >= QtCriticalMsg) {
-            write();
-        }
+void LogBuffer::logMessage(QtMsgType type, const QString &message)
+{
+  if (type >= m_MinMsgType) {
+    Message msg = {type, QTime::currentTime(), message};
+    if (m_NumMessages < m_Messages.size()) {
+      beginInsertRows(QModelIndex(), static_cast<int>(m_NumMessages),
+                      static_cast<int>(m_NumMessages) + 1);
+    }
+    m_Messages.at(m_NumMessages % m_Messages.size()) = msg;
+    if (m_NumMessages < m_Messages.size()) {
+      endInsertRows();
+    } else {
+      emit dataChanged(createIndex(0, 0),
+                       createIndex(static_cast<int>(m_Messages.size()), 0));
+    }
+    ++m_NumMessages;
+    if (type >= QtCriticalMsg) {
+      write();
     }
+  }
 }
 
-void LogBuffer::write() const {
-    if (m_NumMessages == 0) {
-        return;
-    }
+void LogBuffer::write() const
+{
+  if (m_NumMessages == 0) {
+    return;
+  }
 
-    DWORD lastError = ::GetLastError();
+  DWORD lastError = ::GetLastError();
 
-    QFile file(m_OutFileName);
-    if (!file.open(QIODevice::WriteOnly)) {
-        reportError(tr("failed to write log to %1: %2").arg(m_OutFileName).arg(file.errorString()));
-        return;
-    }
+  QFile file(m_OutFileName);
+  if (!file.open(QIODevice::WriteOnly)) {
+    reportError(tr("failed to write log to %1: %2")
+                    .arg(m_OutFileName)
+                    .arg(file.errorString()));
+    return;
+  }
 
-    unsigned int i =
-        (m_NumMessages > m_Messages.size()) ? static_cast<unsigned int>(m_NumMessages - m_Messages.size()) : 0U;
-    for (; i < m_NumMessages; ++i) {
-        file.write(m_Messages.at(i % m_Messages.size()).toString().toUtf8());
-        file.write("\r\n");
-    }
-    ::SetLastError(lastError);
+  unsigned int i
+      = (m_NumMessages > m_Messages.size())
+            ? static_cast<unsigned int>(m_NumMessages - m_Messages.size())
+            : 0U;
+  for (; i < m_NumMessages; ++i) {
+    file.write(m_Messages.at(i % m_Messages.size()).toString().toUtf8());
+    file.write("\r\n");
+  }
+  ::SetLastError(lastError);
 }
 
-void LogBuffer::init(int messageCount, QtMsgType minMsgType, const QString& outputFileName) {
-    QMutexLocker guard(&s_Mutex);
+void LogBuffer::init(int messageCount, QtMsgType minMsgType,
+                     const QString &outputFileName)
+{
+  QMutexLocker guard(&s_Mutex);
 
-    s_Instance.reset(new LogBuffer(messageCount, minMsgType, outputFileName));
-    qInstallMessageHandler(LogBuffer::log);
+  s_Instance.reset(new LogBuffer(messageCount, minMsgType, outputFileName));
+  qInstallMessageHandler(LogBuffer::log);
 }
 
-char LogBuffer::msgTypeID(QtMsgType type) {
-    switch (type) {
+char LogBuffer::msgTypeID(QtMsgType type)
+{
+  switch (type) {
     case QtDebugMsg:
-        return 'D';
+      return 'D';
     case QtWarningMsg:
-        return 'W';
+      return 'W';
     case QtCriticalMsg:
-        return 'C';
+      return 'C';
     case QtFatalMsg:
-        return 'F';
+      return 'F';
     default:
-        return '?';
-    }
+      return '?';
+  }
 }
 
-void LogBuffer::log(QtMsgType type, const QMessageLogContext& context, const QString& message) {
-    // QMutexLocker doesn't support timeout...
-    if (!s_Mutex.tryLock(100)) {
-        fprintf(stderr, "failed to log: %s", qPrintable(message));
-        return;
-    }
-    ON_BLOCK_EXIT([]() { s_Mutex.unlock(); });
+void LogBuffer::log(QtMsgType type, const QMessageLogContext &context,
+                    const QString &message)
+{
+  // QMutexLocker doesn't support timeout...
+  if (!s_Mutex.tryLock(100)) {
+    fprintf(stderr, "failed to log: %s", qPrintable(message));
+    return;
+  }
+  ON_BLOCK_EXIT([]() { s_Mutex.unlock(); });
 
-    if (!s_Instance.isNull()) {
-        s_Instance->logMessage(type, message);
-    }
+  if (!s_Instance.isNull()) {
+    s_Instance->logMessage(type, message);
+  }
 
-    if (type == QtDebugMsg) {
-        fprintf(stdout, "%s [%c] %s\n", qPrintable(QTime::currentTime().toString()), msgTypeID(type),
-                qPrintable(message));
+  if (type == QtDebugMsg) {
+    fprintf(stdout, "%s [%c] %s\n", qPrintable(QTime::currentTime().toString()),
+            msgTypeID(type), qPrintable(message));
+  } else {
+    if (context.line != 0) {
+      fprintf(stdout, "%s [%c] (%s:%u) %s\n",
+              qPrintable(QTime::currentTime().toString()), msgTypeID(type),
+              context.file, context.line, qPrintable(message));
     } else {
-        if (context.line != 0) {
-            fprintf(stdout, "%s [%c] (%s:%u) %s\n", qPrintable(QTime::currentTime().toString()), msgTypeID(type),
-                    context.file, context.line, qPrintable(message));
-        } else {
-            fprintf(stdout, "%s [%c] %s\n", qPrintable(QTime::currentTime().toString()), msgTypeID(type),
-                    qPrintable(message));
-        }
+      fprintf(stdout, "%s [%c] %s\n",
+              qPrintable(QTime::currentTime().toString()), msgTypeID(type),
+              qPrintable(message));
     }
-    fflush(stdout);
+  }
+  fflush(stdout);
 }
 
-QModelIndex LogBuffer::index(int row, int column, const QModelIndex&) const { return createIndex(row, column, row); }
+QModelIndex LogBuffer::index(int row, int column, const QModelIndex &) const
+{
+  return createIndex(row, column, row);
+}
 
-QModelIndex LogBuffer::parent(const QModelIndex&) const { return QModelIndex(); }
+QModelIndex LogBuffer::parent(const QModelIndex &) const
+{
+  return QModelIndex();
+}
 
-int LogBuffer::rowCount(const QModelIndex& parent) const {
-    if (parent.isValid())
-        return 0;
-    else
-        return static_cast<int>(std::min(m_NumMessages, m_Messages.size()));
+int LogBuffer::rowCount(const QModelIndex &parent) const
+{
+  if (parent.isValid())
+    return 0;
+  else
+    return static_cast<int>(std::min(m_NumMessages, m_Messages.size()));
 }
 
-int LogBuffer::columnCount(const QModelIndex&) const { return 2; }
+int LogBuffer::columnCount(const QModelIndex &) const
+{
+  return 2;
+}
 
-QVariant LogBuffer::data(const QModelIndex& index, int role) const {
-    unsigned int offset =
-        m_NumMessages < m_Messages.size() ? 0 : static_cast<unsigned int>(m_NumMessages - m_Messages.size());
-    unsigned int msgIndex = (offset + index.row() + 1) % m_Messages.size();
-    switch (role) {
+QVariant LogBuffer::data(const QModelIndex &index, int role) const
+{
+  unsigned int offset
+      = m_NumMessages < m_Messages.size()
+            ? 0
+            : static_cast<unsigned int>(m_NumMessages - m_Messages.size());
+  unsigned int msgIndex = (offset + index.row() + 1) % m_Messages.size();
+  switch (role) {
     case Qt::DisplayRole: {
-        if (index.column() == 0) {
-            return m_Messages[msgIndex].time;
-        } else if (index.column() == 1) {
-            const QString& msg = m_Messages[msgIndex].message;
-            if (msg.length() < 200) {
-                return msg;
-            } else {
-                return msg.mid(0, 200) + "...";
-            }
+      if (index.column() == 0) {
+        return m_Messages[msgIndex].time;
+      } else if (index.column() == 1) {
+        const QString &msg = m_Messages[msgIndex].message;
+        if (msg.length() < 200) {
+          return msg;
+        } else {
+          return msg.mid(0, 200) + "...";
         }
+      }
     } break;
     case Qt::DecorationRole: {
-        if (index.column() == 1) {
-            switch (m_Messages[msgIndex].type) {
-            case QtDebugMsg:
-                return QIcon(":/MO/gui/information");
-            case QtWarningMsg:
-                return QIcon(":/MO/gui/warning");
-            case QtCriticalMsg:
-                return QIcon(":/MO/gui/important");
-            case QtFatalMsg:
-                return QIcon(":/MO/gui/problem");
-            }
+      if (index.column() == 1) {
+        switch (m_Messages[msgIndex].type) {
+          case QtDebugMsg:
+            return QIcon(":/MO/gui/information");
+          case QtWarningMsg:
+            return QIcon(":/MO/gui/warning");
+          case QtCriticalMsg:
+            return QIcon(":/MO/gui/important");
+          case QtFatalMsg:
+            return QIcon(":/MO/gui/problem");
         }
+      }
     } break;
     case Qt::UserRole: {
-        if (index.column() == 1) {
-            switch (m_Messages[msgIndex].type) {
-            case QtDebugMsg:
-                return "D";
-            case QtWarningMsg:
-                return "W";
-            case QtCriticalMsg:
-                return "C";
-            case QtFatalMsg:
-                return "F";
-            }
+      if (index.column() == 1) {
+        switch (m_Messages[msgIndex].type) {
+          case QtDebugMsg:
+            return "D";
+          case QtWarningMsg:
+            return "W";
+          case QtCriticalMsg:
+            return "C";
+          case QtFatalMsg:
+            return "F";
         }
+      }
     } break;
-    }
-    return QVariant();
+  }
+  return QVariant();
 }
 
-void LogBuffer::writeNow() {
-    QMutexLocker guard(&s_Mutex);
-    if (!s_Instance.isNull()) {
-        s_Instance->write();
-    }
+void LogBuffer::writeNow()
+{
+  QMutexLocker guard(&s_Mutex);
+  if (!s_Instance.isNull()) {
+    s_Instance->write();
+  }
 }
 
-void LogBuffer::cleanQuit() {
-    QMutexLocker guard(&s_Mutex);
-    if (!s_Instance.isNull()) {
-        s_Instance->m_ShutDown = true;
-    }
+void LogBuffer::cleanQuit()
+{
+  QMutexLocker guard(&s_Mutex);
+  if (!s_Instance.isNull()) {
+    s_Instance->m_ShutDown = true;
+  }
 }
 
-void log(const char* format, ...) {
-    va_list argList;
-    va_start(argList, format);
+void log(const char *format, ...)
+{
+  va_list argList;
+  va_start(argList, format);
 
-    static const int BUFFERSIZE = 1000;
+  static const int BUFFERSIZE = 1000;
 
-    char buffer[BUFFERSIZE + 1];
-    buffer[BUFFERSIZE] = '\0';
+  char buffer[BUFFERSIZE + 1];
+  buffer[BUFFERSIZE] = '\0';
 
-    vsnprintf(buffer, BUFFERSIZE, format, argList);
+  vsnprintf(buffer, BUFFERSIZE, format, argList);
 
-    qCritical("%s", buffer);
+  qCritical("%s", buffer);
 
-    va_end(argList);
+  va_end(argList);
 }
 
-QString LogBuffer::Message::toString() const {
-    return QString("%1 [%2] %3").arg(time.toString()).arg(msgTypeID(type)).arg(message);
+QString LogBuffer::Message::toString() const
+{
+  return QString("%1 [%2] %3")
+      .arg(time.toString())
+      .arg(msgTypeID(type))
+      .arg(message);
 }