chore: clean up remaing sonar warnings to use in class intializers where possible
This commit is contained in:
parent
a175c293f6
commit
ede27975d7
6 changed files with 4 additions and 9 deletions
|
|
@ -33,7 +33,6 @@
|
||||||
ServerProxy::ServerProxy(Client *client, deskflow::IStream *stream, IEventQueue *events)
|
ServerProxy::ServerProxy(Client *client, deskflow::IStream *stream, IEventQueue *events)
|
||||||
: m_client(client),
|
: m_client(client),
|
||||||
m_stream(stream),
|
m_stream(stream),
|
||||||
m_parser(&ServerProxy::parseHandshakeMessage),
|
|
||||||
m_events(events)
|
m_events(events)
|
||||||
{
|
{
|
||||||
assert(m_client != nullptr);
|
assert(m_client != nullptr);
|
||||||
|
|
|
||||||
|
|
@ -123,7 +123,7 @@ private:
|
||||||
double m_keepAliveAlarm = 0.0;
|
double m_keepAliveAlarm = 0.0;
|
||||||
EventQueueTimer *m_keepAliveAlarmTimer = nullptr;
|
EventQueueTimer *m_keepAliveAlarmTimer = nullptr;
|
||||||
|
|
||||||
MessageParser m_parser;
|
MessageParser m_parser = &ServerProxy::parseHandshakeMessage;
|
||||||
IEventQueue *m_events = nullptr;
|
IEventQueue *m_events = nullptr;
|
||||||
std::string m_serverLanguage = "";
|
std::string m_serverLanguage = "";
|
||||||
bool m_isUserNotifiedAboutLanguageSyncError = false;
|
bool m_isUserNotifiedAboutLanguageSyncError = false;
|
||||||
|
|
|
||||||
|
|
@ -1381,12 +1381,9 @@ XWindowsClipboard::Reply::Reply(
|
||||||
m_target(target),
|
m_target(target),
|
||||||
m_time(time),
|
m_time(time),
|
||||||
m_property(property),
|
m_property(property),
|
||||||
m_replied(false),
|
|
||||||
m_done(false),
|
|
||||||
m_data(data),
|
m_data(data),
|
||||||
m_type(type),
|
m_type(type),
|
||||||
m_format(format),
|
m_format(format)
|
||||||
m_ptr(0)
|
|
||||||
{
|
{
|
||||||
// do nothing
|
// do nothing
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -22,7 +22,6 @@
|
||||||
|
|
||||||
ClientProxy1_0::ClientProxy1_0(const std::string &name, deskflow::IStream *stream, IEventQueue *events)
|
ClientProxy1_0::ClientProxy1_0(const std::string &name, deskflow::IStream *stream, IEventQueue *events)
|
||||||
: ClientProxy(name, stream),
|
: ClientProxy(name, stream),
|
||||||
m_parser(&ClientProxy1_0::parseHandshakeMessage),
|
|
||||||
m_events(events)
|
m_events(events)
|
||||||
{
|
{
|
||||||
// install event handlers
|
// install event handlers
|
||||||
|
|
|
||||||
|
|
@ -98,6 +98,6 @@ private:
|
||||||
ClientInfo m_info;
|
ClientInfo m_info;
|
||||||
double m_heartbeatAlarm;
|
double m_heartbeatAlarm;
|
||||||
EventQueueTimer *m_heartbeatTimer = nullptr;
|
EventQueueTimer *m_heartbeatTimer = nullptr;
|
||||||
MessageParser m_parser;
|
MessageParser m_parser = &ClientProxy1_0::parseHandshakeMessage;
|
||||||
IEventQueue *m_events;
|
IEventQueue *m_events;
|
||||||
};
|
};
|
||||||
|
|
|
||||||
|
|
@ -721,7 +721,7 @@ const InputFilter::Action &InputFilter::Rule::getAction(bool onActivation, uint3
|
||||||
// -----------------------------------------------------------------------------
|
// -----------------------------------------------------------------------------
|
||||||
// Input Filter Class
|
// Input Filter Class
|
||||||
// -----------------------------------------------------------------------------
|
// -----------------------------------------------------------------------------
|
||||||
InputFilter::InputFilter(IEventQueue *events) : m_primaryClient(nullptr), m_events(events)
|
InputFilter::InputFilter(IEventQueue *events) : m_events(events)
|
||||||
{
|
{
|
||||||
// do nothing
|
// do nothing
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue