smw/deskflow
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor: Use Mutex to protect m_open and m_time in Clipboard.cpp

Browse source
This commit is contained in:
sithlord48 2026-06-18 20:18:59 -04:00 committed by Chris Rizzitello
parent 5eb1d7068e
commit 81e6e13d94
2 changed files with 9 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/lib/deskflow/Clipboard.cpp
View file

@ -21,6 +21,7 @@ Clipboard::Clipboard()
bool Clipboard::empty()
{
std::scoped_lock lock{m_mutex};
if (!m_open) {
LOG_WARN("cannot empty clipboard, not open");
return false;
@ -43,6 +44,7 @@ bool Clipboard::empty()
void Clipboard::add(Format format, const std::string &data)
{
std::scoped_lock lock{m_mutex};
if (!m_open) {
LOG_WARN("cannot add to clipboard, not open");
return;
@ -60,6 +62,7 @@ void Clipboard::add(Format format, const std::string &data)
bool Clipboard::open(Time time) const
{
std::scoped_lock lock{m_mutex};
if (m_open) {
LOG_DEBUG("skipping clipboard open, already open");
return true;
@ -73,6 +76,7 @@ bool Clipboard::open(Time time) const
void Clipboard::close() const
{
std::scoped_lock lock{m_mutex};
if (!m_open) {
LOG_WARN("clipboard is not open");
}
@ -86,6 +90,7 @@ Clipboard::Time Clipboard::getTime() const
bool Clipboard::has(Format format) const
{
std::scoped_lock lock{m_mutex};
if (!m_open) {
LOG_WARN("cannot check for clipboard format, not open");
return false;
@ -95,6 +100,7 @@ bool Clipboard::has(Format format) const
std::string Clipboard::get(Format format) const
{
std::scoped_lock lock{m_mutex};
if (!m_open) {
LOG_WARN("cannot get clipboard format, not open");
return "";

3
src/lib/deskflow/Clipboard.h
View file

@ -9,6 +9,8 @@
#include "deskflow/IClipboard.h"
#include <mutex>
//! Memory buffer clipboard
/*!
This class implements a clipboard that stores data in memory.
@ -53,6 +55,7 @@ public:
private:
mutable bool m_open = false;
mutable std::mutex m_mutex;
mutable Time m_time;
bool m_owner = false;
Time m_timeOwned;