diff --git a/src/GUI/filebrowser.cpp b/src/GUI/filebrowser.cpp index a56df027..c1d66238 100644 --- a/src/GUI/filebrowser.cpp +++ b/src/GUI/filebrowser.cpp @@ -7,8 +7,8 @@ FileBrowser::FileBrowser(QObject *parent) : QObject(parent) { _watcher = new QFileSystemWatcher(this); - connect(_watcher, SIGNAL(directoryChanged(const QString &)), this, - SLOT(reloadDirectory(const QString &))); + connect(_watcher, &QFileSystemWatcher::directoryChanged, this, + &FileBrowser::reloadDirectory); _index = -1; } diff --git a/src/GUI/fileselectwidget.cpp b/src/GUI/fileselectwidget.cpp index a0da7ab6..3a2458c6 100644 --- a/src/GUI/fileselectwidget.cpp +++ b/src/GUI/fileselectwidget.cpp @@ -21,7 +21,7 @@ FileSelectWidget::FileSelectWidget(QWidget *parent) : QWidget(parent) _button = new QToolButton(); _button->setText("..."); #endif // Q_OS_WIN32 - connect(_button, SIGNAL(clicked()), this, SLOT(browse())); + connect(_button, &QToolButton::clicked, this, &FileSelectWidget::browse); QHBoxLayout *layout = new QHBoxLayout(); layout->setContentsMargins(QMargins()); diff --git a/src/GUI/graphview.cpp b/src/GUI/graphview.cpp index 68da6fd5..9837f962 100644 --- a/src/GUI/graphview.cpp +++ b/src/GUI/graphview.cpp @@ -56,8 +56,8 @@ GraphView::GraphView(QWidget *parent) _message->setBrush(QPalette().brush(QPalette::Disabled, QPalette::WindowText)); - connect(_slider, SIGNAL(positionChanged(const QPointF&)), this, - SLOT(emitSliderPositionChanged(const QPointF&))); + connect(_slider, &SliderItem::positionChanged, this, + &GraphView::emitSliderPositionChanged); _width = 1; diff --git a/src/GUI/gui.cpp b/src/GUI/gui.cpp index f0ab6fa7..fdb2a08f 100644 --- a/src/GUI/gui.cpp +++ b/src/GUI/gui.cpp @@ -115,8 +115,7 @@ TreeNode GUI::createMapActions() { _mapsActionGroup = new QActionGroup(this); _mapsActionGroup->setExclusive(true); - connect(_mapsActionGroup, SIGNAL(triggered(QAction*)), this, - SLOT(mapChanged(QAction*))); + connect(_mapsActionGroup, &QActionGroup::triggered, this, &GUI::mapChanged); QString mapDir(ProgramPaths::mapDir()); if (mapDir.isNull()) @@ -137,7 +136,7 @@ TreeNode GUI::createMapActionsNode(const TreeNode &node) Map *map = node.items().at(i); if (map->isValid()) { MapAction *a = new MapAction(map, _mapsActionGroup); - connect(a, SIGNAL(loaded()), this, SLOT(mapInitialized())); + connect(a, &MapAction::loaded, this, &GUI::mapInitialized); tree.addItem(a); } else { qWarning("%s: %s", qPrintable(map->path()), @@ -169,8 +168,8 @@ TreeNode GUI::createPOIActions() { _poisActionGroup = new QActionGroup(this); _poisActionGroup->setExclusive(false); - connect(_poisActionGroup, SIGNAL(triggered(QAction*)), this, - SLOT(poiFileChecked(QAction*))); + connect(_poisActionGroup, &QActionGroup::triggered, this, + &GUI::poiFileChecked); TreeNode poiFiles; QString poiDir(ProgramPaths::poiDir()); @@ -209,64 +208,65 @@ void GUI::createActions(TreeNode &mapActions, _exitAction = new QAction(QIcon(QUIT_ICON), tr("Quit"), this); _exitAction->setShortcut(QUIT_SHORTCUT); _exitAction->setMenuRole(QAction::QuitRole); - connect(_exitAction, SIGNAL(triggered()), this, SLOT(close())); + connect(_exitAction, &QAction::triggered, this, &GUI::close); addAction(_exitAction); // Help & About _pathsAction = new QAction(tr("Paths"), this); _pathsAction->setMenuRole(QAction::NoRole); - connect(_pathsAction, SIGNAL(triggered()), this, SLOT(paths())); + connect(_pathsAction, &QAction::triggered, this, &GUI::paths); _keysAction = new QAction(tr("Keyboard controls"), this); _keysAction->setMenuRole(QAction::NoRole); - connect(_keysAction, SIGNAL(triggered()), this, SLOT(keys())); + connect(_keysAction, &QAction::triggered, this, &GUI::keys); _aboutAction = new QAction(QIcon(APP_ICON), tr("About GPXSee"), this); _aboutAction->setMenuRole(QAction::AboutRole); - connect(_aboutAction, SIGNAL(triggered()), this, SLOT(about())); + connect(_aboutAction, &QAction::triggered, this, &GUI::about); // File actions _openFileAction = new QAction(QIcon(OPEN_FILE_ICON), tr("Open..."), this); _openFileAction->setMenuRole(QAction::NoRole); _openFileAction->setShortcut(OPEN_SHORTCUT); - connect(_openFileAction, SIGNAL(triggered()), this, SLOT(openFile())); + connect(_openFileAction, &QAction::triggered, this, + QOverload<>::of(&GUI::openFile)); addAction(_openFileAction); _printFileAction = new QAction(QIcon(PRINT_FILE_ICON), tr("Print..."), this); _printFileAction->setMenuRole(QAction::NoRole); _printFileAction->setActionGroup(_fileActionGroup); - connect(_printFileAction, SIGNAL(triggered()), this, SLOT(printFile())); + connect(_printFileAction, &QAction::triggered, this, &GUI::printFile); addAction(_printFileAction); _exportPDFFileAction = new QAction(QIcon(EXPORT_FILE_ICON), tr("Export to PDF..."), this); _exportPDFFileAction->setMenuRole(QAction::NoRole); _exportPDFFileAction->setShortcut(PDF_EXPORT_SHORTCUT); _exportPDFFileAction->setActionGroup(_fileActionGroup); - connect(_exportPDFFileAction, SIGNAL(triggered()), this, SLOT(exportPDFFile())); + connect(_exportPDFFileAction, &QAction::triggered, this, &GUI::exportPDFFile); addAction(_exportPDFFileAction); _exportPNGFileAction = new QAction(QIcon(EXPORT_FILE_ICON), tr("Export to PNG..."), this); _exportPNGFileAction->setMenuRole(QAction::NoRole); _exportPNGFileAction->setShortcut(PNG_EXPORT_SHORTCUT); _exportPNGFileAction->setActionGroup(_fileActionGroup); - connect(_exportPNGFileAction, SIGNAL(triggered()), this, SLOT(exportPNGFile())); + connect(_exportPNGFileAction, &QAction::triggered, this, &GUI::exportPNGFile); addAction(_exportPNGFileAction); _closeFileAction = new QAction(QIcon(CLOSE_FILE_ICON), tr("Close"), this); _closeFileAction->setMenuRole(QAction::NoRole); _closeFileAction->setShortcut(CLOSE_SHORTCUT); _closeFileAction->setActionGroup(_fileActionGroup); - connect(_closeFileAction, SIGNAL(triggered()), this, SLOT(closeAll())); + connect(_closeFileAction, &QAction::triggered, this, &GUI::closeAll); addAction(_closeFileAction); _reloadFileAction = new QAction(QIcon(RELOAD_FILE_ICON), tr("Reload"), this); _reloadFileAction->setMenuRole(QAction::NoRole); _reloadFileAction->setShortcut(RELOAD_SHORTCUT); _reloadFileAction->setActionGroup(_fileActionGroup); - connect(_reloadFileAction, SIGNAL(triggered()), this, SLOT(reloadFiles())); + connect(_reloadFileAction, &QAction::triggered, this, &GUI::reloadFiles); addAction(_reloadFileAction); _statisticsAction = new QAction(tr("Statistics..."), this); _statisticsAction->setMenuRole(QAction::NoRole); _statisticsAction->setShortcut(STATISTICS_SHORTCUT); _statisticsAction->setActionGroup(_fileActionGroup); - connect(_statisticsAction, SIGNAL(triggered()), this, SLOT(statistics())); + connect(_statisticsAction, &QAction::triggered, this, &GUI::statistics); addAction(_statisticsAction); // POI actions @@ -274,33 +274,33 @@ void GUI::createActions(TreeNode &mapActions, _openPOIAction = new QAction(QIcon(OPEN_FILE_ICON), tr("Load POI file..."), this); _openPOIAction->setMenuRole(QAction::NoRole); - connect(_openPOIAction, SIGNAL(triggered()), this, SLOT(openPOIFile())); + connect(_openPOIAction, &QAction::triggered, this, + QOverload<>::of(&GUI::openPOIFile)); _selectAllPOIAction = new QAction(tr("Select all files"), this); _selectAllPOIAction->setMenuRole(QAction::NoRole); _selectAllPOIAction->setEnabled(!_poisActionGroup->actions().isEmpty()); - connect(_selectAllPOIAction, SIGNAL(triggered()), this, - SLOT(selectAllPOIs())); + connect(_selectAllPOIAction, &QAction::triggered, this, + &GUI::selectAllPOIs); _unselectAllPOIAction = new QAction(tr("Unselect all files"), this); _unselectAllPOIAction->setMenuRole(QAction::NoRole); _unselectAllPOIAction->setEnabled(_selectAllPOIAction->isEnabled()); - connect(_unselectAllPOIAction, SIGNAL(triggered()), this, - SLOT(unselectAllPOIs())); + connect(_unselectAllPOIAction, &QAction::triggered, this, + &GUI::unselectAllPOIs); _overlapPOIAction = new QAction(tr("Overlap POIs"), this); _overlapPOIAction->setMenuRole(QAction::NoRole); _overlapPOIAction->setCheckable(true); - connect(_overlapPOIAction, SIGNAL(triggered(bool)), _mapView, - SLOT(showOverlappedPOIs(bool))); + connect(_overlapPOIAction, &QAction::triggered, _mapView, + &MapView::showOverlappedPOIs); _showPOILabelsAction = new QAction(tr("Show POI labels"), this); _showPOILabelsAction->setMenuRole(QAction::NoRole); _showPOILabelsAction->setCheckable(true); - connect(_showPOILabelsAction, SIGNAL(triggered(bool)), _mapView, - SLOT(showPOILabels(bool))); + connect(_showPOILabelsAction, &QAction::triggered, _mapView, + &MapView::showPOILabels); _showPOIAction = new QAction(QIcon(SHOW_POI_ICON), tr("Show POIs"), this); _showPOIAction->setMenuRole(QAction::NoRole); _showPOIAction->setCheckable(true); _showPOIAction->setShortcut(SHOW_POI_SHORTCUT); - connect(_showPOIAction, SIGNAL(triggered(bool)), _mapView, - SLOT(showPOI(bool))); + connect(_showPOIAction, &QAction::triggered, _mapView, &MapView::showPOI); addAction(_showPOIAction); // Map actions @@ -311,77 +311,76 @@ void GUI::createActions(TreeNode &mapActions, _showMapAction->setMenuRole(QAction::NoRole); _showMapAction->setCheckable(true); _showMapAction->setShortcut(SHOW_MAP_SHORTCUT); - connect(_showMapAction, SIGNAL(triggered(bool)), _mapView, - SLOT(showMap(bool))); + connect(_showMapAction, &QAction::triggered, _mapView, + &MapView::showMap); addAction(_showMapAction); _loadMapAction = new QAction(QIcon(OPEN_FILE_ICON), tr("Load map..."), this); _loadMapAction->setMenuRole(QAction::NoRole); - connect(_loadMapAction, SIGNAL(triggered()), this, SLOT(loadMap())); + connect(_loadMapAction, &QAction::triggered, this, + QOverload<>::of(&GUI::loadMap)); _loadMapDirAction = new QAction(QIcon(OPEN_FILE_ICON), tr("Load map directory..."), this); _loadMapDirAction->setMenuRole(QAction::NoRole); - connect(_loadMapDirAction, SIGNAL(triggered()), this, SLOT(loadMapDir())); + connect(_loadMapDirAction, &QAction::triggered, this, &GUI::loadMapDir); _clearMapCacheAction = new QAction(tr("Clear tile cache"), this); _clearMapCacheAction->setEnabled(false); _clearMapCacheAction->setMenuRole(QAction::NoRole); - connect(_clearMapCacheAction, SIGNAL(triggered()), this, - SLOT(clearMapCache())); + connect(_clearMapCacheAction, &QAction::triggered, this, + &GUI::clearMapCache); _nextMapAction = new QAction(tr("Next map"), this); _nextMapAction->setMenuRole(QAction::NoRole); _nextMapAction->setShortcut(NEXT_MAP_SHORTCUT); - connect(_nextMapAction, SIGNAL(triggered()), this, SLOT(nextMap())); + connect(_nextMapAction, &QAction::triggered, this, &GUI::nextMap); addAction(_nextMapAction); _prevMapAction = new QAction(tr("Next map"), this); _prevMapAction->setMenuRole(QAction::NoRole); _prevMapAction->setShortcut(PREV_MAP_SHORTCUT); - connect(_prevMapAction, SIGNAL(triggered()), this, SLOT(prevMap())); + connect(_prevMapAction, &QAction::triggered, this, &GUI::prevMap); addAction(_prevMapAction); _showCoordinatesAction = new QAction(tr("Show cursor coordinates"), this); _showCoordinatesAction->setMenuRole(QAction::NoRole); _showCoordinatesAction->setCheckable(true); - connect(_showCoordinatesAction, SIGNAL(triggered(bool)), _mapView, - SLOT(showCoordinates(bool))); + connect(_showCoordinatesAction, &QAction::triggered, _mapView, + &MapView::showCoordinates); // Data actions _showTracksAction = new QAction(tr("Show tracks"), this); _showTracksAction->setMenuRole(QAction::NoRole); _showTracksAction->setCheckable(true); - connect(_showTracksAction, SIGNAL(triggered(bool)), this, - SLOT(showTracks(bool))); + connect(_showTracksAction, &QAction::triggered, this, &GUI::showTracks); _showRoutesAction = new QAction(tr("Show routes"), this); _showRoutesAction->setMenuRole(QAction::NoRole); _showRoutesAction->setCheckable(true); - connect(_showRoutesAction, SIGNAL(triggered(bool)), this, - SLOT(showRoutes(bool))); + connect(_showRoutesAction, &QAction::triggered, this, &GUI::showRoutes); _showWaypointsAction = new QAction(tr("Show waypoints"), this); _showWaypointsAction->setMenuRole(QAction::NoRole); _showWaypointsAction->setCheckable(true); - connect(_showWaypointsAction, SIGNAL(triggered(bool)), _mapView, - SLOT(showWaypoints(bool))); + connect(_showWaypointsAction, &QAction::triggered, _mapView, + &MapView::showWaypoints); _showAreasAction = new QAction(tr("Show areas"), this); _showAreasAction->setMenuRole(QAction::NoRole); _showAreasAction->setCheckable(true); - connect(_showAreasAction, SIGNAL(triggered(bool)), _mapView, - SLOT(showAreas(bool))); + connect(_showAreasAction, &QAction::triggered, _mapView, + &MapView::showAreas); _showWaypointLabelsAction = new QAction(tr("Waypoint labels"), this); _showWaypointLabelsAction->setMenuRole(QAction::NoRole); _showWaypointLabelsAction->setCheckable(true); - connect(_showWaypointLabelsAction, SIGNAL(triggered(bool)), _mapView, - SLOT(showWaypointLabels(bool))); + connect(_showWaypointLabelsAction, &QAction::triggered, _mapView, + &MapView::showWaypointLabels); _showRouteWaypointsAction = new QAction(tr("Route waypoints"), this); _showRouteWaypointsAction->setMenuRole(QAction::NoRole); _showRouteWaypointsAction->setCheckable(true); - connect(_showRouteWaypointsAction, SIGNAL(triggered(bool)), _mapView, - SLOT(showRouteWaypoints(bool))); + connect(_showRouteWaypointsAction, &QAction::triggered, _mapView, + &MapView::showRouteWaypoints); _showTicksAction = new QAction(tr("km/mi markers"), this); _showTicksAction->setMenuRole(QAction::NoRole); _showTicksAction->setCheckable(true); - connect(_showTicksAction, SIGNAL(triggered(bool)), _mapView, - SLOT(showTicks(bool))); + connect(_showTicksAction, &QAction::triggered, _mapView, + &MapView::showTicks); QActionGroup *markerInfoGroup = new QActionGroup(this); - connect(markerInfoGroup, SIGNAL(triggered(QAction*)), this, - SLOT(showPathMarkerInfo(QAction*))); + connect(markerInfoGroup, &QActionGroup::triggered, this, + &GUI::showPathMarkerInfo); _hideMarkersAction = new QAction(tr("Do not show"), this); _hideMarkersAction->setMenuRole(QAction::NoRole); _hideMarkersAction->setCheckable(true); @@ -405,8 +404,7 @@ void GUI::createActions(TreeNode &mapActions, _showGraphsAction->setMenuRole(QAction::NoRole); _showGraphsAction->setCheckable(true); _showGraphsAction->setShortcut(SHOW_GRAPHS_SHORTCUT); - connect(_showGraphsAction, SIGNAL(triggered(bool)), this, - SLOT(showGraphs(bool))); + connect(_showGraphsAction, &QAction::triggered, this, &GUI::showGraphs); addAction(_showGraphsAction); ag = new QActionGroup(this); ag->setExclusive(true); @@ -414,117 +412,110 @@ void GUI::createActions(TreeNode &mapActions, _distanceGraphAction->setMenuRole(QAction::NoRole); _distanceGraphAction->setCheckable(true); _distanceGraphAction->setActionGroup(ag); - connect(_distanceGraphAction, SIGNAL(triggered()), this, - SLOT(setDistanceGraph())); + connect(_distanceGraphAction, &QAction::triggered, this, + &GUI::setDistanceGraph); addAction(_distanceGraphAction); _timeGraphAction = new QAction(tr("Time"), this); _timeGraphAction->setMenuRole(QAction::NoRole); _timeGraphAction->setCheckable(true); _timeGraphAction->setActionGroup(ag); - connect(_timeGraphAction, SIGNAL(triggered()), this, - SLOT(setTimeGraph())); + connect(_timeGraphAction, &QAction::triggered, this, &GUI::setTimeGraph); addAction(_timeGraphAction); _showGraphGridAction = new QAction(tr("Show grid"), this); _showGraphGridAction->setMenuRole(QAction::NoRole); _showGraphGridAction->setCheckable(true); - connect(_showGraphGridAction, SIGNAL(triggered(bool)), this, - SLOT(showGraphGrids(bool))); + connect(_showGraphGridAction, &QAction::triggered, this, + &GUI::showGraphGrids); _showGraphSliderInfoAction = new QAction(tr("Show slider info"), this); _showGraphSliderInfoAction->setMenuRole(QAction::NoRole); _showGraphSliderInfoAction->setCheckable(true); - connect(_showGraphSliderInfoAction, SIGNAL(triggered(bool)), this, - SLOT(showGraphSliderInfo(bool))); + connect(_showGraphSliderInfoAction, &QAction::triggered, this, + &GUI::showGraphSliderInfo); // Settings actions _showToolbarsAction = new QAction(tr("Show toolbars"), this); _showToolbarsAction->setMenuRole(QAction::NoRole); _showToolbarsAction->setCheckable(true); - connect(_showToolbarsAction, SIGNAL(triggered(bool)), this, - SLOT(showToolbars(bool))); + connect(_showToolbarsAction, &QAction::triggered, this, &GUI::showToolbars); ag = new QActionGroup(this); ag->setExclusive(true); _totalTimeAction = new QAction(tr("Total time"), this); _totalTimeAction->setMenuRole(QAction::NoRole); _totalTimeAction->setCheckable(true); _totalTimeAction->setActionGroup(ag); - connect(_totalTimeAction, SIGNAL(triggered()), this, - SLOT(setTotalTime())); + connect(_totalTimeAction, &QAction::triggered, this, &GUI::setTotalTime); _movingTimeAction = new QAction(tr("Moving time"), this); _movingTimeAction->setMenuRole(QAction::NoRole); _movingTimeAction->setCheckable(true); _movingTimeAction->setActionGroup(ag); - connect(_movingTimeAction, SIGNAL(triggered()), this, - SLOT(setMovingTime())); + connect(_movingTimeAction, &QAction::triggered, this, &GUI::setMovingTime); ag = new QActionGroup(this); ag->setExclusive(true); _metricUnitsAction = new QAction(tr("Metric"), this); _metricUnitsAction->setMenuRole(QAction::NoRole); _metricUnitsAction->setCheckable(true); _metricUnitsAction->setActionGroup(ag); - connect(_metricUnitsAction, SIGNAL(triggered()), this, - SLOT(setMetricUnits())); + connect(_metricUnitsAction, &QAction::triggered, this, &GUI::setMetricUnits); _imperialUnitsAction = new QAction(tr("Imperial"), this); _imperialUnitsAction->setMenuRole(QAction::NoRole); _imperialUnitsAction->setCheckable(true); _imperialUnitsAction->setActionGroup(ag); - connect(_imperialUnitsAction, SIGNAL(triggered()), this, - SLOT(setImperialUnits())); + connect(_imperialUnitsAction, &QAction::triggered, this, + &GUI::setImperialUnits); _nauticalUnitsAction = new QAction(tr("Nautical"), this); _nauticalUnitsAction->setMenuRole(QAction::NoRole); _nauticalUnitsAction->setCheckable(true); _nauticalUnitsAction->setActionGroup(ag); - connect(_nauticalUnitsAction, SIGNAL(triggered()), this, - SLOT(setNauticalUnits())); + connect(_nauticalUnitsAction, &QAction::triggered, this, + &GUI::setNauticalUnits); ag = new QActionGroup(this); ag->setExclusive(true); _decimalDegreesAction = new QAction(tr("Decimal degrees (DD)"), this); _decimalDegreesAction->setMenuRole(QAction::NoRole); _decimalDegreesAction->setCheckable(true); _decimalDegreesAction->setActionGroup(ag); - connect(_decimalDegreesAction, SIGNAL(triggered()), this, - SLOT(setDecimalDegrees())); + connect(_decimalDegreesAction, &QAction::triggered, this, + &GUI::setDecimalDegrees); _degreesMinutesAction = new QAction(tr("Degrees and decimal minutes (DMM)"), this); _degreesMinutesAction->setMenuRole(QAction::NoRole); _degreesMinutesAction->setCheckable(true); _degreesMinutesAction->setActionGroup(ag); - connect(_degreesMinutesAction, SIGNAL(triggered()), this, - SLOT(setDegreesMinutes())); + connect(_degreesMinutesAction, &QAction::triggered, this, + &GUI::setDegreesMinutes); _dmsAction = new QAction(tr("Degrees, minutes, seconds (DMS)"), this); _dmsAction->setMenuRole(QAction::NoRole); _dmsAction->setCheckable(true); _dmsAction->setActionGroup(ag); - connect(_dmsAction, SIGNAL(triggered()), this, SLOT(setDMS())); + connect(_dmsAction, &QAction::triggered, this, &GUI::setDMS); _fullscreenAction = new QAction(QIcon(FULLSCREEN_ICON), tr("Fullscreen mode"), this); _fullscreenAction->setMenuRole(QAction::NoRole); _fullscreenAction->setCheckable(true); _fullscreenAction->setShortcut(FULLSCREEN_SHORTCUT); - connect(_fullscreenAction, SIGNAL(triggered(bool)), this, - SLOT(showFullscreen(bool))); + connect(_fullscreenAction, &QAction::triggered, this, &GUI::showFullscreen); addAction(_fullscreenAction); _openOptionsAction = new QAction(tr("Options..."), this); _openOptionsAction->setMenuRole(QAction::PreferencesRole); - connect(_openOptionsAction, SIGNAL(triggered()), this, - SLOT(openOptions())); + connect(_openOptionsAction, &QAction::triggered, this, &GUI::openOptions); // Navigation actions _nextAction = new QAction(QIcon(NEXT_FILE_ICON), tr("Next"), this); _nextAction->setActionGroup(_navigationActionGroup); _nextAction->setMenuRole(QAction::NoRole); - connect(_nextAction, SIGNAL(triggered()), this, SLOT(next())); + connect(_nextAction, &QAction::triggered, this, &GUI::next); _prevAction = new QAction(QIcon(PREV_FILE_ICON), tr("Previous"), this); _prevAction->setMenuRole(QAction::NoRole); _prevAction->setActionGroup(_navigationActionGroup); - connect(_prevAction, SIGNAL(triggered()), this, SLOT(prev())); + connect(_prevAction, &QAction::triggered, this, &GUI::prev); _lastAction = new QAction(QIcon(LAST_FILE_ICON), tr("Last"), this); _lastAction->setMenuRole(QAction::NoRole); _lastAction->setActionGroup(_navigationActionGroup); - connect(_lastAction, SIGNAL(triggered()), this, SLOT(last())); + connect(_lastAction, &QAction::triggered, this, &GUI::last); _firstAction = new QAction(QIcon(FIRST_FILE_ICON), tr("First"), this); _firstAction->setMenuRole(QAction::NoRole); _firstAction->setActionGroup(_navigationActionGroup); - connect(_firstAction, SIGNAL(triggered()), this, SLOT(first())); + connect(_firstAction, &QAction::triggered, this, &GUI::first); } void GUI::createMapNodeMenu(const TreeNode &node, QMenu *menu) @@ -697,8 +688,8 @@ void GUI::createGraphTabs() _graphTabWidget->setDocumentMode(true); #endif // Q_OS_MAC - connect(_graphTabWidget, SIGNAL(currentChanged(int)), this, - SLOT(graphChanged(int))); + connect(_graphTabWidget, &QTabWidget::currentChanged, this, + &GUI::graphChanged); _tabs.append(new ElevationGraph(_graphTabWidget)); _tabs.append(new SpeedGraph(_graphTabWidget)); @@ -709,8 +700,8 @@ void GUI::createGraphTabs() _tabs.append(new GearRatioGraph(_graphTabWidget)); for (int i = 0; i < _tabs.size(); i++) - connect(_tabs.at(i), SIGNAL(sliderPositionChanged(qreal)), _mapView, - SLOT(setMarkerPosition(qreal))); + connect(_tabs.at(i), &GraphTab::sliderPositionChanged, _mapView, + &MapView::setMarkerPosition); } void GUI::createStatusBar() @@ -1553,7 +1544,7 @@ bool GUI::loadMapNode(const TreeNode &node, MapAction *&action, _showMapAction->setEnabled(true); _clearMapCacheAction->setEnabled(true); } else - connect(a, SIGNAL(loaded()), this, SLOT(mapLoaded())); + connect(a, &MapAction::loaded, this, &GUI::mapLoaded); } } else { valid = true; @@ -1637,7 +1628,7 @@ void GUI::loadMapDirNode(const TreeNode &node, QList &actions _clearMapCacheAction->setEnabled(true); actions.append(a); } else - connect(a, SIGNAL(loaded()), this, SLOT(mapLoadedDir())); + connect(a, &MapAction::loaded, this, &GUI::mapLoadedDir); } _areaCount++; @@ -2689,10 +2680,9 @@ void GUI::show() QMainWindow::show(); QWindow *w = windowHandle(); - connect(w->screen(), SIGNAL(logicalDotsPerInchChanged(qreal)), this, - SLOT(logicalDotsPerInchChanged(qreal))); - connect(w, SIGNAL(screenChanged(QScreen*)), this, - SLOT(screenChanged(QScreen*))); + connect(w->screen(), &QScreen::logicalDotsPerInchChanged, this, + &GUI::logicalDotsPerInchChanged); + connect(w, &QWindow::screenChanged, this, &GUI::screenChanged); _mapView->fitContentToSize(); } @@ -2704,8 +2694,8 @@ void GUI::screenChanged(QScreen *screen) disconnect(SIGNAL(logicalDotsPerInchChanged(qreal)), this, SLOT(logicalDotsPerInchChanged(qreal))); - connect(screen, SIGNAL(logicalDotsPerInchChanged(qreal)), this, - SLOT(logicalDotsPerInchChanged(qreal))); + connect(screen, &QScreen::logicalDotsPerInchChanged, this, + &GUI::logicalDotsPerInchChanged); } void GUI::logicalDotsPerInchChanged(qreal dpi) diff --git a/src/GUI/mapaction.h b/src/GUI/mapaction.h index 33facc37..ce9ba961 100644 --- a/src/GUI/mapaction.h +++ b/src/GUI/mapaction.h @@ -18,7 +18,7 @@ public: setMenuRole(QAction::NoRole); setCheckable(true); - connect(map, SIGNAL(mapLoaded()), this, SLOT(mapLoaded())); + connect(map, &Map::mapLoaded, this, &MapAction::mapLoaded); } signals: diff --git a/src/GUI/mapitem.cpp b/src/GUI/mapitem.cpp index a7a46d2c..57eb55c6 100644 --- a/src/GUI/mapitem.cpp +++ b/src/GUI/mapitem.cpp @@ -91,7 +91,7 @@ MapItem::MapItem(MapAction *action, Map *map, GraphicsItem *parent) _fileName = src->path(); _bounds = src->llBounds(); - connect(this, SIGNAL(triggered()), action, SLOT(trigger())); + connect(this, &MapItem::triggered, action, &MapAction::trigger); _map = map; _digitalZoom = 0; diff --git a/src/GUI/mapview.cpp b/src/GUI/mapview.cpp index 54e05ac2..b52ed52d 100644 --- a/src/GUI/mapview.cpp +++ b/src/GUI/mapview.cpp @@ -74,10 +74,10 @@ MapView::MapView(Map *map, POI *poi, QWidget *parent) _map->load(); _map->setOutputProjection(_outputProjection); _map->setInputProjection(_inputProjection); - connect(_map, SIGNAL(tilesLoaded()), this, SLOT(reloadMap())); + connect(_map, &Map::tilesLoaded, this, &MapView::reloadMap); _poi = poi; - connect(_poi, SIGNAL(pointsChanged()), this, SLOT(updatePOI())); + connect(_poi, &POI::pointsChanged, this, &MapView::updatePOI); _mapOpacity = 1.0; _backgroundColor = Qt::white; @@ -371,14 +371,14 @@ void MapView::setMap(Map *map) RectC cr(_map->xy2ll(vr.topLeft()), _map->xy2ll(vr.bottomRight())); _map->unload(); - disconnect(_map, SIGNAL(tilesLoaded()), this, SLOT(reloadMap())); + disconnect(_map, &Map::tilesLoaded, this, &MapView::reloadMap); _map = map; _map->load(); _map->setOutputProjection(_outputProjection); _map->setInputProjection(_inputProjection); _map->setDevicePixelRatio(_deviceRatio, _mapRatio); - connect(_map, SIGNAL(tilesLoaded()), this, SLOT(reloadMap())); + connect(_map, &Map::tilesLoaded, this, &MapView::reloadMap); digitalZoom(0); @@ -408,8 +408,8 @@ void MapView::setMap(Map *map) void MapView::setPOI(POI *poi) { - disconnect(_poi, SIGNAL(pointsChanged()), this, SLOT(updatePOI())); - connect(poi, SIGNAL(pointsChanged()), this, SLOT(updatePOI())); + disconnect(_poi, &POI::pointsChanged, this, &MapView::updatePOI); + connect(poi, &POI::pointsChanged, this, &MapView::updatePOI); _poi = poi; diff --git a/src/GUI/optionsdialog.cpp b/src/GUI/optionsdialog.cpp index 20fd0a8e..e59eb96c 100644 --- a/src/GUI/optionsdialog.cpp +++ b/src/GUI/optionsdialog.cpp @@ -421,8 +421,8 @@ QWidget *OptionsDialog::createDataPage() _pauseInterval->setValue(_options.pauseInterval); _pauseInterval->setEnabled(_manualPause->isChecked()); - connect(_automaticPause, SIGNAL(toggled(bool)), this, - SLOT(automaticPauseDetectionSet(bool))); + connect(_automaticPause, &QRadioButton::toggled, this, + &OptionsDialog::automaticPauseDetectionSet); QHBoxLayout *pauseTypeLayout = new QHBoxLayout(); #ifdef Q_OS_MAC @@ -477,8 +477,8 @@ QWidget *OptionsDialog::createDataPage() for (int i = 0; i < zones.size(); i++) _timeZone->addItem(zones.at(i)); _timeZone->setCurrentText(_options.timeZone.customZone().id()); - connect(_customZone, SIGNAL(toggled(bool)), _timeZone, - SLOT(setEnabled(bool))); + connect(_customZone, &QRadioButton::toggled, _timeZone, + &QComboBox::setEnabled); QHBoxLayout *customZoneLayout = new QHBoxLayout(); customZoneLayout->addSpacing(20); customZoneLayout->addWidget(_timeZone); @@ -750,15 +750,17 @@ OptionsDialog::OptionsDialog(Options &options, Units units, QWidget *parent) pages->setSizePolicy(QSizePolicy::MinimumExpanding, QSizePolicy::Preferred); pages->setMinimumWidth(2 * menu->size().width()); - connect(menu, SIGNAL(currentRowChanged(int)), pages, - SLOT(setCurrentIndex(int))); + connect(menu, &QListWidget::currentRowChanged, pages, + &QStackedWidget::setCurrentIndex); menu->item(0)->setSelected(true); QDialogButtonBox *buttonBox = new QDialogButtonBox(QDialogButtonBox::Ok | QDialogButtonBox::Cancel); - connect(buttonBox, SIGNAL(accepted()), this, SLOT(accept())); - connect(buttonBox, SIGNAL(rejected()), this, SLOT(reject())); + connect(buttonBox, &QDialogButtonBox::accepted, this, + &OptionsDialog::accept); + connect(buttonBox, &QDialogButtonBox::rejected, this, + &OptionsDialog::reject); QVBoxLayout *layout = new QVBoxLayout; layout->addLayout(contentLayout); diff --git a/src/GUI/pathitem.cpp b/src/GUI/pathitem.cpp index d3f25850..d848dda8 100644 --- a/src/GUI/pathitem.cpp +++ b/src/GUI/pathitem.cpp @@ -409,8 +409,8 @@ void PathItem::addGraph(GraphItem *graph) _graphs.append(graph); if (graph) { - connect(this, SIGNAL(selected(bool)), graph, SLOT(hover(bool))); - connect(graph, SIGNAL(selected(bool)), this, SLOT(hover(bool))); + connect(this, &PathItem::selected, graph, &GraphItem::hover); + connect(graph, &GraphItem::selected, this, &PathItem::hover); } } diff --git a/src/GUI/pdfexportdialog.cpp b/src/GUI/pdfexportdialog.cpp index 12cff34d..788fd1e9 100644 --- a/src/GUI/pdfexportdialog.cpp +++ b/src/GUI/pdfexportdialog.cpp @@ -91,8 +91,10 @@ PDFExportDialog::PDFExportDialog(PDFExport &exp, Units units, QWidget *parent) QDialogButtonBox *buttonBox = new QDialogButtonBox(); buttonBox->addButton(tr("Export"), QDialogButtonBox::AcceptRole); buttonBox->addButton(QDialogButtonBox::Cancel); - connect(buttonBox, SIGNAL(accepted()), this, SLOT(accept())); - connect(buttonBox, SIGNAL(rejected()), this, SLOT(reject())); + connect(buttonBox, &QDialogButtonBox::accepted, this, + &PDFExportDialog::accept); + connect(buttonBox, &QDialogButtonBox::rejected, this, + &PDFExportDialog::reject); QVBoxLayout *layout = new QVBoxLayout; #ifdef Q_OS_MAC diff --git a/src/GUI/percentslider.cpp b/src/GUI/percentslider.cpp index 467f716a..dd8a8cd6 100644 --- a/src/GUI/percentslider.cpp +++ b/src/GUI/percentslider.cpp @@ -24,7 +24,7 @@ PercentSlider::PercentSlider(QWidget *parent) : QWidget(parent) _label->setAlignment(Qt::AlignRight); _label->setText(format(_slider->value())); - connect(_slider, SIGNAL(valueChanged(int)), this, SLOT(updateLabel(int))); + connect(_slider, &QSlider::valueChanged, this, &PercentSlider::updateLabel); QHBoxLayout *layout = new QHBoxLayout(); layout->addWidget(_slider); diff --git a/src/GUI/pngexportdialog.cpp b/src/GUI/pngexportdialog.cpp index 81cf0f0b..5a4ebd5b 100644 --- a/src/GUI/pngexportdialog.cpp +++ b/src/GUI/pngexportdialog.cpp @@ -67,8 +67,10 @@ PNGExportDialog::PNGExportDialog(PNGExport &exp, QWidget *parent) QDialogButtonBox *buttonBox = new QDialogButtonBox(); buttonBox->addButton(tr("Export"), QDialogButtonBox::AcceptRole); buttonBox->addButton(QDialogButtonBox::Cancel); - connect(buttonBox, SIGNAL(accepted()), this, SLOT(accept())); - connect(buttonBox, SIGNAL(rejected()), this, SLOT(reject())); + connect(buttonBox, &QDialogButtonBox::accepted, this, + &PNGExportDialog::accept); + connect(buttonBox, &QDialogButtonBox::rejected, this, + &PNGExportDialog::reject); QVBoxLayout *layout = new QVBoxLayout; #ifdef Q_OS_MAC diff --git a/src/map/downloader.cpp b/src/map/downloader.cpp index 529e507f..7a1dccfa 100644 --- a/src/map/downloader.cpp +++ b/src/map/downloader.cpp @@ -126,7 +126,7 @@ bool Downloader::doDownload(const Download &dl, if (reply && reply->isRunning()) { _currentDownloads.insert(url); ReplyTimeout::setTimeout(reply, _timeout); - connect(reply, SIGNAL(finished()), this, SLOT(emitFinished())); + connect(reply, &QNetworkReply::finished, this, &Downloader::emitFinished); } else if (reply) downloadFinished(reply); else diff --git a/src/map/onlinemap.cpp b/src/map/onlinemap.cpp index 110ba533..b6dac165 100644 --- a/src/map/onlinemap.cpp +++ b/src/map/onlinemap.cpp @@ -21,7 +21,7 @@ OnlineMap::OnlineMap(const QString &fileName, const QString &name, _tileLoader->setUrl(url); _tileLoader->setAuthorization(authorization); _tileLoader->setQuadTiles(quadTiles); - connect(_tileLoader, SIGNAL(finished()), this, SIGNAL(tilesLoaded())); + connect(_tileLoader, &TileLoader::finished, this, &OnlineMap::tilesLoaded); } QRectF OnlineMap::bounds() diff --git a/src/map/tileloader.cpp b/src/map/tileloader.cpp index fb09e795..461d4c8d 100644 --- a/src/map/tileloader.cpp +++ b/src/map/tileloader.cpp @@ -61,7 +61,7 @@ TileLoader::TileLoader(const QString &dir, QObject *parent) qWarning("%s: %s", qPrintable(_dir), "Error creating tiles directory"); _downloader = new Downloader(this); - connect(_downloader, SIGNAL(finished()), this, SIGNAL(finished())); + connect(_downloader, &Downloader::finished, this, &TileLoader::finished); } void TileLoader::loadTilesAsync(QVector &list) @@ -132,7 +132,7 @@ void TileLoader::loadTilesSync(QVector &list) if (!dl.empty()) { QEventLoop wait; - QObject::connect(_downloader, SIGNAL(finished()), &wait, SLOT(quit())); + connect(_downloader, &Downloader::finished, &wait, &QEventLoop::quit); if (_downloader->get(dl, _authorization)) wait.exec(); diff --git a/src/map/wms.cpp b/src/map/wms.cpp index 1157b145..b65a3e8f 100644 --- a/src/map/wms.cpp +++ b/src/map/wms.cpp @@ -324,8 +324,8 @@ bool WMS::downloadCapabilities(const QString &url) { if (!_downloader) { _downloader = new Downloader(this); - connect(_downloader, SIGNAL(finished()), this, - SLOT(capabilitiesReady())); + connect(_downloader, &Downloader::finished, this, + &WMS::capabilitiesReady); } QList dl; diff --git a/src/map/wmsmap.cpp b/src/map/wmsmap.cpp index 02702048..4c75f469 100644 --- a/src/map/wmsmap.cpp +++ b/src/map/wmsmap.cpp @@ -76,10 +76,10 @@ WMSMap::WMSMap(const QString &fileName, const QString &name, _tileLoader = new TileLoader(tilesDir, this); _tileLoader->setAuthorization(setup.authorization()); - connect(_tileLoader, SIGNAL(finished()), this, SIGNAL(tilesLoaded())); + connect(_tileLoader, &TileLoader::finished, this, &WMSMap::tilesLoaded); _wms = new WMS(QDir(tilesDir).filePath(CAPABILITIES_FILE), setup, this); - connect(_wms, SIGNAL(downloadFinished()), this, SLOT(wmsReady())); + connect(_wms, &WMS::downloadFinished, this, &WMSMap::wmsReady); if (_wms->isReady()) init(); } diff --git a/src/map/wmts.cpp b/src/map/wmts.cpp index afd17596..ecc69c8a 100644 --- a/src/map/wmts.cpp +++ b/src/map/wmts.cpp @@ -306,8 +306,8 @@ bool WMTS::downloadCapabilities(const QString &url) { if (!_downloader) { _downloader = new Downloader(this); - connect(_downloader, SIGNAL(finished()), this, - SLOT(capabilitiesReady())); + connect(_downloader, &Downloader::finished, this, + &WMTS::capabilitiesReady); } QList dl; diff --git a/src/map/wmtsmap.cpp b/src/map/wmtsmap.cpp index 47a0eb40..a97faa90 100644 --- a/src/map/wmtsmap.cpp +++ b/src/map/wmtsmap.cpp @@ -21,10 +21,10 @@ WMTSMap::WMTSMap(const QString &fileName, const QString &name, _tileLoader = new TileLoader(tilesDir, this); _tileLoader->setAuthorization(setup.authorization()); - connect(_tileLoader, SIGNAL(finished()), this, SIGNAL(tilesLoaded())); + connect(_tileLoader, &TileLoader::finished, this, &WMTSMap::tilesLoaded); _wmts = new WMTS(QDir(tilesDir).filePath(CAPABILITIES_FILE), setup, this); - connect(_wmts, SIGNAL(downloadFinished()), this, SLOT(wmtsReady())); + connect(_wmts, &WMTS::downloadFinished, this, &WMTSMap::wmtsReady); if (_wmts->isReady()) init(); }