D21445: Switch from KMime::Headers::Date -> QDateTime for message header.

Daniel Vrátil noreply at phabricator.kde.org
Tue May 28 12:14:06 BST 2019


dvratil added inline comments.

INLINE COMMENTS

> grantleeheaderformatter.cpp:124
>  template<>
> -inline QVariant TypeAccessor<const KMime::Headers::Date *>::lookUp(const KMime::Headers::Date *const object, const QString &property)
> +inline QVariant TypeAccessor<QDateTime &>::lookUp(QDateTime const &object, const QString &property)
>  {

Why is the specialization for a non-const ref? The argument is a const reference, shouldn't it be const, too?

REPOSITORY
  R94 PIM: Message Library

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

To: knauss, #kde_pim
Cc: dvratil, kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20190528/d2c425d7/attachment.html>


More information about the kde-pim mailing list