D8639: Optimize SortFilterModel role names

Milian Wolff noreply at phabricator.kde.org
Fri Nov 3 11:19:42 UTC 2017


mwolff accepted this revision.
mwolff added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> datamodel.h:141
>  protected:
> -    int roleNameToId(const QString &name);  //FIXME TODO KF6: This should have been const.
> +    int roleNameToId(const QString &name) const;
>      bool filterAcceptsRow(int source_row, const QModelIndex &source_parent) const Q_DECL_OVERRIDE;

why was that kf6 note there?

REPOSITORY
  R242 Plasma Framework (Library)

REVISION DETAIL
  https://phabricator.kde.org/D8639

To: broulik, #plasma, mwolff
Cc: mwolff, plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171103/96db5f22/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list