D13629: Fix crashing devicetest

Aleix Pol Gonzalez noreply at phabricator.kde.org
Thu Jul 12 15:10:41 BST 2018


apol added inline comments.

INLINE COMMENTS

> mousepadplugin.cpp:85
>      NetworkPacket np(PACKET_TYPE_MOUSEPAD_KEYBOARDSTATE);
> -    np.set<bool>(QStringLiteral("state"), m_impl->hasKeyboardSupport());
> +    if (m_impl) {
> +        np.set<bool>(QStringLiteral("state"), m_impl->hasKeyboardSupport());

in which cases m_impl is null?

REPOSITORY
  R224 KDE Connect

BRANCH
  fixtest

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

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


More information about the KDEConnect mailing list