<table><tr><td style="">jeanv created this revision.<br />jeanv added a project: KDE Connect.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D7312" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Change all member variables to the form m_fooBar because it is the preferred form in Qt (it was half and half between this and mFooBar, and a minority didn't have anything).<br />
Place all references and pointers on the side of the type since it is the majority.</p>

<p>Basically:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">mFoo -> m_foo</li>
<li class="remarkup-list-item">foo -> m_foo (if it is a member variable)</li>
<li class="remarkup-list-item">Type &ref -> Type& ref</li>
<li class="remarkup-list-item">Type *ptr -> Type* ptr</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7312" rel="noreferrer">https://phabricator.kde.org/D7312</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>app/main.cpp<br />
cli/kdeconnect-cli.cpp<br />
core/backends/bluetooth/bluetoothdownloadjob.cpp<br />
core/backends/bluetooth/bluetoothdownloadjob.h<br />
core/backends/bluetooth/bluetoothuploadjob.cpp<br />
core/backends/bluetooth/bluetoothuploadjob.h<br />
core/backends/devicelinereader.cpp<br />
core/backends/devicelinereader.h<br />
core/backends/devicelink.cpp<br />
core/backends/devicelink.h<br />
core/backends/lan/downloadjob.cpp<br />
core/backends/lan/downloadjob.h<br />
core/backends/lan/landevicelink.cpp<br />
core/backends/lan/landevicelink.h<br />
core/backends/lan/lanlinkprovider.cpp<br />
core/backends/lan/lanlinkprovider.h<br />
core/backends/lan/server.cpp<br />
core/backends/lan/socketlinereader.cpp<br />
core/backends/lan/socketlinereader.h<br />
core/backends/lan/uploadjob.cpp<br />
core/backends/lan/uploadjob.h<br />
core/backends/pairinghandler.cpp<br />
core/core_debug.cpp<br />
core/daemon.cpp<br />
core/daemon.h<br />
core/device.cpp<br />
core/device.h<br />
core/filetransferjob.cpp<br />
core/filetransferjob.h<br />
core/kdeconnectconfig.cpp<br />
core/kdeconnectconfig.h<br />
core/kdeconnectplugin.cpp<br />
core/kdeconnectplugin.h<br />
core/kdeconnectpluginconfig.cpp<br />
core/kdeconnectpluginconfig.h<br />
core/networkpackage.cpp<br />
core/networkpackage.h<br />
core/pluginloader.cpp<br />
core/pluginloader.h<br />
fileitemactionplugin/sendfileitemaction.cpp<br />
fileitemactionplugin/sendfileitemaction.h<br />
indicator/deviceindicator.h<br />
indicator/main.cpp<br />
interfaces/dbusinterfaces.cpp<br />
interfaces/dbusinterfaces.h<br />
interfaces/devicesmodel.cpp<br />
interfaces/devicesmodel.h<br />
interfaces/devicessortproxymodel.cpp<br />
interfaces/devicessortproxymodel.h<br />
interfaces/modeltest.cpp<br />
interfaces/modeltest.h<br />
interfaces/notificationsmodel.cpp<br />
kcm/kcm.cpp<br />
kcm/kcm.h<br />
kcmplugin/kdeconnectpluginkcm.cpp<br />
kcmplugin/kdeconnectpluginkcm.h<br />
kio/kiokdeconnect.cpp<br />
kio/kiokdeconnect.h<br />
plasmoid/declarativeplugin/kdeconnectdeclarativeplugin.cpp<br />
plasmoid/declarativeplugin/kdeconnectdeclarativeplugin.h<br />
plasmoid/declarativeplugin/objectfactory.h<br />
plasmoid/declarativeplugin/processrunner.cpp<br />
plasmoid/declarativeplugin/processrunner.h<br />
plasmoid/declarativeplugin/responsewaiter.h<br />
plugins/battery/batterydbusinterface.cpp<br />
plugins/battery/batterydbusinterface.h<br />
plugins/battery/batteryplugin.cpp<br />
plugins/battery/batteryplugin.h<br />
plugins/clipboard/clipboardplugin.cpp<br />
plugins/clipboard/clipboardplugin.h<br />
plugins/findmyphone/findmyphoneplugin.h<br />
plugins/lockdevice/lockdeviceplugin.h<br />
plugins/mousepad/mousepadplugin.cpp<br />
plugins/mousepad/mousepadplugin.h<br />
plugins/mousepad_windows/mousepadplugin_windows.h<br />
plugins/mpriscontrol/mpriscontrolplugin.h<br />
plugins/mprisremote/mprisremoteplugin.h<br />
plugins/notifications/notification.cpp<br />
plugins/notifications/notification.h<br />
plugins/notifications/notificationsdbusinterface.cpp<br />
plugins/notifications/notificationsdbusinterface.h<br />
plugins/notifications/notificationsplugin.h<br />
plugins/notifications/sendreplydialog.cpp<br />
plugins/notifications/sendreplydialog.h<br />
plugins/pausemusic/pausemusic_config.cpp<br />
plugins/pausemusic/pausemusic_config.h<br />
plugins/pausemusic/pausemusicplugin.h<br />
plugins/ping/pingplugin.h<br />
plugins/remotecommands/remotecommandsplugin.cpp<br />
plugins/remotecommands/remotecommandsplugin.h<br />
plugins/remotecontrol/remotecontrolplugin.cpp<br />
plugins/remotecontrol/remotecontrolplugin.h<br />
plugins/remotekeyboard/remotekeyboardplugin.h<br />
plugins/runcommand/runcommand_config.cpp<br />
plugins/runcommand/runcommand_config.h<br />
plugins/runcommand/runcommandplugin.h<br />
plugins/screensaver-inhibit/screensaverinhibitplugin.h<br />
plugins/sendnotifications/notificationslistener.cpp<br />
plugins/sendnotifications/notificationslistener.h<br />
plugins/sendnotifications/notifyingapplication.cpp<br />
plugins/sendnotifications/notifyingapplication.h<br />
plugins/sendnotifications/notifyingapplicationmodel.cpp<br />
plugins/sendnotifications/notifyingapplicationmodel.h<br />
plugins/sendnotifications/sendnotifications_config.cpp<br />
plugins/sendnotifications/sendnotifications_config.h<br />
plugins/sendnotifications/sendnotificationsplugin.h<br />
plugins/sftp/mounter.h<br />
plugins/sftp/sftpplugin.cpp<br />
plugins/sftp/sftpplugin.h<br />
plugins/share/share_config.cpp<br />
plugins/share/share_config.h<br />
plugins/share/shareplugin.h<br />
plugins/telephony/sendsmsdialog.cpp<br />
plugins/telephony/sendsmsdialog.h<br />
plugins/telephony/telephonyplugin.cpp<br />
plugins/telephony/telephonyplugin.h<br />
tests/downloadjobtest.cpp<br />
tests/lanlinkprovidertest.cpp<br />
tests/networkpackagetests.cpp<br />
tests/pluginloadtest.cpp<br />
tests/sendfiletest.cpp<br />
tests/testnotificationlistener.cpp<br />
tests/testsocketlinereader.cpp<br />
tests/testsslsocketlinereader.cpp</div></div></div><br /><div><strong>To: </strong>jeanv, KDE Connect<br /><strong>Cc: </strong>KDE Connect, jeanv, tfella, aboudhar, seebauer, progwolff, MayeulC, menasshock, ach, apol, hkaelberer<br /></div>