From kde-commits Sat Sep 30 21:33:50 2017 From: Michael Pyne Date: Sat, 30 Sep 2017 21:33:50 +0000 To: kde-commits Subject: [juk/frameworks] /: Fix miscellaneous remaining warnings. Message-Id: X-MARC-Message: https://marc.info/?l=kde-commits&m=150680817827411 Git commit 4e13f805b134a0b78bcaa7e6ee3aded34b500d59 by Michael Pyne. Committed on 30/09/2017 at 21:25. Pushed by mpyne into branch 'frameworks'. Fix miscellaneous remaining warnings. Most are just worked around in some fashion but better to weed out warnings from features that don't work right now so that actual issues that might be introduced are that much more apparent. M +1 -1 juk.cpp M +9 -64 playlist.cpp M +3 -4 tageditor.cpp M +3 -0 viewmode.cpp M +1 -1 viewmode.h https://commits.kde.org/juk/4e13f805b134a0b78bcaa7e6ee3aded34b500d59 diff --git a/juk.cpp b/juk.cpp index 295e0bc..53ac0b1 100644 --- a/juk.cpp +++ b/juk.cpp @@ -427,7 +427,7 @@ void JuK::readConfig() ActionCollection::action("albumRandomPlay")->setChecked(t= rue); = bool loopPlaylist =3D playerConfig.readEntry("LoopPlaylist", false); - //ActionCollection::action("loopPlaylist")->setChecked(loopPl= aylist); + ActionCollection::action("loopPlaylist")->setChecked(loopPlay= list); = // general settings = diff --git a/playlist.cpp b/playlist.cpp index 1d9be1a..dceeed4 100644 --- a/playlist.cpp +++ b/playlist.cpp @@ -99,55 +99,6 @@ static bool manualResize() * A tooltip specialized to show full filenames over the file name column. */ = -#ifdef __GNUC__ -#warning disabling the tooltip for now -#endif -#if 0 -class PlaylistToolTip : public QToolTip -{ -public: - PlaylistToolTip(QWidget *parent, Playlist *playlist) : - QToolTip(parent), m_playlist(playlist) {} - - virtual void maybeTip(const QPoint &p) - { - PlaylistItem *item =3D static_cast(m_playlist->ite= mAt(p)); - - if(!item) - return; - - QPoint contentsPosition =3D m_playlist->viewportToContents(p); - - int column =3D m_playlist->header()->sectionAt(contentsPosition.x(= )); - - if(column =3D=3D m_playlist->columnOffset() + PlaylistItem::FileNa= meColumn || - item->cachedWidths()[column] > m_playlist->columnWidth(column) = || - (column =3D=3D m_playlist->columnOffset() + PlaylistItem::Cover= Column && - item->file().coverInfo()->hasCover())) - { - QRect r =3D m_playlist->itemRect(item); - int headerPosition =3D m_playlist->header()->sectionPos(column= ); - r.setLeft(headerPosition); - r.setRight(headerPosition + m_playlist->header()->sectionSize(= column)); - - if(column =3D=3D m_playlist->columnOffset() + PlaylistItem::Fi= leNameColumn) - tip(r, item->file().absFilePath()); - else if(column =3D=3D m_playlist->columnOffset() + PlaylistIte= m::CoverColumn) { - Q3MimeSourceFactory *f =3D Q3MimeSourceFactory::defaultFac= tory(); - f->setImage("coverThumb", - QImage(item->file().coverInfo()->pixmap(CoverI= nfo::Thumbnail).convertToImage())); - tip(r, "
"); - } - else - tip(r, item->text(column)); - } - } - -private: - Playlist *m_playlist; -}; -#endif - //////////////////////////////////////////////////////////////////////////= ////// // Playlist::SharedSettings definition //////////////////////////////////////////////////////////////////////////= ////// @@ -1100,8 +1051,13 @@ QMimeData* Playlist::mimeData(const QList items) const = bool Playlist::dropMimeData(QTreeWidgetItem *parent, int index, const QMim= eData *data, Qt::DropAction action) { - qCDebug(JUK_LOG) << index; - return true; + // TODO: Re-add DND + Q_UNUSED(parent); + Q_UNUSED(index); + Q_UNUSED(data); + Q_UNUSED(action); + + return false; } = void Playlist::dropEvent(QDropEvent *e) @@ -1472,11 +1428,6 @@ void Playlist::slotInitialize() // setup header RMB menu ////////////////////////////////////////////////// = -#ifdef __GNUC__ - #warning should be fixed... -#endif - /* m_headerMenu->insertTitle(i18n("Show")); */ - QAction *showAction; = for(int i =3D 0; i < header()->count(); ++i) { @@ -1525,10 +1476,6 @@ void Playlist::slotInitialize() setDragEnabled(true); = m_disableColumnWidthUpdates =3D false; - - // FIXME tooltips - //setShowToolTips(false); - /* m_toolTip =3D new PlaylistToolTip(viewport(), this); */ } = void Playlist::setupItem(PlaylistItem *item) @@ -1704,9 +1651,7 @@ void Playlist::setPlaying(PlaylistItem *item, bool ad= dToHistory) } = TrackSequenceManager::instance()->setCurrent(item); -#ifdef __GNUC__ -#warning "kde4^H^H^H^Hkde5: port it" -#endif + // TODO is this replaced by MPRIS2? //kapp->dcopClient()->emitDCOPSignal("Player", "trackChanged()", data); = if(!item) @@ -2193,7 +2138,7 @@ bool Playlist::editTag(PlaylistItem *item, const QStr= ing &text, int column) return true; } = -void Playlist::slotInlineEditDone(QTreeWidgetItem *, const QString &, int = column) +void Playlist::slotInlineEditDone(QTreeWidgetItem *, const QString &, int) { // FIXME rename /*QString text =3D renameLineEdit()->text(); diff --git a/tageditor.cpp b/tageditor.cpp index 6962a70..f8c4421 100644 --- a/tageditor.cpp +++ b/tageditor.cpp @@ -461,11 +461,10 @@ void TagEditor::readConfig() = void TagEditor::readCompletionMode(const KConfigGroup &config, KComboBox *= box, const QString &key) { - KGlobalSettings::Completion mode =3D - KGlobalSettings::Completion(config.readEntry(key, (int)KGlobalSett= ings::CompletionAuto)); + KCompletion::CompletionMode mode =3D + KCompletion::CompletionMode(config.readEntry(key, (int)KCompletion= ::CompletionAuto)); = - // FIXME tag completion - //box->setCompletionMode(mode); + box->setCompletionMode(mode); } = void TagEditor::saveConfig() diff --git a/viewmode.cpp b/viewmode.cpp index d397679..2b5e134 100644 --- a/viewmode.cpp +++ b/viewmode.cpp @@ -179,6 +179,7 @@ void ViewMode::updateIcons(int size) = void ViewMode::setupItem(PlaylistBox::Item *item) const { + Q_UNUSED(item); // FIXME /*const PlaylistBox *box =3D item->listView(); const int width =3D box->width() - box->verticalScrollBar()->width() -= border * 2; @@ -236,6 +237,8 @@ QStringList ViewMode::lines(const PlaylistBox::Item *it= em, = QStringList l; = + Q_UNUSED(fm); + while(!line.isEmpty()) { int textLength =3D line.length(); // FIXME diff --git a/viewmode.h b/viewmode.h index 4cf9990..7b9d0c9 100644 --- a/viewmode.h +++ b/viewmode.h @@ -111,7 +111,7 @@ public: const QColorGroup &colorGroup, int column, int width, int align);*/ = - virtual void setupItem(PlaylistBox::Item *item) const { /*item->QTreeW= idgetItem::setup();*/ } + virtual void setupItem(PlaylistBox::Item *) const { /*item->QTreeWidge= tItem::setup();*/ } protected: virtual void updateHeights(); };