/[svn]/qsampler/trunk/src/qsamplerMainForm.cpp

Diff of /qsampler/trunk/src/qsamplerMainForm.cpp

Parent Directory | Revision Log | View Patch Patch

-revision 2459 by capela,
Mon Jul  8 10:06:57 2013 UTC
+revision 3681 by capela,
Thu Jan  2 14:39:02 2020 UTC
 Line 1
  // qsamplerMainForm.cpp
  //
  /****************************************************************************
-    Copyright (C) 2004-2013, rncbc aka Rui Nuno Capela. All rights reserved.
+    Copyright (C) 2004-2020, rncbc aka Rui Nuno Capela. All rights reserved.
-    Copyright (C) 2007, 2008 Christian Schoenebeck
+    Copyright (C) 2007-2019 Christian Schoenebeck
     This program is free software; you can redistribute it and/or
     modify it under the terms of the GNU General Public License
 Line 58
  #include <QTimer>
  #include <QDateTime>
- #if QT_VERSION >= 0x050000
+ #if QT_VERSION >= QT_VERSION_CHECK(5, 0, 0)
  #include <QMimeData>
  #endif
- #if QT_VERSION < 0x040500
+ #if QT_VERSION < QT_VERSION_CHECK(4, 5, 0)
  namespace Qt {
  const WindowFlags WindowCloseButtonHint = WindowFlags(0x08000000);
  }
  #endif
- #ifdef HAVE_SIGNAL_H
- #include <signal.h>
- #endif
  #ifdef CONFIG_LIBGIG
  #include <gig.h>
  #endif
-Line 91 
 static inline long lroundf ( float x )
+Line 87 
 static inline long lroundf ( float x )
  // All winsock apps needs this.
- #if defined(WIN32)
+ #if defined(__WIN32__) || defined(_WIN32) || defined(WIN32)
  static WSADATA _wsaData;
+ #undef HAVE_SIGNAL_H
  #endif
-Line 103 
 static WSADATA _wsaData;
+Line 100 
 static WSADATA _wsaData;
  #include <QSocketNotifier>
+ #include <unistd.h>
  #include <sys/types.h>
  #include <sys/socket.h>
  #include <signal.h>
  // File descriptor for SIGUSR1 notifier.
- static int g_fdUsr1[2];
+ static int g_fdSigusr1[2] = { -1, -1 };
  // Unix SIGUSR1 signal handler.
  static void qsampler_sigusr1_handler ( int /* signo */ )
  {
          char c = 1;
-         (::write(g_fdUsr1[0], &c, sizeof(c)) > 0);
+         (::write(g_fdSigusr1[0], &c, sizeof(c)) > 0);
+ }
+ // File descriptor for SIGTERM notifier.
+ static int g_fdSigterm[2] = { -1, -1 };
+ // Unix SIGTERM signal handler.
+ static void qsampler_sigterm_handler ( int /* signo */ )
+ {
+         char c = 1;
+         (::write(g_fdSigterm[0], &c, sizeof(c)) > 0);
  }
  #endif  // HAVE_SIGNAL_H
  //-------------------------------------------------------------------------
