Skip to content

Commit

Permalink
pass event callback as reference
Browse files Browse the repository at this point in the history
  • Loading branch information
rex-schilasky committed Jan 29, 2025
1 parent 59d9199 commit dfda196
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion ecal/core/include/ecal/service/server.h
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ namespace eCAL
* @param event_callback_ Callback function for server events.
**/
ECAL_API_EXPORTED_MEMBER
explicit CServiceServer(const std::string& service_name_, const ServerEventCallbackT event_callback_ = ServerEventCallbackT());
explicit CServiceServer(const std::string& service_name_, const ServerEventCallbackT& event_callback_ = ServerEventCallbackT());

/**
* @brief Destructor.
Expand Down
10 changes: 5 additions & 5 deletions ecal/core/src/service/ecal_service_server.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@

namespace eCAL
{
CServiceServer::CServiceServer(const std::string & service_name_, const ServerEventCallbackT event_callback_)
CServiceServer::CServiceServer(const std::string& service_name_, const ServerEventCallbackT& event_callback_)
: m_service_server_impl(nullptr)
{
// create server implementation
Expand All @@ -48,12 +48,12 @@ namespace eCAL
if (g_servicegate() != nullptr) g_servicegate()->Unregister(m_service_server_impl->GetServiceName(), m_service_server_impl);
}

CServiceServer::CServiceServer(CServiceServer && rhs) noexcept
CServiceServer::CServiceServer(CServiceServer&& rhs) noexcept
: m_service_server_impl(std::move(rhs.m_service_server_impl))
{
}

CServiceServer & CServiceServer::operator=(CServiceServer && rhs) noexcept
CServiceServer& CServiceServer::operator=(CServiceServer&& rhs) noexcept
{
if (this != &rhs)
{
Expand All @@ -62,13 +62,13 @@ namespace eCAL
return *this;
}

bool CServiceServer::SetMethodCallback(const SServiceMethodInformation& method_info_, const ServiceMethodCallbackT & callback_)
bool CServiceServer::SetMethodCallback(const SServiceMethodInformation& method_info_, const ServiceMethodCallbackT& callback_)
{
if (m_service_server_impl == nullptr) return false;
return m_service_server_impl->SetMethodCallback(method_info_, callback_);
}

bool CServiceServer::RemoveMethodCallback(const std::string & method_)
bool CServiceServer::RemoveMethodCallback(const std::string& method_)
{
if (m_service_server_impl == nullptr) return false;
return m_service_server_impl->RemoveMethodCallback(method_);
Expand Down

0 comments on commit dfda196

Please sign in to comment.