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

[Simon Redman]

sredman added inline comments.

INLINE COMMENTS

> nicolasfella wrote in conversationmodel.cpp:70
> Please use the new connect syntax.
> 
>   connect(sender, &Sender::signal, receiver, &Receiver::slot)

It looks like, because of the way our build system builds the dbus interfaces, the member function definitions are not available to the linker to build the plugin (or, in this case, model). Changing to the new syntax gives lots of undefined reference errors, which is why we rely on runtime connections. Does anyone have a way around this?

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/20180915/4a849470/attachment.html>