From 42fcebd473fe1323aeae3fbbf9d0c8134ce8f067 Mon Sep 17 00:00:00 2001 From: SynaptiCAD User Date: Sun, 22 Feb 2015 13:02:18 -0500 Subject: [PATCH] Remove unnecessary locking around log sending --- src/log/gelf_appender.cpp | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/src/log/gelf_appender.cpp b/src/log/gelf_appender.cpp index 14bb17d..b10cf49 100644 --- a/src/log/gelf_appender.cpp +++ b/src/log/gelf_appender.cpp @@ -4,14 +4,12 @@ #include #include #include -#include #include #include #include #include #include -#include #include #include #include @@ -26,7 +24,6 @@ namespace fc config cfg; optional gelf_endpoint; udp_socket gelf_socket; - boost::mutex socket_mutex; impl(const config& c) : cfg(c) @@ -159,11 +156,8 @@ namespace fc memcpy(send_buffer.get(), gelf_message_as_string.c_str(), gelf_message_as_string.size()); - { - scoped_lock lock(my->socket_mutex); - my->gelf_socket.send_to(send_buffer, gelf_message_as_string.size(), - *my->gelf_endpoint); - } + my->gelf_socket.send_to(send_buffer, gelf_message_as_string.size(), + *my->gelf_endpoint); } else { @@ -196,11 +190,8 @@ namespace fc *(unsigned char*)(ptr++) = total_number_of_packets; memcpy(ptr, gelf_message_as_string.c_str() + bytes_sent, bytes_to_send); - { - scoped_lock lock(my->socket_mutex); - my->gelf_socket.send_to(send_buffer, header_length + bytes_to_send, - *my->gelf_endpoint); - } + my->gelf_socket.send_to(send_buffer, header_length + bytes_to_send, + *my->gelf_endpoint); ++number_of_packets_sent; bytes_sent += bytes_to_send; }