Added a timestamp to track clipboard changes, so when a new device connects it will sync the most recently updated clipboard to both devices.
Details
Details
- Reviewers
albertvaka - Group Reviewers
KDE Connect - Commits
- R224:779997337179: Clipboard sync on new connection
Diff Detail
Diff Detail
- Repository
- R224 KDE Connect
- Branch
- clipboardNewConnectionSync
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 14256 Build 14274: arc lint + arc unit
plugins/clipboard/clipboardlistener.h | ||
---|---|---|
40 | Typically local variables are prefixed with m_<varname>, so like m_updateTimestamp. Of course, this should apply to currentContent as well... | |
61–62 | Usually getters are just named <varname>, so like qint64 updateTimestamp() | |
plugins/clipboard/clipboardplugin.cpp | ||
61 | Space before { | |
64 | } else { (Spaces) |
plugins/clipboard/clipboardplugin.cpp | ||
---|---|---|
65 | This can be simplified with the other if branch. |
plugins/clipboard/clipboardplugin.cpp | ||
---|---|---|
40 | As discussed, use a new packet type so this doesn't affect old apps. Also on Android. |