From acffacd67db1a73a1a07d12edeed1439a7669351 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?St=C3=A9phane=20L?= Date: Tue, 4 Jul 2017 21:20:34 +0200 Subject: [PATCH] Request/Response: refactor logging code --- WSRequestHandler.cpp | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/WSRequestHandler.cpp b/WSRequestHandler.cpp index cfe5bee2..59916321 100644 --- a/WSRequestHandler.cpp +++ b/WSRequestHandler.cpp @@ -116,9 +116,7 @@ void WSRequestHandler::processIncomingMessage(QString textMessage) } if (Config::Current()->DebugEnabled) - { blog(LOG_DEBUG, "Request >> '%s'", msg); - } if (!hasField("request-type") || !hasField("message-id")) @@ -177,14 +175,11 @@ void WSRequestHandler::SendErrorResponse(const char* errorMessage) void WSRequestHandler::SendResponse(obs_data_t* response) { const char *json = obs_data_get_json(response); - obs_data_release(response); - - if (Config::Current()->DebugEnabled) - { - blog(LOG_DEBUG, "Response << '%s'", json); - } - _client->sendTextMessage(json); + if (Config::Current()->DebugEnabled) + blog(LOG_DEBUG, "Response << '%s'", json); + + obs_data_release(response); } bool WSRequestHandler::hasField(const char* name) @@ -1541,4 +1536,4 @@ void WSRequestHandler::HandleSetBrowserSourceProperties(WSRequestHandler* req) req->SendErrorResponse("specified scene item doesn't exist"); } obs_source_release(scene); -} \ No newline at end of file +}