Skip to content

Commit 627b9cb

Browse files
committed
PR feedback
1 parent b58beec commit 627b9cb

File tree

1 file changed

+6
-12
lines changed

1 file changed

+6
-12
lines changed

src/Profilers/MonitorProfiler/Communication/CommandServer.cpp

+6-12
Original file line numberDiff line numberDiff line change
@@ -196,22 +196,16 @@ bool CommandServer::IsControlCommand(const IpcMessage& message)
196196

197197
HRESULT CommandServer::ReceiveMessage(std::shared_ptr<IpcCommClient> client, IpcMessage& message)
198198
{
199-
HRESULT hr = client->Receive(message);
200-
if (FAILED(hr))
201-
{
202-
_logger->Log(LogLevel::Error, _LS("Unexpected error when receiving data: 0x%08x"), hr);
203-
}
204-
return hr;
199+
HRESULT hr;
200+
IfFailLogRet_(_logger, client->Receive(message));
201+
return S_OK;
205202
}
206203

207204
HRESULT CommandServer::SendMessage(std::shared_ptr<IpcCommClient> client, const IpcMessage& message)
208205
{
209-
HRESULT hr = client->Send(message);
210-
if (FAILED(hr))
211-
{
212-
_logger->Log(LogLevel::Error, _LS("Unexpected error when sending data: 0x%08x"), hr);
213-
}
214-
return hr;
206+
HRESULT hr;
207+
IfFailLogRet_(_logger, client->Send(message));
208+
return S_OK;
215209
}
216210

217211
HRESULT CommandServer::Shutdown(std::shared_ptr<IpcCommClient> client)

0 commit comments

Comments
 (0)