D15409: [WIP] Allow SMS app to handle newly recieved messages

Nicolas Fella noreply at phabricator.kde.org
Thu Sep 13 00:59:00 BST 2018


nicolasfella added inline comments.

INLINE COMMENTS

> conversationmodel.cpp:70
>      connect(m_conversationsInterface, SIGNAL(conversationMessageReceived(QVariantMap, int)), this, SLOT(createRowFromMessage(QVariantMap, int)));
> +    connect(m_conversationsInterface, SIGNAL(conversationUpdated(QVariantMap)), this, SLOT(handleConversationUpdate(QVariantMap)));
>  }

Please use the new connect syntax.

  connect(sender, &Sender::signal, receiver, &Receiver::slot)

REPOSITORY
  R224 KDE Connect

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

To: sredman, #kde_connect
Cc: nicolasfella, kdeconnect, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, ndavis, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, tctara, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20180912/d06a3c2a/attachment.html>


More information about the KDEConnect mailing list