[vlc-devel] [PATCH] qt: medialib: fix signal name
rom at rom1v.com
rom at rom1v.com
Thu Dec 3 17:33:40 CET 2020
From: Romain Vimont <rom1v at videolabs.io>
By convention, xxxChanged() is for signals, while onXxxChanged() is for
slots.
---
modules/gui/qt/medialibrary/mlvideo.cpp | 2 +-
modules/gui/qt/medialibrary/mlvideo.hpp | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/modules/gui/qt/medialibrary/mlvideo.cpp b/modules/gui/qt/medialibrary/mlvideo.cpp
index cefb9504d6..8adedbb4d8 100644
--- a/modules/gui/qt/medialibrary/mlvideo.cpp
+++ b/modules/gui/qt/medialibrary/mlvideo.cpp
@@ -177,7 +177,7 @@ void MLVideo::onMlEvent( const vlc_ml_event_t* event )
.p_media->thumbnails[event->media_thumbnail_generated.i_size].psz_mrl;
m_thumbnail = QString::fromUtf8( thumbnailMrl );
vlc_ml_event_unregister_from_callback( m_ml, m_ml_event_handle.release() );
- emit onThumbnailChanged( m_thumbnail );
+ emit thumbnailChanged( m_thumbnail );
}
QString MLVideo::getTitle() const
diff --git a/modules/gui/qt/medialibrary/mlvideo.hpp b/modules/gui/qt/medialibrary/mlvideo.hpp
index 8041881280..37cdbbab71 100644
--- a/modules/gui/qt/medialibrary/mlvideo.hpp
+++ b/modules/gui/qt/medialibrary/mlvideo.hpp
@@ -85,7 +85,7 @@ class MLVideo : public QObject, public MLItem
Q_PROPERTY(MLItemId id READ getId CONSTANT);
Q_PROPERTY(QString title READ getTitle CONSTANT);
- Q_PROPERTY(QString thumbnail READ getThumbnail NOTIFY onThumbnailChanged);
+ Q_PROPERTY(QString thumbnail READ getThumbnail NOTIFY thumbnailChanged);
Q_PROPERTY(QString duration READ getDuration CONSTANT);
Q_PROPERTY(QString durationShort READ getDurationShort CONSTANT);
Q_PROPERTY(QString mrl READ getMRL CONSTANT);
@@ -118,7 +118,7 @@ public:
MLVideo* clone(QObject* parent = nullptr) const;
signals:
- void onThumbnailChanged( QString );
+ void thumbnailChanged( QString );
private:
MLVideo(const MLVideo& video, QObject* parent = nullptr);
--
2.29.2
More information about the vlc-devel
mailing list