D17553: Use typedef threadID_t qint64 instead of hard-coding
Simon Redman
noreply at phabricator.kde.org
Thu Dec 13 06:37:00 GMT 2018
sredman created this revision.
sredman added a reviewer: KDE Connect.
Herald added a project: KDE Connect.
Herald added a subscriber: kdeconnect.
sredman requested review of this revision.
REVISION SUMMARY
Due to the recent fiasco with needing to change from qint32 to qint64 for threadID, it became painfully obvious that I ought to have avoided hard-coding from the beginning. This patch solves that problem.
Unfortunately, this patch cannot build due to qtdbuscpp2xml not understanding `typedef threadID qint64;`. We either need some way to manually add this information, or the Qt tool to be updated to understand typedef on dbus-compatible types.
TEST PLAN
N.B.: This patch cannot build at the current time
- Build app
- Verfiy SMS is able to send/receive as before
REPOSITORY
R224 KDE Connect
BRANCH
threadID-typedef
REVISION DETAIL
https://phabricator.kde.org/D17553
AFFECTED FILES
interfaces/conversationmessage.cpp
interfaces/conversationmessage.h
plugins/sms/conversationsdbusinterface.cpp
plugins/sms/conversationsdbusinterface.h
plugins/sms/smsplugin.cpp
plugins/sms/smsplugin.h
smsapp/conversationlistmodel.cpp
smsapp/conversationlistmodel.h
smsapp/conversationmodel.cpp
smsapp/conversationmodel.h
To: sredman, #kde_connect
Cc: kdeconnect, mschroeder, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, anoopv, dvalencia, rmenezes, julioc, Leptopoda, timothyc, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20181213/836d11ef/attachment.html>
More information about the KDEConnect
mailing list