- // qsampler -- namespace
+ // QSampler -- namespace
  namespace QSampler {
-Line 143 
 namespace QSampler {
+Line 151 
 namespace QSampler {
  //-------------------------------------------------------------------------
- // LscpEvent -- specialty for LSCP callback comunication.
+ // QSampler::LscpEvent -- specialty for LSCP callback comunication.
  class LscpEvent : public QEvent
  {
-Line 159 
 public:
+Line 166 
 public:
          }
          // Accessors.
-         lscp_event_t event() { return m_event; }
+         lscp_event_t  event() { return m_event; }
-         QString&     data()  { return m_data;  }
+         const QString& data() { return m_data;  }
  private:
-Line 172 
 private:
+Line 179 
 private:
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- Main window form implementation.
+ // QSampler::Workspace -- Main window workspace (MDI Area) decl.
+ class Workspace : public QMdiArea
+ {
+ public:
+         Workspace(MainForm *pMainForm) : QMdiArea(pMainForm) {}
+ protected:
+         void resizeEvent(QResizeEvent *)
+         {
+                 MainForm *pMainForm = static_cast<MainForm *> (parentWidget());
+                 if (pMainForm)
+                         pMainForm->channelsArrangeAuto();
+         }
+ };
+ //-------------------------------------------------------------------------
+ // QSampler::MainForm -- Main window form implementation.
  // Kind of singleton reference.
- MainForm* MainForm::g_pMainForm = NULL;
+ MainForm *MainForm::g_pMainForm = nullptr;
  MainForm::MainForm ( QWidget *pParent )
          : QMainWindow(pParent)
-Line 186 
 MainForm::MainForm ( QWidget *pParent )
+Line 213 
 MainForm::MainForm ( QWidget *pParent )
          g_pMainForm = this;
          // Initialize some pointer references.
-         m_pOptions = NULL;
+         m_pOptions = nullptr;
          // All child forms are to be created later, not earlier than setup.
-         m_pMessages = NULL;
+         m_pMessages = nullptr;
-         m_pInstrumentListForm = NULL;
+         m_pInstrumentListForm = nullptr;
-         m_pDeviceForm = NULL;
+         m_pDeviceForm = nullptr;
          // We'll start clean.
          m_iUntitled   = 0;
+         m_iDirtySetup = 0;
          m_iDirtyCount = 0;
-         m_pServer = NULL;
+         m_pServer = nullptr;
-         m_pClient = NULL;
+         m_pClient = nullptr;
          m_iStartDelay = 0;
          m_iTimerDelay = 0;
-Line 213 
 MainForm::MainForm ( QWidget *pParent )
+Line 241 
 MainForm::MainForm ( QWidget *pParent )
          // LADISH Level 1 suport.
          // Initialize file descriptors for SIGUSR1 socket notifier.
-         ::socketpair(AF_UNIX, SOCK_STREAM, 0, g_fdUsr1);
+         ::socketpair(AF_UNIX, SOCK_STREAM, 0, g_fdSigusr1);
-         m_pUsr1Notifier
+         m_pSigusr1Notifier
-                 = new QSocketNotifier(g_fdUsr1[1], QSocketNotifier::Read, this);
+                 = new QSocketNotifier(g_fdSigusr1[1], QSocketNotifier::Read, this);
-         QObject::connect(m_pUsr1Notifier,
+         QObject::connect(m_pSigusr1Notifier,
                  SIGNAL(activated(int)),
                  SLOT(handle_sigusr1()));
          // Install SIGUSR1 signal handler.
-     struct sigaction usr1;
+         struct sigaction sigusr1;
-     usr1.sa_handler = qsampler_sigusr1_handler;
+         sigusr1.sa_handler = qsampler_sigusr1_handler;
-     sigemptyset(&usr1.sa_mask);
+         sigemptyset(&sigusr1.sa_mask);
-     usr1.sa_flags = 0;
+         sigusr1.sa_flags = 0;
-     usr1.sa_flags |= SA_RESTART;
+         sigusr1.sa_flags |= SA_RESTART;
-     ::sigaction(SIGUSR1, &usr1, NULL);
+         ::sigaction(SIGUSR1, &sigusr1, nullptr);
+         // Initialize file descriptors for SIGTERM socket notifier.
+         ::socketpair(AF_UNIX, SOCK_STREAM, 0, g_fdSigterm);
+         m_pSigtermNotifier
+                 = new QSocketNotifier(g_fdSigterm[1], QSocketNotifier::Read, this);
+         QObject::connect(m_pSigtermNotifier,
+                 SIGNAL(activated(int)),
+                 SLOT(handle_sigterm()));
+         // Install SIGTERM signal handler.
+         struct sigaction sigterm;
+         sigterm.sa_handler = qsampler_sigterm_handler;
+         sigemptyset(&sigterm.sa_mask);
+         sigterm.sa_flags = 0;
+         sigterm.sa_flags |= SA_RESTART;
+         ::sigaction(SIGTERM, &sigterm, nullptr);
+         ::sigaction(SIGQUIT, &sigterm, nullptr);
+         // Ignore SIGHUP/SIGINT signals.
+         ::signal(SIGHUP, SIG_IGN);
+         ::signal(SIGINT, SIG_IGN);
  #else   // HAVE_SIGNAL_H
-         m_pUsr1Notifier = NULL;
+         m_pSigusr1Notifier = nullptr;
+         m_pSigtermNotifier = nullptr;
  #endif  // !HAVE_SIGNAL_H
-Line 254 
 MainForm::MainForm ( QWidget *pParent )
+Line 305 
 MainForm::MainForm ( QWidget *pParent )
          QObject::connect(m_pVolumeSlider,
                  SIGNAL(valueChanged(int)),
                  SLOT(volumeChanged(int)));
-         //m_ui.channelsToolbar->setHorizontallyStretchable(true);
-         //m_ui.channelsToolbar->setStretchableWidget(m_pVolumeSlider);
          m_ui.channelsToolbar->addWidget(m_pVolumeSlider);
          // Volume spin-box
          m_ui.channelsToolbar->addSeparator();
          m_pVolumeSpinBox = new QSpinBox(m_ui.channelsToolbar);
-         m_pVolumeSpinBox->setMaximumHeight(24);
          m_pVolumeSpinBox->setSuffix(" %");
          m_pVolumeSpinBox->setMinimum(0);
          m_pVolumeSpinBox->setMaximum(100);
-Line 272 
 MainForm::MainForm ( QWidget *pParent )
+Line 320 
 MainForm::MainForm ( QWidget *pParent )
  #endif
          // Make it an MDI workspace.
-         m_pWorkspace = new QMdiArea(this);
+         m_pWorkspace = new Workspace(this);
          m_pWorkspace->setVerticalScrollBarPolicy(Qt::ScrollBarAsNeeded);
          m_pWorkspace->setHorizontalScrollBarPolicy(Qt::ScrollBarAsNeeded);
          // Set the activation connection.
-Line 307 
 MainForm::MainForm ( QWidget *pParent )
+Line 355 
 MainForm::MainForm ( QWidget *pParent )
          m_statusItem[QSAMPLER_STATUS_SESSION] = pLabel;
          statusBar()->addWidget(pLabel);
- #if defined(WIN32)
+ #if defined(__WIN32__) || defined(_WIN32) || defined(WIN32)
          WSAStartup(MAKEWORD(1, 1), &_wsaData);
  #endif
-Line 395 
 MainForm::MainForm ( QWidget *pParent )
+Line 443 
 MainForm::MainForm ( QWidget *pParent )
          QObject::connect(m_ui.channelsMenu,
                  SIGNAL(aboutToShow()),
                  SLOT(channelsMenuAboutToShow()));
+ #ifdef CONFIG_VOLUME
+         QObject::connect(m_ui.channelsToolbar,
+                 SIGNAL(orientationChanged(Qt::Orientation)),
+                 SLOT(channelsToolbarOrientation(Qt::Orientation)));
+ #endif
  }
  // Destructor.
-Line 403 
 MainForm::~MainForm()
+Line 456 
 MainForm::~MainForm()
          // Do final processing anyway.
          processServerExit();
- #if defined(WIN32)
+ #if defined(__WIN32__) || defined(_WIN32) || defined(WIN32)
          WSACleanup();
  #endif
  #if defined(HAVE_SIGNAL_H) && defined(HAVE_SYS_SOCKET_H)
-         if (m_pUsr1Notifier)
+         if (m_pSigusr1Notifier)
-                 delete m_pUsr1Notifier;
+                 delete m_pSigusr1Notifier;
+         if (m_pSigtermNotifier)
+                 delete m_pSigtermNotifier;
  #endif
          // Finally drop any widgets around...
-Line 438 
 MainForm::~MainForm()
+Line 493 
 MainForm::~MainForm()
  #endif
          // Pseudo-singleton reference shut-down.
-         g_pMainForm = NULL;
+         g_pMainForm = nullptr;
  }
-Line 476 
 void MainForm::setup ( Options *pOptions
+Line 531 
 void MainForm::setup ( Options *pOptions
          updateMessagesFont();
          updateMessagesLimit();
          updateMessagesCapture();
          // Set the visibility signal.
          QObject::connect(m_pMessages,
                  SIGNAL(visibilityChanged(bool)),
-Line 541 
 bool MainForm::queryClose (void)
+Line 597 
 bool MainForm::queryClose (void)
                                  || m_ui.channelsToolbar->isVisible());
                          m_pOptions->bStatusbar = statusBar()->isVisible();
                          // Save the dock windows state.
-                         const QString sDockables = saveState().toBase64().data();
                          m_pOptions->settings().setValue("/Layout/DockWindows", saveState());
                          // And the children, and the main windows state,.
                          m_pOptions->saveWidgetGeometry(m_pDeviceForm);
-Line 575 
 void MainForm::closeEvent ( QCloseEvent
+Line 630 
 void MainForm::closeEvent ( QCloseEvent
  void MainForm::dragEnterEvent ( QDragEnterEvent* pDragEnterEvent )
  {
          // Accept external drags only...
-         if (pDragEnterEvent->source() == NULL
+         if (pDragEnterEvent->source() == nullptr
                  && pDragEnterEvent->mimeData()->hasUrls()) {
                  pDragEnterEvent->accept();
          } else {
-Line 584 
 void MainForm::dragEnterEvent ( QDragEnt
+Line 639 
 void MainForm::dragEnterEvent ( QDragEnt
  }
- void MainForm::dropEvent ( QDropEvent* pDropEvent )
+ void MainForm::dropEvent ( QDropEvent *pDropEvent )
  {
          // Accept externally originated drops only...
          if (pDropEvent->source())
-Line 599 
 void MainForm::dropEvent ( QDropEvent* p
+Line 654 
 void MainForm::dropEvent ( QDropEvent* p
                          if (QFileInfo(sPath).exists()) {
                                  // Try to create a new channel from instrument file...
                                  Channel *pChannel = new Channel();
-                                 if (pChannel == NULL)
+                                 if (pChannel == nullptr)
                                          return;
                                  // Start setting the instrument filename...
                                  pChannel->setInstrument(sPath, 0);
-Line 640 
 void MainForm::customEvent ( QEvent* pEv
+Line 695 
 void MainForm::customEvent ( QEvent* pEv
                                  updateAllChannelStrips(true);
                                  break;
                          case LSCP_EVENT_CHANNEL_INFO: {
-                                 int iChannelID = pLscpEvent->data().toInt();
+                                 const int iChannelID = pLscpEvent->data().toInt();
                                  ChannelStrip *pChannelStrip = channelStrip(iChannelID);
                                  if (pChannelStrip)
                                          channelStripChanged(pChannelStrip);
-Line 699 
 void MainForm::handle_sigusr1 (void)
+Line 754 
 void MainForm::handle_sigusr1 (void)
          char c;
-         if (::read(g_fdUsr1[1], &c, sizeof(c)) > 0)
+         if (::read(g_fdSigusr1[1], &c, sizeof(c)) > 0)
                  saveSession(false);
  #endif
  }
+ void MainForm::handle_sigterm (void)
+ {
+ #if defined(HAVE_SIGNAL_H) && defined(HAVE_SYS_SOCKET_H)
+         char c;
+         if (::read(g_fdSigterm[1], &c, sizeof(c)) > 0)
+                 close();
+ #endif
+ }
  void MainForm::updateViewMidiDeviceStatusMenu (void)
  {
          m_ui.viewMidiDeviceStatusMenu->clear();
-Line 731 
 void MainForm::contextMenuEvent( QContex
+Line 799 
 void MainForm::contextMenuEvent( QContex
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- Brainless public property accessors.
+ // QSampler::MainForm -- Brainless public property accessors.
  // The global options settings property.
  Options *MainForm::options (void) const
-Line 755 
 MainForm *MainForm::getInstance (void)
+Line 823 
 MainForm *MainForm::getInstance (void)
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- Session file stuff.
+ // QSampler::MainForm -- Session file stuff.
  // Format the displayable session filename.
  QString MainForm::sessionName ( const QString& sFilename )
  {
-         bool bCompletePath = (m_pOptions && m_pOptions->bCompletePath);
+         const bool bCompletePath = (m_pOptions && m_pOptions->bCompletePath);
          QString sSessionName = sFilename;
          if (sSessionName.isEmpty())
                  sSessionName = tr("Untitled") + QString::number(m_iUntitled);
-Line 784 
 bool MainForm::newSession (void)
+Line 852 
 bool MainForm::newSession (void)
          m_iUntitled++;
          // Stabilize form.
-         m_sFilename = QString::null;
+         m_sFilename = QString();
          m_iDirtyCount = 0;
          appendMessages(tr("New session: \"%1\".").arg(sessionName(m_sFilename)));
          stabilizeForm();
-Line 796 
 bool MainForm::newSession (void)
+Line 864 
 bool MainForm::newSession (void)
  // Open an existing sampler session.
  bool MainForm::openSession (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return false;
          // Ask for the filename to open...
          QString sFilename = QFileDialog::getOpenFileName(this,
-                 QSAMPLER_TITLE ": " + tr("Open Session"), // Caption.
+                 tr("Open Session"),                       // Caption.
                  m_pOptions->sSessionDir,                  // Start here.
                  tr("LSCP Session files") + " (*.lscp)"    // Filter (LSCP files)
          );
-Line 822 
 bool MainForm::openSession (void)
+Line 890 
 bool MainForm::openSession (void)
  // Save current sampler session with another name.
  bool MainForm::saveSession ( bool bPrompt )
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return false;
          QString sFilename = m_sFilename;
-Line 834 
 bool MainForm::saveSession ( bool bPromp
+Line 902 
 bool MainForm::saveSession ( bool bPromp
                          sFilename = m_pOptions->sSessionDir;
                  // Prompt the guy...
                  sFilename = QFileDialog::getSaveFileName(this,
-                         QSAMPLER_TITLE ": " + tr("Save Session"), // Caption.
+                         tr("Save Session"),                       // Caption.
                          sFilename,                                // Start here.
                          tr("LSCP Session files") + " (*.lscp)"    // Filter (LSCP files)
                  );
-Line 844 
 bool MainForm::saveSession ( bool bPromp
+Line 912 
 bool MainForm::saveSession ( bool bPromp
                  // Enforce .lscp extension...
                  if (QFileInfo(sFilename).suffix().isEmpty())
                          sFilename += ".lscp";
+         #if 0
                  // Check if already exists...
                  if (sFilename != m_sFilename && QFileInfo(sFilename).exists()) {
                          if (QMessageBox::warning(this,
-                                 QSAMPLER_TITLE ": " + tr("Warning"),
+                                 tr("Warning"),
                                  tr("The file already exists:\n\n"
                                  "\"%1\"\n\n"
                                  "Do you want to replace it?")
-Line 856 
 bool MainForm::saveSession ( bool bPromp
+Line 925 
 bool MainForm::saveSession ( bool bPromp
                                  == QMessageBox::No)
                                  return false;
                  }
+         #endif
          }
          // Save it right away.
-Line 871 
 bool MainForm::closeSession ( bool bForc
+Line 941 
 bool MainForm::closeSession ( bool bForc
          // Are we dirty enough to prompt it?
          if (m_iDirtyCount > 0) {
                  switch (QMessageBox::warning(this,
-                         QSAMPLER_TITLE ": " + tr("Warning"),
+                         tr("Warning"),
                          tr("The current session has been changed:\n\n"
                          "\"%1\"\n\n"
                          "Do you want to save the changes?")
-Line 894 
 bool MainForm::closeSession ( bool bForc
+Line 964 
 bool MainForm::closeSession ( bool bForc
          if (bClose) {
                  // Remove all channel strips from sight...
                  m_pWorkspace->setUpdatesEnabled(false);
-                 QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+                 const QList<QMdiSubWindow *>& wlist
-                 for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+                         = m_pWorkspace->subWindowList();
-                         ChannelStrip *pChannelStrip = NULL;
+                 foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                         QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+                         ChannelStrip *pChannelStrip
-                         if (pMdiSubWindow)
+                                 = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
-                                 pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
                          if (pChannelStrip) {
                                  Channel *pChannel = pChannelStrip->channel();
                                  if (bForce && pChannel)
                                          pChannel->removeChannel();
                                  delete pChannelStrip;
                          }
-                         if (pMdiSubWindow)
+                         delete pMdiSubWindow;
-                                 delete pMdiSubWindow;
                  }
                  m_pWorkspace->setUpdatesEnabled(true);
                  // We're now clean, for sure.
-Line 921 
 bool MainForm::closeSession ( bool bForc
+Line 989 
 bool MainForm::closeSession ( bool bForc
  // Load a session from specific file path.
  bool MainForm::loadSessionFile ( const QString& sFilename )
  {
-         if (m_pClient == NULL)
+         if (m_pClient == nullptr)
                  return false;
          // Open and read from real file.
-Line 997 
 bool MainForm::loadSessionFile ( const Q
+Line 1065 
 bool MainForm::loadSessionFile ( const Q
  // Save current session to specific file path.
  bool MainForm::saveSessionFile ( const QString& sFilename )
  {
-         if (m_pClient == NULL)
+         if (m_pClient == nullptr)
                  return false;
          // Check whether server is apparently OK...
-Line 1019 
 bool MainForm::saveSessionFile ( const Q
+Line 1087 
 bool MainForm::saveSessionFile ( const Q
          QApplication::setOverrideCursor(QCursor(Qt::WaitCursor));
          // Write the file.
-         int  iErrors = 0;
+         int iErrors = 0;
          QTextStream ts(&file);
          ts << "# " << QSAMPLER_TITLE " - " << tr(QSAMPLER_SUBTITLE) << endl;
-         ts << "# " << tr("Version")
+         ts << "# " << tr("Version") << ": " CONFIG_BUILD_VERSION << endl;
-         << ": " QSAMPLER_VERSION << endl;
+ //      ts << "# " << tr("Build") << ": " CONFIG_BUILD_DATE << endl;
-         ts << "# " << tr("Build")
-         << ": " __DATE__ " " __TIME__ << endl;
          ts << "#"  << endl;
          ts << "# " << tr("File")
          << ": " << QFileInfo(sFilename).fileName() << endl;
-Line 1038 
 bool MainForm::saveSessionFile ( const Q
+Line 1104 
 bool MainForm::saveSessionFile ( const Q
          // It is assumed that this new kind of device+session file
          // will be loaded from a complete initialized server...
          int *piDeviceIDs;
-         int  iDevice;
+         int  i, iDevice;
          ts << "RESET" << endl;
          // Audio device mapping.
-         QMap<int, int> audioDeviceMap;
+         QMap<int, int> audioDeviceMap; iDevice = 0;
          piDeviceIDs = Device::getDevices(m_pClient, Device::Audio);
-         for (iDevice = 0; piDeviceIDs && piDeviceIDs[iDevice] >= 0; iDevice++) {
+         for (i = 0; piDeviceIDs && piDeviceIDs[i] >= 0; ++i) {
-                 ts << endl;
+                 Device device(Device::Audio, piDeviceIDs[i]);
-                 Device device(Device::Audio, piDeviceIDs[iDevice]);
+                 // Avoid plug-in driver devices...
+                 if (device.driverName().toUpper() == "PLUGIN")
+                         continue;
                  // Audio device specification...
+                 ts << endl;
                  ts << "# " << device.deviceTypeName() << " " << device.driverName()
                          << " " << tr("Device") << " " << iDevice << endl;
                  ts << "CREATE AUDIO_OUTPUT_DEVICE " << device.driverName();
-Line 1078 
 bool MainForm::saveSessionFile ( const Q
+Line 1147 
 bool MainForm::saveSessionFile ( const Q
                          iPort++;
                  }
                  // Audio device index/id mapping.
-                 audioDeviceMap[device.deviceID()] = iDevice;
+                 audioDeviceMap.insert(device.deviceID(), iDevice++);
                  // Try to keep it snappy :)
                  QApplication::processEvents(QEventLoop::ExcludeUserInputEvents);
          }
          // MIDI device mapping.
-         QMap<int, int> midiDeviceMap;
+         QMap<int, int> midiDeviceMap; iDevice = 0;
          piDeviceIDs = Device::getDevices(m_pClient, Device::Midi);
-         for (iDevice = 0; piDeviceIDs && piDeviceIDs[iDevice] >= 0; iDevice++) {
+         for (i = 0; piDeviceIDs && piDeviceIDs[i] >= 0; ++i) {
-                 ts << endl;
+                 Device device(Device::Midi, piDeviceIDs[i]);
-                 Device device(Device::Midi, piDeviceIDs[iDevice]);
+                 // Avoid plug-in driver devices...
+                 if (device.driverName().toUpper() == "PLUGIN")
+                         continue;
                  // MIDI device specification...
+                 ts << endl;
                  ts << "# " << device.deviceTypeName() << " " << device.driverName()
                          << " " << tr("Device") << " " << iDevice << endl;
                  ts << "CREATE MIDI_INPUT_DEVICE " << device.driverName();
-Line 1120 
 bool MainForm::saveSessionFile ( const Q
+Line 1192 
 bool MainForm::saveSessionFile ( const Q
                          iPort++;
                  }
                  // MIDI device index/id mapping.
-                 midiDeviceMap[device.deviceID()] = iDevice;
+                 midiDeviceMap.insert(device.deviceID(), iDevice++);
                  // Try to keep it snappy :)
                  QApplication::processEvents(QEventLoop::ExcludeUserInputEvents);
          }
-Line 1131 
 bool MainForm::saveSessionFile ( const Q
+Line 1203 
 bool MainForm::saveSessionFile ( const Q
          QMap<int, int> midiInstrumentMap;
          int *piMaps = ::lscp_list_midi_instrument_maps(m_pClient);
          for (int iMap = 0; piMaps && piMaps[iMap] >= 0; iMap++) {
-                 int iMidiMap = piMaps[iMap];
+                 const int iMidiMap = piMaps[iMap];
                  const char *pszMapName
                          = ::lscp_get_midi_instrument_map_name(m_pClient, iMidiMap);
                  ts << "# " << tr("MIDI instrument map") << " " << iMap;
-Line 1183 
 bool MainForm::saveSessionFile ( const Q
+Line 1255 
 bool MainForm::saveSessionFile ( const Q
                  }
                  ts << endl;
                  // Check for errors...
-                 if (pInstrs == NULL && ::lscp_client_get_errno(m_pClient)) {
+                 if (pInstrs == nullptr && ::lscp_client_get_errno(m_pClient)) {
                          appendMessagesClient("lscp_list_midi_instruments");
                          iErrors++;
                  }
                  // MIDI strument index/id mapping.
-                 midiInstrumentMap[iMidiMap] = iMap;
+                 midiInstrumentMap.insert(iMidiMap, iMap);
          }
          // Check for errors...
-         if (piMaps == NULL && ::lscp_client_get_errno(m_pClient)) {
+         if (piMaps == nullptr && ::lscp_client_get_errno(m_pClient)) {
                  appendMessagesClient("lscp_list_midi_instrument_maps");
                  iErrors++;
          }
  #endif  // CONFIG_MIDI_INSTRUMENT
-         // Sampler channel mapping.
+         // Sampler channel mapping...
-         QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+         int iChannelID = 0;
-         for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+         const QList<QMdiSubWindow *>& wlist
-                 ChannelStrip *pChannelStrip = NULL;
+                 = m_pWorkspace->subWindowList();
-                 QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+         foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                 if (pMdiSubWindow)
+                 ChannelStrip *pChannelStrip
-                         pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
+                         = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
                  if (pChannelStrip) {
                          Channel *pChannel = pChannelStrip->channel();
                          if (pChannel) {
-                                 ts << "# " << tr("Channel") << " " << iChannel << endl;
+                                 // Avoid "artifial" plug-in devices...
+                                 const int iAudioDevice = pChannel->audioDevice();
+                                 if (!audioDeviceMap.contains(iAudioDevice))
+                                         continue;
+                                 const int iMidiDevice = pChannel->midiDevice();
+                                 if (!midiDeviceMap.contains(iMidiDevice))
+                                         continue;
+                                 // Go for regular, canonical devices...
+                                 ts << "# " << tr("Channel") << " " << iChannelID << endl;
                                  ts << "ADD CHANNEL" << endl;
                                  if (audioDeviceMap.isEmpty()) {
-                                         ts << "SET CHANNEL AUDIO_OUTPUT_TYPE " << iChannel
+                                         ts << "SET CHANNEL AUDIO_OUTPUT_TYPE " << iChannelID
                                                  << " " << pChannel->audioDriver() << endl;
                                  } else {
-                                         ts << "SET CHANNEL AUDIO_OUTPUT_DEVICE " << iChannel
+                                         ts << "SET CHANNEL AUDIO_OUTPUT_DEVICE " << iChannelID
-                                                 << " " << audioDeviceMap[pChannel->audioDevice()] << endl;
+                                                 << " " << audioDeviceMap.value(iAudioDevice) << endl;
                                  }
                                  if (midiDeviceMap.isEmpty()) {
-                                         ts << "SET CHANNEL MIDI_INPUT_TYPE " << iChannel
+                                         ts << "SET CHANNEL MIDI_INPUT_TYPE " << iChannelID
                                                  << " " << pChannel->midiDriver() << endl;
                                  } else {
-                                         ts << "SET CHANNEL MIDI_INPUT_DEVICE " << iChannel
+                                         ts << "SET CHANNEL MIDI_INPUT_DEVICE " << iChannelID
-                                                 << " " << midiDeviceMap[pChannel->midiDevice()] << endl;
+                                                 << " " << midiDeviceMap.value(iMidiDevice) << endl;
                                  }
-                                 ts << "SET CHANNEL MIDI_INPUT_PORT " << iChannel
+                                 ts << "SET CHANNEL MIDI_INPUT_PORT " << iChannelID
                                          << " " << pChannel->midiPort() << endl;
-                                 ts << "SET CHANNEL MIDI_INPUT_CHANNEL " << iChannel << " ";
+                                 ts << "SET CHANNEL MIDI_INPUT_CHANNEL " << iChannelID << " ";
                                  if (pChannel->midiChannel() == LSCP_MIDI_CHANNEL_ALL)
                                          ts << "ALL";
                                  else
                                          ts << pChannel->midiChannel();
                                  ts << endl;
                                  ts << "LOAD ENGINE " << pChannel->engineName()
-                                         << " " << iChannel << endl;
+                                         << " " << iChannelID << endl;
                                  if (pChannel->instrumentStatus() < 100) ts << "# ";
                                  ts << "LOAD INSTRUMENT NON_MODAL '"
                                          << pChannel->instrumentFile() << "' "
-                                         << pChannel->instrumentNr() << " " << iChannel << endl;
+                                         << pChannel->instrumentNr() << " " << iChannelID << endl;
                                  ChannelRoutingMap::ConstIterator audioRoute;
                                  for (audioRoute = pChannel->audioRouting().begin();
                                                  audioRoute != pChannel->audioRouting().end();
                                                          ++audioRoute) {
-                                         ts << "SET CHANNEL AUDIO_OUTPUT_CHANNEL " << iChannel
+                                         ts << "SET CHANNEL AUDIO_OUTPUT_CHANNEL " << iChannelID
                                                  << " " << audioRoute.key()
                                                  << " " << audioRoute.value() << endl;
                                  }
-                                 ts << "SET CHANNEL VOLUME " << iChannel
+                                 ts << "SET CHANNEL VOLUME " << iChannelID
                                          << " " << pChannel->volume() << endl;
                                  if (pChannel->channelMute())
-                                         ts << "SET CHANNEL MUTE " << iChannel << " 1" << endl;
+                                         ts << "SET CHANNEL MUTE " << iChannelID << " 1" << endl;
                                  if (pChannel->channelSolo())
-                                         ts << "SET CHANNEL SOLO " << iChannel << " 1" << endl;
+                                         ts << "SET CHANNEL SOLO " << iChannelID << " 1" << endl;
                          #ifdef CONFIG_MIDI_INSTRUMENT
-                                 if (pChannel->midiMap() >= 0) {
+                                 const int iMidiMap = pChannel->midiMap();
-                                         ts << "SET CHANNEL MIDI_INSTRUMENT_MAP " << iChannel
+                                 if (midiInstrumentMap.contains(iMidiMap)) {
-                                                 << " " << midiInstrumentMap[pChannel->midiMap()] << endl;
+                                         ts << "SET CHANNEL MIDI_INSTRUMENT_MAP " << iChannelID
+                                                 << " " << midiInstrumentMap.value(iMidiMap) << endl;
                                  }
                          #endif
                          #ifdef CONFIG_FXSEND
-                                 int iChannelID = pChannel->channelID();
                                  int *piFxSends = ::lscp_list_fxsends(m_pClient, iChannelID);
                                  for (int iFxSend = 0;
                                                  piFxSends && piFxSends[iFxSend] >= 0;
-Line 1266 
 bool MainForm::saveSessionFile ( const Q
+Line 1346 
 bool MainForm::saveSessionFile ( const Q
                                          lscp_fxsend_info_t *pFxSendInfo = ::lscp_get_fxsend_info(
                                                  m_pClient, iChannelID, piFxSends[iFxSend]);
                                          if (pFxSendInfo) {
-                                                 ts << "CREATE FX_SEND " << iChannel
+                                                 ts << "CREATE FX_SEND " << iChannelID
                                                          << " " << pFxSendInfo->midi_controller;
                                                  if (pFxSendInfo->name)
                                                          ts << " '" << pFxSendInfo->name << "'";
-Line 1276 
 bool MainForm::saveSessionFile ( const Q
+Line 1356 
 bool MainForm::saveSessionFile ( const Q
                                                                  piRouting && piRouting[iAudioSrc] >= 0;
                                                                          iAudioSrc++) {
                                                          ts << "SET FX_SEND AUDIO_OUTPUT_CHANNEL "
-                                                                 << iChannel
+                                                                 << iChannelID
                                                                  << " " << iFxSend
                                                                  << " " << iAudioSrc
                                                                  << " " << piRouting[iAudioSrc] << endl;
                                                  }
                                          #ifdef CONFIG_FXSEND_LEVEL
-                                                 ts << "SET FX_SEND LEVEL " << iChannel
+                                                 ts << "SET FX_SEND LEVEL " << iChannelID
                                                          << " " << iFxSend
                                                          << " " << pFxSendInfo->level << endl;
                                          #endif
-Line 1294 
 bool MainForm::saveSessionFile ( const Q
+Line 1374 
 bool MainForm::saveSessionFile ( const Q
                                  }
                          #endif
                                  ts << endl;
+                                 // Go for next channel...
+                                 ++iChannelID;
                          }
                  }
                  // Try to keep it snappy :)
-Line 1345 
 void MainForm::sessionDirty (void)
+Line 1427 
 void MainForm::sessionDirty (void)
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- File Action slots.
+ // QSampler::MainForm -- File Action slots.
  // Create a new sampler session.
  void MainForm::fileNew (void)
-Line 1369 
 void MainForm::fileOpenRecent (void)
+Line 1451 
 void MainForm::fileOpenRecent (void)
          // Retrive filename index from action data...
          QAction *pAction = qobject_cast<QAction *> (sender());
          if (pAction && m_pOptions) {
-                 int iIndex = pAction->data().toInt();
+                 const int iIndex = pAction->data().toInt();
                  if (iIndex >= 0 && iIndex < m_pOptions->recentFiles.count()) {
                          QString sFilename = m_pOptions->recentFiles[iIndex];
                          // Check if we can safely close the current session...
-Line 1399 
 void MainForm::fileSaveAs (void)
+Line 1481 
 void MainForm::fileSaveAs (void)
  // Reset the sampler instance.
  void MainForm::fileReset (void)
  {
-         if (m_pClient == NULL)
+         if (m_pClient == nullptr)
                  return;
          // Ask user whether he/she want's an internal sampler reset...
-         if (QMessageBox::warning(this,
+         if (m_pOptions && m_pOptions->bConfirmReset) {
-                 QSAMPLER_TITLE ": " + tr("Warning"),
+                 const QString& sTitle = tr("Warning");
-                 tr("Resetting the sampler instance will close\n"
+                 const QString& sText = tr(
-                 "all device and channel configurations.\n\n"
+                         "Resetting the sampler instance will close\n"
-                 "Please note that this operation may cause\n"
+                         "all device and channel configurations.\n\n"
-                 "temporary MIDI and Audio disruption.\n\n"
+                         "Please note that this operation may cause\n"
-                 "Do you want to reset the sampler engine now?"),
+                         "temporary MIDI and Audio disruption.\n\n"
-                 QMessageBox::Ok | QMessageBox::Cancel)
+                         "Do you want to reset the sampler engine now?");
-                 == QMessageBox::Cancel)
+         #if 0
-                 return;
+                 if (QMessageBox::warning(this, sTitle, sText,
+                         QMessageBox::Ok | QMessageBox::Cancel) == QMessageBox::Cancel)
+                         return;
+         #else
+                 QMessageBox mbox(this);
+                 mbox.setIcon(QMessageBox::Warning);
+                 mbox.setWindowTitle(sTitle);
+                 mbox.setText(sText);
+                 mbox.setStandardButtons(QMessageBox::Ok | QMessageBox::Cancel);
+                 QCheckBox cbox(tr("Don't ask this again"));
+                 cbox.setChecked(false);
+                 cbox.blockSignals(true);
+                 mbox.addButton(&cbox, QMessageBox::ActionRole);
+                 if (mbox.exec() == QMessageBox::Cancel)
+                         return;
+                 if (cbox.isChecked())
+                         m_pOptions->bConfirmReset = false;
+         #endif
+         }
          // Trye closing the current session, first...
          if (!closeSession(true))
-Line 1437 
 void MainForm::fileReset (void)
+Line 1537 
 void MainForm::fileReset (void)
  // Restart the client/server instance.
  void MainForm::fileRestart (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          bool bRestart = true;
          // Ask user whether he/she want's a complete restart...
          // (if we're currently up and running)
-         if (bRestart && m_pClient) {
+         if (m_pOptions && m_pOptions->bConfirmRestart) {
-                 bRestart = (QMessageBox::warning(this,
+                 const QString& sTitle = tr("Warning");
-                         QSAMPLER_TITLE ": " + tr("Warning"),
+                 const QString& sText = tr(
-                         tr("New settings will be effective after\n"
+                         "New settings will be effective after\n"
                          "restarting the client/server connection.\n\n"
                          "Please note that this operation may cause\n"
                          "temporary MIDI and Audio disruption.\n\n"
-                         "Do you want to restart the connection now?"),
+                         "Do you want to restart the connection now?");
-                         QMessageBox::Ok | QMessageBox::Cancel) == QMessageBox::Ok);
+         #if 0
+                 if (QMessageBox::warning(this, sTitle, sText,
+                         QMessageBox::Ok | QMessageBox::Cancel) == QMessageBox::Cancel)
+                         bRestart = false;
+         #else
+                 QMessageBox mbox(this);
+                 mbox.setIcon(QMessageBox::Warning);
+                 mbox.setWindowTitle(sTitle);
+                 mbox.setText(sText);
+                 mbox.setStandardButtons(QMessageBox::Ok | QMessageBox::Cancel);
+                 QCheckBox cbox(tr("Don't ask this again"));
+                 cbox.setChecked(false);
+                 cbox.blockSignals(true);
+                 mbox.addButton(&cbox, QMessageBox::ActionRole);
+                 if (mbox.exec() == QMessageBox::Cancel)
+                         bRestart = false;
+                 else
+                 if (cbox.isChecked())
+                         m_pOptions->bConfirmRestart = false;
+         #endif
          }
          // Are we still for it?
-Line 1474 
 void MainForm::fileExit (void)
+Line 1593 
 void MainForm::fileExit (void)
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- Edit Action slots.
+ // QSampler::MainForm -- Edit Action slots.
  // Add a new sampler channel.
  void MainForm::editAddChannel (void)
  {
-         if (m_pClient == NULL)
+         ++m_iDirtySetup;
+         addChannelStrip();
+         --m_iDirtySetup;
+ }
+ void MainForm::addChannelStrip (void)
+ {
+         if (m_pClient == nullptr)
                  return;
          // Just create the channel instance...
          Channel *pChannel = new Channel();
-         if (pChannel == NULL)
+         if (pChannel == nullptr)
                  return;
          // Before we show it up, may be we'll
-Line 1502 
 void MainForm::editAddChannel (void)
+Line 1628 
 void MainForm::editAddChannel (void)
          }
          // Do we auto-arrange?
-         if (m_pOptions && m_pOptions->bAutoArrange)
+         channelsArrangeAuto();
-                 channelsArrange();
          // Make that an overall update.
          m_iDirtyCount++;
-Line 1514 
 void MainForm::editAddChannel (void)
+Line 1639 
 void MainForm::editAddChannel (void)
  // Remove current sampler channel.
  void MainForm::editRemoveChannel (void)
  {
-         if (m_pClient == NULL)
+         ++m_iDirtySetup;
+         removeChannelStrip();
+         --m_iDirtySetup;
+ }
+ void MainForm::removeChannelStrip (void)
+ {
+         if (m_pClient == nullptr)
                  return;
          ChannelStrip *pChannelStrip = activeChannelStrip();
-         if (pChannelStrip == NULL)
+         if (pChannelStrip == nullptr)
                  return;
          Channel *pChannel = pChannelStrip->channel();
-         if (pChannel == NULL)
+         if (pChannel == nullptr)
                  return;
          // Prompt user if he/she's sure about this...
          if (m_pOptions && m_pOptions->bConfirmRemove) {
-                 if (QMessageBox::warning(this,
+                 const QString& sTitle = tr("Warning");
-                         QSAMPLER_TITLE ": " + tr("Warning"),
+                 const QString& sText = tr(
-                         tr("About to remove channel:\n\n"
+                         "About to remove channel:\n\n"
                          "%1\n\n"
                          "Are you sure?")
-                         .arg(pChannelStrip->windowTitle()),
+                         .arg(pChannelStrip->windowTitle());
-                         QMessageBox::Ok | QMessageBox::Cancel)
+         #if 0
-                         == QMessageBox::Cancel)
+                 if (QMessageBox::warning(this, sTitle, sText,
+                         QMessageBox::Ok | QMessageBox::Cancel) == QMessageBox::Cancel)
                          return;
+         #else
+                 QMessageBox mbox(this);
+                 mbox.setIcon(QMessageBox::Warning);
+                 mbox.setWindowTitle(sTitle);
+                 mbox.setText(sText);
+                 mbox.setStandardButtons(QMessageBox::Ok | QMessageBox::Cancel);
+                 QCheckBox cbox(tr("Don't ask this again"));
+                 cbox.setChecked(false);
+                 cbox.blockSignals(true);
+                 mbox.addButton(&cbox, QMessageBox::ActionRole);
+                 if (mbox.exec() == QMessageBox::Cancel)
+                         return;
+                 if (cbox.isChecked())
+                         m_pOptions->bConfirmRemove = false;
+         #endif
          }
          // Remove the existing sampler channel.
          if (!pChannel->removeChannel())
                  return;
-         // We'll be dirty, for sure...
-         m_iDirtyCount++;
          // Just delete the channel strip.
          destroyChannelStrip(pChannelStrip);
+         // We'll be dirty, for sure...
+         m_iDirtyCount++;
+         stabilizeForm();
  }
  // Setup current sampler channel.
  void MainForm::editSetupChannel (void)
  {
-         if (m_pClient == NULL)
+         if (m_pClient == nullptr)
                  return;
          ChannelStrip *pChannelStrip = activeChannelStrip();
-         if (pChannelStrip == NULL)
+         if (pChannelStrip == nullptr)
                  return;
          // Just invoque the channel strip procedure.
-Line 1568 
 void MainForm::editSetupChannel (void)
+Line 1717 
 void MainForm::editSetupChannel (void)
  // Edit current sampler channel.
  void MainForm::editEditChannel (void)
  {
-         if (m_pClient == NULL)
+         if (m_pClient == nullptr)
                  return;
          ChannelStrip *pChannelStrip = activeChannelStrip();
-         if (pChannelStrip == NULL)
+         if (pChannelStrip == nullptr)
                  return;
          // Just invoque the channel strip procedure.
-Line 1583 
 void MainForm::editEditChannel (void)
+Line 1732 
 void MainForm::editEditChannel (void)
  // Reset current sampler channel.
  void MainForm::editResetChannel (void)
  {
-         if (m_pClient == NULL)
+         if (m_pClient == nullptr)
                  return;
          ChannelStrip *pChannelStrip = activeChannelStrip();
-         if (pChannelStrip == NULL)
+         if (pChannelStrip == nullptr)
                  return;
          // Just invoque the channel strip procedure.
-Line 1598 
 void MainForm::editResetChannel (void)
+Line 1747 
 void MainForm::editResetChannel (void)
  // Reset all sampler channels.
  void MainForm::editResetAllChannels (void)
  {
-         if (m_pClient == NULL)
+         if (m_pClient == nullptr)
                  return;
          // Invoque the channel strip procedure,
          // for all channels out there...
          m_pWorkspace->setUpdatesEnabled(false);
-         QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+         const QList<QMdiSubWindow *>& wlist
-         for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+                 = m_pWorkspace->subWindowList();
-                 ChannelStrip *pChannelStrip = NULL;
+         foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                 QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+                 ChannelStrip *pChannelStrip
-                 if (pMdiSubWindow)
+                         = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
-                         pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
                  if (pChannelStrip)
                          pChannelStrip->channelReset();
          }
-Line 1618 
 void MainForm::editResetAllChannels (voi
+Line 1766 
 void MainForm::editResetAllChannels (voi
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- View Action slots.
+ // QSampler::MainForm -- View Action slots.
  // Show/hide the main program window menubar.
  void MainForm::viewMenubar ( bool bOn )
-Line 1668 
 void MainForm::viewMessages ( bool bOn )
+Line 1816 
 void MainForm::viewMessages ( bool bOn )
  // Show/hide the MIDI instrument list-view form.
  void MainForm::viewInstruments (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          if (m_pInstrumentListForm) {
-Line 1687 
 void MainForm::viewInstruments (void)
+Line 1835 
 void MainForm::viewInstruments (void)
  // Show/hide the device configurator form.
  void MainForm::viewDevices (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          if (m_pDeviceForm) {
-Line 1706 
 void MainForm::viewDevices (void)
+Line 1854 
 void MainForm::viewDevices (void)
  // Show options dialog.
  void MainForm::viewOptions (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          OptionsForm* pOptionsForm = new OptionsForm(this);
-Line 1718 
 void MainForm::viewOptions (void)
+Line 1866 
 void MainForm::viewOptions (void)
                  if (m_pOptions->sMessagesFont.isEmpty() && m_pMessages)
                          m_pOptions->sMessagesFont = m_pMessages->messagesFont().toString();
                  // To track down deferred or immediate changes.
-                 QString sOldServerHost      = m_pOptions->sServerHost;
+                 const QString sOldServerHost      = m_pOptions->sServerHost;
-                 int     iOldServerPort      = m_pOptions->iServerPort;
+                 const int     iOldServerPort      = m_pOptions->iServerPort;
-                 int     iOldServerTimeout   = m_pOptions->iServerTimeout;
+                 const int     iOldServerTimeout   = m_pOptions->iServerTimeout;
-                 bool    bOldServerStart     = m_pOptions->bServerStart;
+                 const bool    bOldServerStart     = m_pOptions->bServerStart;
-                 QString sOldServerCmdLine   = m_pOptions->sServerCmdLine;
+                 const QString sOldServerCmdLine   = m_pOptions->sServerCmdLine;
-                 bool    bOldMessagesLog     = m_pOptions->bMessagesLog;
+                 const bool    bOldMessagesLog     = m_pOptions->bMessagesLog;
-                 QString sOldMessagesLogPath = m_pOptions->sMessagesLogPath;
+                 const QString sOldMessagesLogPath = m_pOptions->sMessagesLogPath;
-                 QString sOldDisplayFont     = m_pOptions->sDisplayFont;
+                 const QString sOldDisplayFont     = m_pOptions->sDisplayFont;
-                 bool    bOldDisplayEffect   = m_pOptions->bDisplayEffect;
+                 const bool    bOldDisplayEffect   = m_pOptions->bDisplayEffect;
-                 int     iOldMaxVolume       = m_pOptions->iMaxVolume;
+                 const int     iOldMaxVolume       = m_pOptions->iMaxVolume;
-                 QString sOldMessagesFont    = m_pOptions->sMessagesFont;
+                 const QString sOldMessagesFont    = m_pOptions->sMessagesFont;
-                 bool    bOldKeepOnTop       = m_pOptions->bKeepOnTop;
+                 const bool    bOldKeepOnTop       = m_pOptions->bKeepOnTop;
-                 bool    bOldStdoutCapture   = m_pOptions->bStdoutCapture;
+                 const bool    bOldStdoutCapture   = m_pOptions->bStdoutCapture;
-                 int     bOldMessagesLimit   = m_pOptions->bMessagesLimit;
+                 const int     bOldMessagesLimit   = m_pOptions->bMessagesLimit;
-                 int     iOldMessagesLimitLines = m_pOptions->iMessagesLimitLines;
+                 const int     iOldMessagesLimitLines = m_pOptions->iMessagesLimitLines;
-                 bool    bOldCompletePath    = m_pOptions->bCompletePath;
+                 const bool    bOldCompletePath    = m_pOptions->bCompletePath;
-                 bool    bOldInstrumentNames = m_pOptions->bInstrumentNames;
+                 const bool    bOldInstrumentNames = m_pOptions->bInstrumentNames;
-                 int     iOldMaxRecentFiles  = m_pOptions->iMaxRecentFiles;
+                 const int     iOldMaxRecentFiles  = m_pOptions->iMaxRecentFiles;
-                 int     iOldBaseFontSize    = m_pOptions->iBaseFontSize;
+                 const int     iOldBaseFontSize    = m_pOptions->iBaseFontSize;
                  // Load the current setup settings.
                  pOptionsForm->setup(m_pOptions);
                  // Show the setup dialog...
-Line 1748 
 void MainForm::viewOptions (void)
+Line 1896 
 void MainForm::viewOptions (void)
                                  (!bOldKeepOnTop     &&  m_pOptions->bKeepOnTop)     ||
                                  (iOldBaseFontSize   !=  m_pOptions->iBaseFontSize)) {
                                  QMessageBox::information(this,
-                                         QSAMPLER_TITLE ": " + tr("Information"),
+                                         tr("Information"),
                                          tr("Some settings may be only effective\n"
                                          "next time you start this program."));
                                  updateMessagesCapture();
-Line 1799 
 void MainForm::viewOptions (void)
+Line 1947 
 void MainForm::viewOptions (void)
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- Channels action slots.
+ // QSampler::MainForm -- Channels action slots.
  // Arrange channel strips.
  void MainForm::channelsArrange (void)
  {
          // Full width vertical tiling
-         QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+         const QList<QMdiSubWindow *>& wlist
+                 = m_pWorkspace->subWindowList();
          if (wlist.isEmpty())
                  return;
          m_pWorkspace->setUpdatesEnabled(false);
          int y = 0;
-         for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+         foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                 ChannelStrip *pChannelStrip = NULL;
+                 pMdiSubWindow->adjustSize();
-                 QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+                 const QRect& frameRect
-                 if (pMdiSubWindow)
+                         = pMdiSubWindow->frameGeometry();
-                         pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
+                 int w = m_pWorkspace->width();
-                 if (pChannelStrip) {
+                 if (w < frameRect.width())
-                 /*  if (pChannelStrip->testWState(WState_Maximized | WState_Minimized)) {
+                         w = frameRect.width();
-                                 // Prevent flicker...
+                 const int h = frameRect.height();
-                                 pChannelStrip->hide();
+                 pMdiSubWindow->setGeometry(0, y, w, h);
-                                 pChannelStrip->showNormal();
+                 y += h;
-                         }   */
-                         pChannelStrip->adjustSize();
-                         int iWidth  = m_pWorkspace->width();
-                         if (iWidth < pChannelStrip->width())
-                                 iWidth = pChannelStrip->width();
-                 //  int iHeight = pChannelStrip->height()
-                 //              + pChannelStrip->parentWidget()->baseSize().height();
-                         int iHeight = pChannelStrip->parentWidget()->frameGeometry().height();
-                         pChannelStrip->parentWidget()->setGeometry(0, y, iWidth, iHeight);
-                         y += iHeight;
-                 }
          }
          m_pWorkspace->setUpdatesEnabled(true);
-Line 1842 
 void MainForm::channelsArrange (void)
+Line 1980 
 void MainForm::channelsArrange (void)
  // Auto-arrange channel strips.
  void MainForm::channelsAutoArrange ( bool bOn )
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          // Toggle the auto-arrange flag.
          m_pOptions->bAutoArrange = bOn;
          // If on, update whole workspace...
-         if (m_pOptions->bAutoArrange)
+         channelsArrangeAuto();
+ }
+ void MainForm::channelsArrangeAuto (void)
+ {
+         if (m_pOptions && m_pOptions->bAutoArrange)
                  channelsArrange();
  }
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- Help Action slots.
+ // QSampler::MainForm -- Help Action slots.
  // Show information about the Qt toolkit.
  void MainForm::helpAboutQt (void)
-Line 1867 
 void MainForm::helpAboutQt (void)
+Line 2011 
 void MainForm::helpAboutQt (void)
  // Show information about application program.
  void MainForm::helpAbout (void)
  {
-         // Stuff the about box text...
+         QStringList list;
-         QString sText = "<p>\n";
-         sText += "<b>" QSAMPLER_TITLE " - " + tr(QSAMPLER_SUBTITLE) + "</b><br />\n";
-         sText += "<br />\n";
-         sText += tr("Version") + ": <b>" QSAMPLER_VERSION "</b><br />\n";
-         sText += "<small>" + tr("Build") + ": " __DATE__ " " __TIME__ "</small><br />\n";
  #ifdef CONFIG_DEBUG
-         sText += "<small><font color=\"red\">";
+         list << tr("Debugging option enabled.");
-         sText += tr("Debugging option enabled.");
-         sText += "</font></small><br />";
  #endif
  #ifndef CONFIG_LIBGIG
-         sText += "<small><font color=\"red\">";
+         list << tr("GIG (libgig) file support disabled.");
-         sText += tr("GIG (libgig) file support disabled.");
-         sText += "</font></small><br />";
  #endif
  #ifndef CONFIG_INSTRUMENT_NAME
-         sText += "<small><font color=\"red\">";
+         list << tr("LSCP (liblscp) instrument_name support disabled.");
-         sText += tr("LSCP (liblscp) instrument_name support disabled.");
-         sText += "</font></small><br />";
  #endif
  #ifndef CONFIG_MUTE_SOLO
-         sText += "<small><font color=\"red\">";
+         list << tr("Sampler channel Mute/Solo support disabled.");
-         sText += tr("Sampler channel Mute/Solo support disabled.");
-         sText += "</font></small><br />";
  #endif
  #ifndef CONFIG_AUDIO_ROUTING
-         sText += "<small><font color=\"red\">";
+         list << tr("LSCP (liblscp) audio_routing support disabled.");
-         sText += tr("LSCP (liblscp) audio_routing support disabled.");
-         sText += "</font></small><br />";
  #endif
  #ifndef CONFIG_FXSEND
-         sText += "<small><font color=\"red\">";
+         list << tr("Sampler channel Effect Sends support disabled.");
-         sText += tr("Sampler channel Effect Sends support disabled.");
-         sText += "</font></small><br />";
  #endif
  #ifndef CONFIG_VOLUME
-         sText += "<small><font color=\"red\">";
+         list << tr("Global volume support disabled.");
-         sText += tr("Global volume support disabled.");
-         sText += "</font></small><br />";
  #endif
  #ifndef CONFIG_MIDI_INSTRUMENT
-         sText += "<small><font color=\"red\">";
+         list << tr("MIDI instrument mapping support disabled.");
-         sText += tr("MIDI instrument mapping support disabled.");
-         sText += "</font></small><br />";
  #endif
  #ifndef CONFIG_EDIT_INSTRUMENT
-         sText += "<small><font color=\"red\">";
+         list << tr("Instrument editing support disabled.");
-         sText += tr("Instrument editing support disabled.");
-         sText += "</font></small><br />";
  #endif
  #ifndef CONFIG_EVENT_CHANNEL_MIDI
-         sText += "<small><font color=\"red\">";
+         list << tr("Channel MIDI event support disabled.");
-         sText += tr("Channel MIDI event support disabled.");
-         sText += "</font></small><br />";
  #endif
  #ifndef CONFIG_EVENT_DEVICE_MIDI
-         sText += "<small><font color=\"red\">";
+         list << tr("Device MIDI event support disabled.");
-         sText += tr("Device MIDI event support disabled.");
-         sText += "</font></small><br />";
  #endif
  #ifndef CONFIG_MAX_VOICES
-         sText += "<small><font color=\"red\">";
+         list << tr("Runtime max. voices / disk streams support disabled.");
-         sText += tr("Runtime max. voices / disk streams support disabled.");
-         sText += "</font></small><br />";
  #endif
+         // Stuff the about box text...
+         QString sText = "<p>\n";
+         sText += "<b>" QSAMPLER_TITLE " - " + tr(QSAMPLER_SUBTITLE) + "</b><br />\n";
+         sText += "<br />\n";
+         sText += tr("Version") + ": <b>" CONFIG_BUILD_VERSION "</b><br />\n";
+ //      sText += "<small>" + tr("Build") + ": " CONFIG_BUILD_DATE "</small><br />\n";
+         if (!list.isEmpty()) {
+                 sText += "<small><font color=\"red\">";
+                 sText += list.join("<br />\n");
+                 sText += "</font></small>";
+         }
          sText += "<br />\n";
          sText += tr("Using") + ": ";
          sText += ::lscp_client_package();
-Line 1957 
 void MainForm::helpAbout (void)
+Line 2084 
 void MainForm::helpAbout (void)
          sText += "</small>";
          sText += "</p>\n";
-         QMessageBox::about(this, tr("About") + " " QSAMPLER_TITLE, sText);
+         QMessageBox::about(this, tr("About"), sText);
  }
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- Main window stabilization.
+ // QSampler::MainForm -- Main window stabilization.
  void MainForm::stabilizeForm (void)
  {
-Line 1970 
 void MainForm::stabilizeForm (void)
+Line 2097 
 void MainForm::stabilizeForm (void)
          QString sSessionName = sessionName(m_sFilename);
          if (m_iDirtyCount > 0)
                  sSessionName += " *";
-         setWindowTitle(tr(QSAMPLER_TITLE " - [%1]").arg(sSessionName));
+         setWindowTitle(sSessionName);
          // Update the main menu state...
          ChannelStrip *pChannelStrip = activeChannelStrip();
-         bool bHasClient = (m_pOptions != NULL && m_pClient != NULL);
+         const QList<QMdiSubWindow *>& wlist = m_pWorkspace->subWindowList();
-         bool bHasChannel = (bHasClient && pChannelStrip != NULL);
+         const bool bHasClient = (m_pOptions != nullptr && m_pClient != nullptr);
-         bool bHasChannels = (bHasClient && m_pWorkspace->subWindowList().count() > 0);
+         const bool bHasChannel = (bHasClient && pChannelStrip != nullptr);
+         const bool bHasChannels = (bHasClient && wlist.count() > 0);
          m_ui.fileNewAction->setEnabled(bHasClient);
          m_ui.fileOpenAction->setEnabled(bHasClient);
          m_ui.fileSaveAction->setEnabled(bHasClient && m_iDirtyCount > 0);
          m_ui.fileSaveAsAction->setEnabled(bHasClient);
          m_ui.fileResetAction->setEnabled(bHasClient);
-         m_ui.fileRestartAction->setEnabled(bHasClient || m_pServer == NULL);
+         m_ui.fileRestartAction->setEnabled(bHasClient || m_pServer == nullptr);
          m_ui.editAddChannelAction->setEnabled(bHasClient);
          m_ui.editRemoveChannelAction->setEnabled(bHasChannel);
          m_ui.editSetupChannelAction->setEnabled(bHasChannel);
-Line 2056 
 void MainForm::volumeChanged ( int iVolu
+Line 2184 
 void MainForm::volumeChanged ( int iVolu
                  m_pVolumeSpinBox->setValue(iVolume);
          // Do it as commanded...
-         float fVolume = 0.01f * float(iVolume);
+         const float fVolume = 0.01f * float(iVolume);
          if (::lscp_set_volume(m_pClient, fVolume) == LSCP_OK)
                  appendMessages(QObject::tr("Volume: %1.").arg(fVolume));
          else
-Line 2091 
 void MainForm::channelStripChanged ( Cha
+Line 2219 
 void MainForm::channelStripChanged ( Cha
  void MainForm::updateSession (void)
  {
  #ifdef CONFIG_VOLUME
-         int iVolume = ::lroundf(100.0f * ::lscp_get_volume(m_pClient));
+         const int iVolume = ::lroundf(100.0f * ::lscp_get_volume(m_pClient));
          m_iVolumeChanging++;
          m_pVolumeSlider->setValue(iVolume);
          m_pVolumeSpinBox->setValue(iVolume);
-Line 2099 
 void MainForm::updateSession (void)
+Line 2227 
 void MainForm::updateSession (void)
  #endif
  #ifdef CONFIG_MIDI_INSTRUMENT
          // FIXME: Make some room for default instrument maps...
-         int iMaps = ::lscp_get_midi_instrument_maps(m_pClient);
+         const int iMaps = ::lscp_get_midi_instrument_maps(m_pClient);
          if (iMaps < 0)
                  appendMessagesClient("lscp_get_midi_instrument_maps");
          else if (iMaps < 1) {
-Line 2113 
 void MainForm::updateSession (void)
+Line 2241 
 void MainForm::updateSession (void)
          updateAllChannelStrips(false);
          // Do we auto-arrange?
-         if (m_pOptions && m_pOptions->bAutoArrange)
+         channelsArrangeAuto();
-                 channelsArrange();
          // Remember to refresh devices and instruments...
          if (m_pInstrumentListForm)
-Line 2126 
 void MainForm::updateSession (void)
+Line 2253 
 void MainForm::updateSession (void)
  void MainForm::updateAllChannelStrips ( bool bRemoveDeadStrips )
  {
+         // Skip if setting up a new channel strip...
+         if (m_iDirtySetup > 0)
+                 return;
          // Retrieve the current channel list.
          int *piChannelIDs = ::lscp_list_channels(m_pClient);
-         if (piChannelIDs == NULL) {
+         if (piChannelIDs == nullptr) {
                  if (::lscp_client_get_errno(m_pClient)) {
                          appendMessagesClient("lscp_list_channels");
                          appendMessagesError(
-Line 2143 
 void MainForm::updateAllChannelStrips (
+Line 2274 
 void MainForm::updateAllChannelStrips (
                                  createChannelStrip(new Channel(piChannelIDs[iChannel]));
                  }
                  // Do we auto-arrange?
-                 if (m_pOptions && m_pOptions->bAutoArrange)
+                 channelsArrangeAuto();
-                         channelsArrange();
                  // remove dead channel strips
                  if (bRemoveDeadStrips) {
-                         QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+                         const QList<QMdiSubWindow *>& wlist
-                         for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+                                 = m_pWorkspace->subWindowList();
-                                 ChannelStrip *pChannelStrip = NULL;
+                         foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                                 QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+                                 ChannelStrip *pChannelStrip
-                                 if (pMdiSubWindow)
+                                         = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
-                                         pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
                                  if (pChannelStrip) {
                                          bool bExists = false;
-                                         for (int j = 0; piChannelIDs[j] >= 0; ++j) {
+                                         for (int iChannel = 0; piChannelIDs[iChannel] >= 0; ++iChannel) {
-                                                 if (!pChannelStrip->channel())
+                                                 Channel *pChannel = pChannelStrip->channel();
+                                                 if (pChannel == nullptr)
                                                          break;
-                                                 if (piChannelIDs[j] == pChannelStrip->channel()->channelID()) {
+                                                 if (piChannelIDs[iChannel] == pChannel->channelID()) {
                                                          // strip exists, don't touch it
                                                          bExists = true;
                                                          break;
-Line 2179 
 void MainForm::updateAllChannelStrips (
+Line 2309 
 void MainForm::updateAllChannelStrips (
  // Update the recent files list and menu.
  void MainForm::updateRecentFiles ( const QString& sFilename )
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          // Remove from list if already there (avoid duplicates)
-         int iIndex = m_pOptions->recentFiles.indexOf(sFilename);
+         const int iIndex = m_pOptions->recentFiles.indexOf(sFilename);
          if (iIndex >= 0)
                  m_pOptions->recentFiles.removeAt(iIndex);
          // Put it to front...
-Line 2194 
 void MainForm::updateRecentFiles ( const
+Line 2324 
 void MainForm::updateRecentFiles ( const
  // Update the recent files list and menu.
  void MainForm::updateRecentFilesMenu (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          // Time to keep the list under limits.
-Line 2222 
 void MainForm::updateRecentFilesMenu (vo
+Line 2352 
 void MainForm::updateRecentFilesMenu (vo
  void MainForm::updateInstrumentNames (void)
  {
          // Full channel list update...
-         QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+         const QList<QMdiSubWindow *>& wlist
+                 = m_pWorkspace->subWindowList();
          if (wlist.isEmpty())
                  return;
          m_pWorkspace->setUpdatesEnabled(false);
-         for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+         foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                 ChannelStrip *pChannelStrip = (ChannelStrip *) wlist.at(iChannel);
+                 ChannelStrip *pChannelStrip
+                         = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
                  if (pChannelStrip)
                          pChannelStrip->updateInstrumentName(true);
          }
-Line 2239 
 void MainForm::updateInstrumentNames (vo
+Line 2371 
 void MainForm::updateInstrumentNames (vo
  // Force update of the channels display font.
  void MainForm::updateDisplayFont (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          // Check if display font is legal.
          if (m_pOptions->sDisplayFont.isEmpty())
                  return;
          // Realize it.
          QFont font;
          if (!font.fromString(m_pOptions->sDisplayFont))
                  return;
          // Full channel list update...
-         QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+         const QList<QMdiSubWindow *>& wlist
+                 = m_pWorkspace->subWindowList();
          if (wlist.isEmpty())
                  return;
          m_pWorkspace->setUpdatesEnabled(false);
-         for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+         foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                 ChannelStrip *pChannelStrip = NULL;
+                 ChannelStrip *pChannelStrip
-                 QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+                         = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
-                 if (pMdiSubWindow)
-                         pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
                  if (pChannelStrip)
                          pChannelStrip->setDisplayFont(font);
          }
-Line 2272 
 void MainForm::updateDisplayFont (void)
+Line 2404 
 void MainForm::updateDisplayFont (void)
  void MainForm::updateDisplayEffect (void)
  {
          // Full channel list update...
-         QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+         const QList<QMdiSubWindow *>& wlist
+                 = m_pWorkspace->subWindowList();
          if (wlist.isEmpty())
                  return;
          m_pWorkspace->setUpdatesEnabled(false);
-         for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+         foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                 ChannelStrip *pChannelStrip = NULL;
+                 ChannelStrip *pChannelStrip
-                 QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+                         = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
-                 if (pMdiSubWindow)
-                         pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
                  if (pChannelStrip)
                          pChannelStrip->setDisplayEffect(m_pOptions->bDisplayEffect);
          }
-Line 2292 
 void MainForm::updateDisplayEffect (void
+Line 2423 
 void MainForm::updateDisplayEffect (void
  // Force update of the channels maximum volume setting.
  void MainForm::updateMaxVolume (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
  #ifdef CONFIG_VOLUME
-Line 2303 
 void MainForm::updateMaxVolume (void)
+Line 2434 
 void MainForm::updateMaxVolume (void)
  #endif
          // Full channel list update...
-         QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+         const QList<QMdiSubWindow *>& wlist
+                 = m_pWorkspace->subWindowList();
          if (wlist.isEmpty())
                  return;
          m_pWorkspace->setUpdatesEnabled(false);
-         for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+         foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                 ChannelStrip *pChannelStrip = NULL;
+                 ChannelStrip *pChannelStrip
-                 QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+                         = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
-                 if (pMdiSubWindow)
-                         pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
                  if (pChannelStrip)
                          pChannelStrip->setMaxVolume(m_pOptions->iMaxVolume);
          }
-Line 2321 
 void MainForm::updateMaxVolume (void)
+Line 2451 
 void MainForm::updateMaxVolume (void)
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- Messages window form handlers.
+ // QSampler::MainForm -- Messages window form handlers.
  // Messages output methods.
  void MainForm::appendMessages( const QString& s )
-Line 2346 
 void MainForm::appendMessagesText( const
+Line 2476 
 void MainForm::appendMessagesText( const
                  m_pMessages->appendMessagesText(s);
  }
- void MainForm::appendMessagesError( const QString& s )
+ void MainForm::appendMessagesError( const QString& sText )
  {
          if (m_pMessages)
                  m_pMessages->show();
-         appendMessagesColor(s.simplified(), "#ff0000");
+         appendMessagesColor(sText.simplified(), "#ff0000");
          // Make it look responsive...:)
          QApplication::processEvents(QEventLoop::ExcludeUserInputEvents);
-         QMessageBox::critical(this,
+         if (m_pOptions && m_pOptions->bConfirmError) {
-                 QSAMPLER_TITLE ": " + tr("Error"), s, QMessageBox::Cancel);
+                 const QString& sTitle = tr("Error");
+         #if 0
+                 QMessageBox::critical(this, sTitle, sText, QMessageBox::Cancel);
+         #else
+                 QMessageBox mbox(this);
+                 mbox.setIcon(QMessageBox::Critical);
+                 mbox.setWindowTitle(sTitle);
+                 mbox.setText(sText);
+                 mbox.setStandardButtons(QMessageBox::Cancel);
+                 QCheckBox cbox(tr("Don't show this again"));
+                 cbox.setChecked(false);
+                 cbox.blockSignals(true);
+                 mbox.addButton(&cbox, QMessageBox::ActionRole);
+                 if (mbox.exec() && cbox.isChecked())
+                         m_pOptions->bConfirmError = false;
+         #endif
+         }
  }
  // This is a special message format, just for client results.
  void MainForm::appendMessagesClient( const QString& s )
  {
-         if (m_pClient == NULL)
+         if (m_pClient == nullptr)
                  return;
          appendMessagesColor(s + QString(": %1 (errno=%2)")
-Line 2379 
 void MainForm::appendMessagesClient( con
+Line 2525 
 void MainForm::appendMessagesClient( con
  // Force update of the messages font.
  void MainForm::updateMessagesFont (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          if (m_pMessages && !m_pOptions->sMessagesFont.isEmpty()) {
-Line 2393 
 void MainForm::updateMessagesFont (void)
+Line 2539 
 void MainForm::updateMessagesFont (void)
  // Update messages window line limit.
  void MainForm::updateMessagesLimit (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          if (m_pMessages) {
-Line 2408 
 void MainForm::updateMessagesLimit (void
+Line 2554 
 void MainForm::updateMessagesLimit (void
  // Enablement of the messages capture feature.
  void MainForm::updateMessagesCapture (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          if (m_pMessages)
-Line 2417 
 void MainForm::updateMessagesCapture (vo
+Line 2563 
 void MainForm::updateMessagesCapture (vo
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- MDI channel strip management.
+ // QSampler::MainForm -- MDI channel strip management.
  // The channel strip creation executive.
  ChannelStrip *MainForm::createChannelStrip ( Channel *pChannel )
  {
-         if (m_pClient == NULL || pChannel == NULL)
+         if (m_pClient == nullptr || pChannel == nullptr)
-                 return NULL;
+                 return nullptr;
          // Add a new channel itema...
          ChannelStrip *pChannelStrip = new ChannelStrip();
-         if (pChannelStrip == NULL)
+         if (pChannelStrip == nullptr)
-                 return NULL;
+                 return nullptr;
          // Set some initial channel strip options...
          if (m_pOptions) {
-Line 2436 
 ChannelStrip *MainForm::createChannelStr
+Line 2582 
 ChannelStrip *MainForm::createChannelStr
                  pChannelStrip->setDisplayEffect(m_pOptions->bDisplayEffect);
                  // We'll need a display font.
                  QFont font;
-                 if (font.fromString(m_pOptions->sDisplayFont))
+                 if (!m_pOptions->sDisplayFont.isEmpty() &&
+                         font.fromString(m_pOptions->sDisplayFont))
                          pChannelStrip->setDisplayFont(font);
                  // Maximum allowed volume setting.
                  pChannelStrip->setMaxVolume(m_pOptions->iMaxVolume);
          }
          // Add it to workspace...
-         m_pWorkspace->addSubWindow(pChannelStrip,
+         QMdiSubWindow *pMdiSubWindow
-                 Qt::SubWindow | Qt::FramelessWindowHint);
+                 = m_pWorkspace->addSubWindow(pChannelStrip,
+                         Qt::SubWindow | Qt::FramelessWindowHint);
+         pMdiSubWindow->setAttribute(Qt::WA_DeleteOnClose);
          // Actual channel strip setup...
          pChannelStrip->setup(pChannel);
-Line 2468 
 void MainForm::destroyChannelStrip ( Cha
+Line 2617 
 void MainForm::destroyChannelStrip ( Cha
  {
          QMdiSubWindow *pMdiSubWindow
                  = static_cast<QMdiSubWindow *> (pChannelStrip->parentWidget());
-         if (pMdiSubWindow == NULL)
+         if (pMdiSubWindow == nullptr)
                  return;
          // Just delete the channel strip.
-Line 2476 
 void MainForm::destroyChannelStrip ( Cha
+Line 2625 
 void MainForm::destroyChannelStrip ( Cha
          delete pMdiSubWindow;
          // Do we auto-arrange?
-         if (m_pOptions && m_pOptions->bAutoArrange)
+         channelsArrangeAuto();
-                 channelsArrange();
-         stabilizeForm();
  }
-Line 2490 
 ChannelStrip *MainForm::activeChannelStr
+Line 2636 
 ChannelStrip *MainForm::activeChannelStr
          if (pMdiSubWindow)
                  return static_cast<ChannelStrip *> (pMdiSubWindow->widget());
          else
-                 return NULL;
+                 return nullptr;
  }
  // Retrieve a channel strip by index.
- ChannelStrip *MainForm::channelStripAt ( int iChannel )
+ ChannelStrip *MainForm::channelStripAt ( int iStrip )
  {
-         if (!m_pWorkspace) return NULL;
+         if (!m_pWorkspace) return nullptr;
-         QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+         const QList<QMdiSubWindow *>& wlist
+                 = m_pWorkspace->subWindowList();
          if (wlist.isEmpty())
-                 return NULL;
+                 return nullptr;
-         if (iChannel < 0 || iChannel >= wlist.size())
+         if (iStrip < 0 || iStrip >= wlist.count())
-                 return NULL;
+                 return nullptr;
-         QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+         QMdiSubWindow *pMdiSubWindow = wlist.at(iStrip);
          if (pMdiSubWindow)
                  return static_cast<ChannelStrip *> (pMdiSubWindow->widget());
          else
-                 return NULL;
+                 return nullptr;
  }
  // Retrieve a channel strip by sampler channel id.
  ChannelStrip *MainForm::channelStrip ( int iChannelID )
  {
-         QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+         const QList<QMdiSubWindow *>& wlist
+                 = m_pWorkspace->subWindowList();
          if (wlist.isEmpty())
-                 return NULL;
+                 return nullptr;
-         for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+         foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                 ChannelStrip *pChannelStrip = NULL;
+                 ChannelStrip *pChannelStrip
-                 QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+                         = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
-                 if (pMdiSubWindow)
-                         pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
                  if (pChannelStrip) {
                          Channel *pChannel = pChannelStrip->channel();
                          if (pChannel && pChannel->channelID() == iChannelID)
-Line 2534 
 ChannelStrip *MainForm::channelStrip ( i
+Line 2680 
 ChannelStrip *MainForm::channelStrip ( i
          }
          // Not found.
-         return NULL;
+         return nullptr;
  }
-Line 2545 
 void MainForm::channelsMenuAboutToShow (
+Line 2691 
 void MainForm::channelsMenuAboutToShow (
          m_ui.channelsMenu->addAction(m_ui.channelsArrangeAction);
          m_ui.channelsMenu->addAction(m_ui.channelsAutoArrangeAction);
-         QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+         const QList<QMdiSubWindow *>& wlist
+                 = m_pWorkspace->subWindowList();
          if (!wlist.isEmpty()) {
                  m_ui.channelsMenu->addSeparator();
-                 for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+                 int iStrip = 0;
-                         ChannelStrip *pChannelStrip = NULL;
+                 foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                         QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+                         ChannelStrip *pChannelStrip
-                         if (pMdiSubWindow)
+                                 = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
-                                 pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
                          if (pChannelStrip) {
                                  QAction *pAction = m_ui.channelsMenu->addAction(
                                          pChannelStrip->windowTitle(),
                                          this, SLOT(channelsMenuActivated()));
                                  pAction->setCheckable(true);
                                  pAction->setChecked(activeChannelStrip() == pChannelStrip);
-                                 pAction->setData(iChannel);
+                                 pAction->setData(iStrip);
                          }
+                         ++iStrip;
                  }
          }
  }
-Line 2571 
 void MainForm::channelsMenuActivated (vo
+Line 2718 
 void MainForm::channelsMenuActivated (vo
  {
          // Retrive channel index from action data...
          QAction *pAction = qobject_cast<QAction *> (sender());
-         if (pAction == NULL)
+         if (pAction == nullptr)
                  return;
          ChannelStrip *pChannelStrip = channelStripAt(pAction->data().toInt());
-Line 2583 
 void MainForm::channelsMenuActivated (vo
+Line 2730 
 void MainForm::channelsMenuActivated (vo
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- Timer stuff.
+ // QSampler::MainForm -- Timer stuff.
  // Set the pseudo-timer delay schedule.
  void MainForm::startSchedule ( int iStartDelay )
-Line 2602 
 void MainForm::stopSchedule (void)
+Line 2749 
 void MainForm::stopSchedule (void)
  // Timer slot funtion.
  void MainForm::timerSlot (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          // Is it the first shot on server start after a few delay?
-Line 2624 
 void MainForm::timerSlot (void)
+Line 2771 
 void MainForm::timerSlot (void)
                          ChannelStrip *pChannelStrip = iter.next();
                          // If successfull, remove from pending list...
                          if (pChannelStrip->updateChannelInfo()) {
-                                 int iChannelStrip = m_changedStrips.indexOf(pChannelStrip);
+                                 const int iChannelStrip = m_changedStrips.indexOf(pChannelStrip);
                                  if (iChannelStrip >= 0)
                                          m_changedStrips.removeAt(iChannelStrip);
                          }
-Line 2635 
 void MainForm::timerSlot (void)
+Line 2782 
 void MainForm::timerSlot (void)
                          if (m_iTimerSlot >= m_pOptions->iAutoRefreshTime)  {
                                  m_iTimerSlot = 0;
                                  // Update the channel stream usage for each strip...
-                                 QList<QMdiSubWindow *> wlist = m_pWorkspace->subWindowList();
+                                 const QList<QMdiSubWindow *>& wlist
-                                 for (int iChannel = 0; iChannel < (int) wlist.count(); ++iChannel) {
+                                         = m_pWorkspace->subWindowList();
-                                         ChannelStrip *pChannelStrip = NULL;
+                                 foreach (QMdiSubWindow *pMdiSubWindow, wlist) {
-                                         QMdiSubWindow *pMdiSubWindow = wlist.at(iChannel);
+                                         ChannelStrip *pChannelStrip
-                                         if (pMdiSubWindow)
+                                                 = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
-                                                 pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
                                          if (pChannelStrip && pChannelStrip->isVisible())
                                                  pChannelStrip->updateChannelUsage();
                                  }
                          }
                  }
+         #if CONFIG_LSCP_CLIENT_CONNECTION_LOST
+                 // If we lost connection to server: Try to automatically reconnect if we
+                 // did not start the server.
+                 //
+                 // TODO: If we started the server, then we might inform the user that
+                 // the server probably crashed and asking user ONCE whether we should
+                 // restart the server.
+                 if (lscp_client_connection_lost(m_pClient) && !m_pServer)
+                         startAutoReconnectClient();
+         #endif // CONFIG_LSCP_CLIENT_CONNECTION_LOST
          }
          // Register the next timer slot.
-Line 2654 
 void MainForm::timerSlot (void)
+Line 2811 
 void MainForm::timerSlot (void)
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- Server stuff.
+ // QSampler::MainForm -- Server stuff.
  // Start linuxsampler server...
  void MainForm::startServer (void)
  {
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return;
          // Aren't already a client, are we?
-Line 2669 
 void MainForm::startServer (void)
+Line 2826 
 void MainForm::startServer (void)
          // Is the server process instance still here?
          if (m_pServer) {
                  if (QMessageBox::warning(this,
-                         QSAMPLER_TITLE ": " + tr("Warning"),
+                         tr("Warning"),
                          tr("Could not start the LinuxSampler server.\n\n"
                          "Maybe it is already started."),
                          QMessageBox::Ok | QMessageBox::Cancel) == QMessageBox::Ok) {
-Line 2688 
 void MainForm::startServer (void)
+Line 2845 
 void MainForm::startServer (void)
          // OK. Let's build the startup process...
          m_pServer = new QProcess();
-         bForceServerStop = true;
+         m_bForceServerStop = true;
          // Setup stdout/stderr capture...
- //      if (m_pOptions->bStdoutCapture) {
+         m_pServer->setProcessChannelMode(QProcess::ForwardedChannels);
-                 m_pServer->setProcessChannelMode(QProcess::ForwardedChannels);
+         QObject::connect(m_pServer,
-                 QObject::connect(m_pServer,
+                 SIGNAL(readyReadStandardOutput()),
-                         SIGNAL(readyReadStandardOutput()),
+                 SLOT(readServerStdout()));
-                         SLOT(readServerStdout()));
+         QObject::connect(m_pServer,
-                 QObject::connect(m_pServer,
+                 SIGNAL(readyReadStandardError()),
-                         SIGNAL(readyReadStandardError()),
+                 SLOT(readServerStdout()));
-                         SLOT(readServerStdout()));
- //      }
          // The unforgiveable signal communication...
          QObject::connect(m_pServer,
-Line 2734 
 void MainForm::startServer (void)
+Line 2889 
 void MainForm::startServer (void)
  // Stop linuxsampler server...
- void MainForm::stopServer (bool bInteractive)
+ void MainForm::stopServer ( bool bInteractive )
  {
          // Stop client code.
          stopClient();
          if (m_pServer && bInteractive) {
                  if (QMessageBox::question(this,
-                         QSAMPLER_TITLE ": " + tr("The backend's fate ..."),
+                         tr("The backend's fate ..."),
                          tr("You have the option to keep the sampler backend (LinuxSampler)\n"
                          "running in the background. The sampler would continue to work\n"
                          "according to your current sampler session and you could alter the\n"
                          "sampler session at any time by relaunching QSampler.\n\n"
                          "Do you want LinuxSampler to stop?"),
-                         QMessageBox::Yes | QMessageBox::No) == QMessageBox::No)
+                         QMessageBox::Yes | QMessageBox::No,
-                 {
+                         QMessageBox::Yes) == QMessageBox::No) {
-                         bForceServerStop = false;
+                         m_bForceServerStop = false;
                  }
          }
+         bool bGraceWait = true;
          // And try to stop server.
-         if (m_pServer && bForceServerStop) {
+         if (m_pServer && m_bForceServerStop) {
                  appendMessages(tr("Server is stopping..."));
                  if (m_pServer->state() == QProcess::Running) {
-                 #if defined(WIN32)
+                 #if defined(__WIN32__) || defined(_WIN32) || defined(WIN32)
                          // Try harder...
                          m_pServer->kill();
                  #else
                          // Try softly...
                          m_pServer->terminate();
+                         bool bFinished = m_pServer->waitForFinished(QSAMPLER_TIMER_MSECS * 1000);
+                         if (bFinished) bGraceWait = false;
                  #endif
                  }
          }       // Do final processing anyway.
          else processServerExit();
          // Give it some time to terminate gracefully and stabilize...
-         QTime t;
+         if (bGraceWait) {
-         t.start();
+                 QTime t;
-         while (t.elapsed() < QSAMPLER_TIMER_MSECS)
+                 t.start();
-                 QApplication::processEvents(QEventLoop::ExcludeUserInputEvents);
+                 while (t.elapsed() < QSAMPLER_TIMER_MSECS)
+                         QApplication::processEvents(QEventLoop::ExcludeUserInputEvents);
+         }
  }
-Line 2794 
 void MainForm::processServerExit (void)
+Line 2955 
 void MainForm::processServerExit (void)
          if (m_pMessages)
                  m_pMessages->flushStdoutBuffer();
-         if (m_pServer && bForceServerStop) {
+         if (m_pServer && m_bForceServerStop) {
                  if (m_pServer->state() != QProcess::NotRunning) {
                          appendMessages(tr("Server is being forced..."));
                          // Force final server shutdown...
-Line 2810 
 void MainForm::processServerExit (void)
+Line 2971 
 void MainForm::processServerExit (void)
                          tr("Server was stopped with exit status %1.")
                          .arg(m_pServer->exitStatus()));
                  delete m_pServer;
-                 m_pServer = NULL;
+                 m_pServer = nullptr;
          }
          // Again, make status visible stable.
-Line 2819 
 void MainForm::processServerExit (void)
+Line 2980 
 void MainForm::processServerExit (void)
  //-------------------------------------------------------------------------
- // qsamplerMainForm -- Client stuff.
+ // QSampler::MainForm -- Client stuff.
  // The LSCP client callback procedure.
  lscp_status_t qsampler_client_callback ( lscp_client_t */*pClient*/,
          lscp_event_t event, const char *pchData, int cchData, void *pvData )
  {
          MainForm* pMainForm = (MainForm *) pvData;
-         if (pMainForm == NULL)
+         if (pMainForm == nullptr)
                  return LSCP_FAILED;
          // ATTN: DO NOT EVER call any GUI code here,
-Line 2840 
 lscp_status_t qsampler_client_callback (
+Line 3001 
 lscp_status_t qsampler_client_callback (
  // Start our almighty client...
- bool MainForm::startClient (void)
+ bool MainForm::startClient (bool bReconnectOnly)
  {
          // Have it a setup?
-         if (m_pOptions == NULL)
+         if (m_pOptions == nullptr)
                  return false;
          // Aren't we already started, are we?
-Line 2857 
 bool MainForm::startClient (void)
+Line 3018 
 bool MainForm::startClient (void)
          m_pClient = ::lscp_client_create(
                  m_pOptions->sServerHost.toUtf8().constData(),
                  m_pOptions->iServerPort, qsampler_client_callback, this);
-         if (m_pClient == NULL) {
+         if (m_pClient == nullptr) {
                  // Is this the first try?
                  // maybe we need to start a local server...
                  if ((m_pServer && m_pServer->state() == QProcess::Running)
-                         || !m_pOptions->bServerStart) {
+                         || !m_pOptions->bServerStart || bReconnectOnly)
-                         appendMessagesError(
+                 {
-                                 tr("Could not connect to server as client.\n\nSorry."));
+                         // if this method is called from autoReconnectClient()
+                         // then don't bother user with an error message...
+                         if (!bReconnectOnly) {
+                                 appendMessagesError(
+                                         tr("Could not connect to server as client.\n\nSorry.")
+                                 );
+                         }
                  } else {
                          startServer();
                  }
-Line 2871 
 bool MainForm::startClient (void)
+Line 3038 
 bool MainForm::startClient (void)
                  stabilizeForm();
                  return false;
          }
          // Just set receive timeout value, blindly.
          ::lscp_client_set_timeout(m_pClient, m_pOptions->iServerTimeout);
          appendMessages(
-Line 2926 
 bool MainForm::startClient (void)
+Line 3094 
 bool MainForm::startClient (void)
          if (!m_pOptions->sSessionFile.isEmpty()) {
                  // Just load the prabably startup session...
                  if (loadSessionFile(m_pOptions->sSessionFile)) {
-                         m_pOptions->sSessionFile = QString::null;
+                         m_pOptions->sSessionFile = QString();
                          return true;
                  }
          }
-Line 2942 
 bool MainForm::startClient (void)
+Line 3110 
 bool MainForm::startClient (void)
  // Stop client...
  void MainForm::stopClient (void)
  {
-         if (m_pClient == NULL)
+         if (m_pClient == nullptr)
                  return;
          // Log prepare here.
-Line 2974 
 void MainForm::stopClient (void)
+Line 3142 
 void MainForm::stopClient (void)
          ::lscp_client_unsubscribe(m_pClient, LSCP_EVENT_CHANNEL_INFO);
          ::lscp_client_unsubscribe(m_pClient, LSCP_EVENT_CHANNEL_COUNT);
          ::lscp_client_destroy(m_pClient);
-         m_pClient = NULL;
+         m_pClient = nullptr;
          // Hard-notify instrumnet and device configuration forms,
          // if visible, that we're running out...
-Line 2991 
 void MainForm::stopClient (void)
+Line 3159 
 void MainForm::stopClient (void)
  }
+ void MainForm::startAutoReconnectClient (void)
+ {
+         stopClient();
+         appendMessages(tr("Trying to reconnect..."));
+         QTimer::singleShot(QSAMPLER_TIMER_MSECS, this, SLOT(autoReconnectClient()));
+ }
+ void MainForm::autoReconnectClient (void)
+ {
+         const bool bSuccess = startClient(true);
+         if (!bSuccess)
+                 QTimer::singleShot(QSAMPLER_TIMER_MSECS, this, SLOT(autoReconnectClient()));
+ }
  // Channel strip activation/selection.
  void MainForm::activateStrip ( QMdiSubWindow *pMdiSubWindow )
  {
-         ChannelStrip *pChannelStrip = NULL;
+         ChannelStrip *pChannelStrip = nullptr;
          if (pMdiSubWindow)
                  pChannelStrip = static_cast<ChannelStrip *> (pMdiSubWindow->widget());
          if (pChannelStrip)
-Line 3004 
 void MainForm::activateStrip ( QMdiSubWi
+Line 3188 
 void MainForm::activateStrip ( QMdiSubWi
  }
+ // Channel toolbar orientation change.
+ void MainForm::channelsToolbarOrientation ( Qt::Orientation orientation )
+ {
+ #ifdef CONFIG_VOLUME
+         m_pVolumeSlider->setOrientation(orientation);
+         if (orientation == Qt::Horizontal) {
+                 m_pVolumeSlider->setMinimumHeight(24);
+                 m_pVolumeSlider->setMaximumHeight(32);
+                 m_pVolumeSlider->setMinimumWidth(120);
+                 m_pVolumeSlider->setMaximumWidth(640);
+                 m_pVolumeSpinBox->setMaximumWidth(64);
+                 m_pVolumeSpinBox->setButtonSymbols(QSpinBox::UpDownArrows);
+         } else {
+                 m_pVolumeSlider->setMinimumHeight(120);
+                 m_pVolumeSlider->setMaximumHeight(480);
+                 m_pVolumeSlider->setMinimumWidth(24);
+                 m_pVolumeSlider->setMaximumWidth(32);
+                 m_pVolumeSpinBox->setMaximumWidth(32);
+                 m_pVolumeSpinBox->setButtonSymbols(QSpinBox::NoButtons);
+         }
+ #endif
+ }
  } // namespace QSampler

 Legend:



Removed from v.2459
 


changed lines


 
Added in v.3681
 Legend:



Removed from v.2459
 


changed lines


 
Added in v.3681
-Removed from v.2459
+Added in v.3681

	ViewVC Help
Powered by ViewVC