From 6bc854f191a337b069ff750c8a45e5a78fe0cfb5 Mon Sep 17 00:00:00 2001 From: Mathias Kraus Date: Wed, 8 Jan 2025 01:38:23 +0100 Subject: [PATCH] iox-#1755 Run clang-format-18 --- .../posix/sync/source/signal_watcher.cpp | 2 +- .../custom/default/error_reporting_impl.hpp | 48 +++++++++---------- ...tional_interface_concat_multiple_calls.cpp | 12 ++--- .../moduletests/test_functional_function.cpp | 4 +- .../iceoryx_posh/internal/popo/trigger.inl | 18 +++---- .../test_interface_port_stack_blowup.cpp | 4 +- ...est_publisher_subscriber_communication.cpp | 8 ++-- iceoryx_posh/test/mocks/gateway_base_mock.hpp | 2 +- iceoryx_posh/test/mocks/publisher_mock.hpp | 8 ++-- iceoryx_posh/test/mocks/subscriber_mock.hpp | 4 +- .../test/mocks/trigger_handle_mock.hpp | 5 +- .../test/moduletests/test_capro_service.cpp | 4 +- .../test/moduletests/test_gw_channel.cpp | 8 ++-- .../moduletests/test_gw_gateway_discovery.cpp | 4 +- .../moduletests/test_gw_gateway_generic.cpp | 8 ++-- .../test/moduletests/test_mepoo_config.cpp | 7 +-- .../moduletests/test_mepoo_typed_mempool.cpp | 4 +- .../moduletests/test_popo_base_publisher.cpp | 2 +- .../moduletests/test_popo_chunk_queue.cpp | 12 ++--- .../moduletests/test_popo_client_port.cpp | 2 +- .../test_popo_server_port_common.hpp | 2 +- .../test_popo_smart_chunk_common.hpp | 6 +-- .../moduletests/test_popo_used_chunk_list.cpp | 2 +- .../test/moduletests/test_popo_waitset.cpp | 6 +-- .../test_posh_runtime_single_process.cpp | 4 +- ...udi_cmd_line_parser_config_file_option.cpp | 2 +- .../test_roudi_mempool_introspection.cpp | 4 +- .../test/moduletests/test_roudi_process.cpp | 2 +- .../test/moduletests/test_version_info.cpp | 8 ++-- .../test/stubs/stub_gateway_generic.hpp | 2 +- 30 files changed, 96 insertions(+), 108 deletions(-) diff --git a/iceoryx_hoofs/posix/sync/source/signal_watcher.cpp b/iceoryx_hoofs/posix/sync/source/signal_watcher.cpp index 4a69c2e519..64a72c04b5 100644 --- a/iceoryx_hoofs/posix/sync/source/signal_watcher.cpp +++ b/iceoryx_hoofs/posix/sync/source/signal_watcher.cpp @@ -43,7 +43,7 @@ void internalSignalHandler(int) noexcept SignalWatcher::SignalWatcher() noexcept : m_sigTermGuard( - registerSignalHandler(PosixSignal::TERM, internalSignalHandler).expect("Unable to register Signal::TERM")) + registerSignalHandler(PosixSignal::TERM, internalSignalHandler).expect("Unable to register Signal::TERM")) , m_sigIntGuard( registerSignalHandler(PosixSignal::INT, internalSignalHandler).expect("Unable to register Signal::INT")) { diff --git a/iceoryx_hoofs/reporting/include/iox/error_reporting/custom/default/error_reporting_impl.hpp b/iceoryx_hoofs/reporting/include/iox/error_reporting/custom/default/error_reporting_impl.hpp index d250d70be0..8ac19afc87 100644 --- a/iceoryx_hoofs/reporting/include/iox/error_reporting/custom/default/error_reporting_impl.hpp +++ b/iceoryx_hoofs/reporting/include/iox/error_reporting/custom/default/error_reporting_impl.hpp @@ -90,13 +90,12 @@ inline void report(const SourceLocation& location, Kind, const Error& error, con auto moduleName = toModuleName(error); auto errorName = toErrorName(error); - IOX_ERROR_INTERNAL_LOG( - location, - [stringifiedCondition](auto& stream) -> auto& { - return detail::logStringifiedCondition(stream, stringifiedCondition); - } << "[" - << errorName << " (code = " << code.value << ")] in module [" << moduleName << " (id = " << module.value - << ")]"); + IOX_ERROR_INTERNAL_LOG(location, + [stringifiedCondition](auto& stream) -> auto& { + return detail::logStringifiedCondition(stream, stringifiedCondition); + } << "[" + << errorName << " (code = " << code.value << ")] in module [" << moduleName + << " (id = " << module.value << ")]"); auto& h = ErrorHandler::get(); h.onReportError(ErrorDescriptor(location, code, module)); } @@ -115,13 +114,12 @@ report(const SourceLocation& location, iox::er::FatalKind kind, const Error& err auto moduleName = toModuleName(error); auto errorName = toErrorName(error); - IOX_ERROR_INTERNAL_LOG_FATAL( - location, - [stringifiedCondition](auto& stream) -> auto& { - return detail::logStringifiedCondition(stream, stringifiedCondition); - } << "[" - << kind.name << "] [" << errorName << " (code = " << code.value << ")] in module [" << moduleName - << " (id = " << module.value << ")]"); + IOX_ERROR_INTERNAL_LOG_FATAL(location, + [stringifiedCondition](auto& stream) -> auto& { + return detail::logStringifiedCondition(stream, stringifiedCondition); + } << "[" + << kind.name << "] [" << errorName << " (code = " << code.value << ")] in module [" + << moduleName << " (id = " << module.value << ")]"); auto& h = ErrorHandler::get(); h.onReportError(ErrorDescriptor(location, code, module)); } @@ -141,21 +139,19 @@ report(const SourceLocation& location, Kind kind, const Error& error, const char auto module = toModule(error); if constexpr (std::is_same::value) { - IOX_ERROR_INTERNAL_LOG_FATAL( - location, - [stringifiedCondition](auto& stream) -> auto& { - return detail::logStringifiedCondition(stream, stringifiedCondition); - } << "[" - << kind.name << "]"); + IOX_ERROR_INTERNAL_LOG_FATAL(location, + [stringifiedCondition](auto& stream) -> auto& { + return detail::logStringifiedCondition(stream, stringifiedCondition); + } << "[" + << kind.name << "]"); } else { - IOX_ERROR_INTERNAL_LOG_FATAL( - location, - [stringifiedCondition](auto& stream) -> auto& { - return detail::logStringifiedCondition(stream, stringifiedCondition); - } << "[" - << kind.name << "] " << std::forward(msg)); + IOX_ERROR_INTERNAL_LOG_FATAL(location, + [stringifiedCondition](auto& stream) -> auto& { + return detail::logStringifiedCondition(stream, stringifiedCondition); + } << "[" + << kind.name << "] " << std::forward(msg)); } auto& h = ErrorHandler::get(); h.onReportViolation(ErrorDescriptor(location, code, module)); diff --git a/iceoryx_hoofs/test/moduletests/test_design_functional_interface_concat_multiple_calls.cpp b/iceoryx_hoofs/test/moduletests/test_design_functional_interface_concat_multiple_calls.cpp index 99ca2e94ef..35375fcde0 100644 --- a/iceoryx_hoofs/test/moduletests/test_design_functional_interface_concat_multiple_calls.cpp +++ b/iceoryx_hoofs/test/moduletests/test_design_functional_interface_concat_multiple_calls.cpp @@ -51,8 +51,7 @@ struct AndThenOrElseConcatenatedWorksWhenInvalid { Arg() = default; explicit Arg(int32_t value) - : value(value){}; + : value(value) {}; Arg(const Arg&) = default; Arg& operator=(const Arg&) = default; @@ -557,7 +557,7 @@ TEST_F(function_test, CallWithRValueReferenceArgumentsWorks) // NOLINTNEXTLINE(cppcoreguidelines-rvalue-reference-param-not-moved) this is okay for this test auto lambda = [](Arg&& a) { return a.value + 1; }; - function sut(lambda); + function sut(lambda); auto result = sut(std::move(arg)); diff --git a/iceoryx_posh/include/iceoryx_posh/internal/popo/trigger.inl b/iceoryx_posh/include/iceoryx_posh/internal/popo/trigger.inl index 6751a59b4b..24fc4feda5 100644 --- a/iceoryx_posh/include/iceoryx_posh/internal/popo/trigger.inl +++ b/iceoryx_posh/include/iceoryx_posh/internal/popo/trigger.inl @@ -75,15 +75,15 @@ inline Trigger::Trigger(EventBasedTrigger_t, const uint64_t notificationType, const uint64_t notificationTypeHash) noexcept : Trigger( - notificationOrigin, - []() { return false; }, - resetCallback, - notificationId, - callback, - uniqueId, - TriggerType::EVENT_BASED, - notificationType, - notificationTypeHash) + notificationOrigin, + []() { return false; }, + resetCallback, + notificationId, + callback, + uniqueId, + TriggerType::EVENT_BASED, + notificationType, + notificationTypeHash) { } } // namespace popo diff --git a/iceoryx_posh/test/integrationtests/test_interface_port_stack_blowup.cpp b/iceoryx_posh/test/integrationtests/test_interface_port_stack_blowup.cpp index 1c7aea6b40..1831afaf3a 100644 --- a/iceoryx_posh/test/integrationtests/test_interface_port_stack_blowup.cpp +++ b/iceoryx_posh/test/integrationtests/test_interface_port_stack_blowup.cpp @@ -39,8 +39,8 @@ class InterfacePortRequestStackBlowup_test : public RouDi_GTest { } - void SetUp(){}; - void TearDown(){}; + void SetUp() {}; + void TearDown() {}; }; TEST_F(InterfacePortRequestStackBlowup_test, RouDiMustContinue) diff --git a/iceoryx_posh/test/integrationtests/test_publisher_subscriber_communication.cpp b/iceoryx_posh/test/integrationtests/test_publisher_subscriber_communication.cpp index 749869f943..fa764e6a81 100644 --- a/iceoryx_posh/test/integrationtests/test_publisher_subscriber_communication.cpp +++ b/iceoryx_posh/test/integrationtests/test_publisher_subscriber_communication.cpp @@ -159,10 +159,10 @@ class PublisherSubscriberCommunicationWithBigPayload_test : public PublisherSubs public: PublisherSubscriberCommunicationWithBigPayload_test() : PublisherSubscriberCommunication_test( - MinimalIceoryxConfigBuilder() - .payloadChunkSize(SIZE_LARGER_THAN_4GB + additionalSizeForUserHeader) - .payloadChunkCount(2) - .create()) + MinimalIceoryxConfigBuilder() + .payloadChunkSize(SIZE_LARGER_THAN_4GB + additionalSizeForUserHeader) + .payloadChunkCount(2) + .create()) { } diff --git a/iceoryx_posh/test/mocks/gateway_base_mock.hpp b/iceoryx_posh/test/mocks/gateway_base_mock.hpp index 90a479bcdf..1f36f6d8c1 100644 --- a/iceoryx_posh/test/mocks/gateway_base_mock.hpp +++ b/iceoryx_posh/test/mocks/gateway_base_mock.hpp @@ -24,7 +24,7 @@ class MockGatewayBase { public: - MockGatewayBase(const iox::capro::Interfaces){}; + MockGatewayBase(const iox::capro::Interfaces) {}; MOCK_METHOD1(getCaProMessage, bool(iox::capro::CaproMessage&)); MOCK_METHOD0(getInterface, iox::capro::Interfaces()); diff --git a/iceoryx_posh/test/mocks/publisher_mock.hpp b/iceoryx_posh/test/mocks/publisher_mock.hpp index f12f7fd336..6dd865b979 100644 --- a/iceoryx_posh/test/mocks/publisher_mock.hpp +++ b/iceoryx_posh/test/mocks/publisher_mock.hpp @@ -36,10 +36,10 @@ class MockPublisherPortUser MockPublisherPortUser(std::nullptr_t) { } - MockPublisherPortUser(MemberType_t*){}; + MockPublisherPortUser(MemberType_t*) {}; - MockPublisherPortUser(const MockPublisherPortUser& rhs [[maybe_unused]]){}; - MockPublisherPortUser(MockPublisherPortUser&& rhs [[maybe_unused]]){}; + MockPublisherPortUser(const MockPublisherPortUser& rhs [[maybe_unused]]) {}; + MockPublisherPortUser(MockPublisherPortUser&& rhs [[maybe_unused]]) {}; MockPublisherPortUser& operator=(const MockPublisherPortUser& rhs [[maybe_unused]]) { return *this; @@ -79,7 +79,7 @@ class MockBasePublisher public: using PortType = MockPublisherPortUser; - MockBasePublisher(const iox::capro::ServiceDescription&, const iox::popo::PublisherOptions&){}; + MockBasePublisher(const iox::capro::ServiceDescription&, const iox::popo::PublisherOptions&) {}; MOCK_CONST_METHOD0(getUid, iox::popo::uid_t()); MOCK_CONST_METHOD0(getServiceDescription, iox::capro::ServiceDescription()); MOCK_METHOD0(offer, void(void)); diff --git a/iceoryx_posh/test/mocks/subscriber_mock.hpp b/iceoryx_posh/test/mocks/subscriber_mock.hpp index a627ce84a1..8594df1527 100644 --- a/iceoryx_posh/test/mocks/subscriber_mock.hpp +++ b/iceoryx_posh/test/mocks/subscriber_mock.hpp @@ -42,7 +42,7 @@ class MockSubscriberPortUser { } - MockSubscriberPortUser(iox::popo::SubscriberPortData*){}; + MockSubscriberPortUser(iox::popo::SubscriberPortData*) {}; iox::capro::ServiceDescription getCaProServiceDescription() const noexcept { return getServiceDescription(); @@ -70,7 +70,7 @@ class MockBaseSubscriber using SelfType = MockBaseSubscriber; using PortType = Port; - MockBaseSubscriber(const iox::capro::ServiceDescription&, const iox::popo::SubscriberOptions&){}; + MockBaseSubscriber(const iox::capro::ServiceDescription&, const iox::popo::SubscriberOptions&) {}; MOCK_CONST_METHOD0(getUid, iox::popo::uid_t()); MOCK_CONST_METHOD0(getServiceDescription, iox::capro::ServiceDescription()); MOCK_METHOD1(subscribe, void(uint64_t)); diff --git a/iceoryx_posh/test/mocks/trigger_handle_mock.hpp b/iceoryx_posh/test/mocks/trigger_handle_mock.hpp index bfd5124157..7de7f9a96d 100644 --- a/iceoryx_posh/test/mocks/trigger_handle_mock.hpp +++ b/iceoryx_posh/test/mocks/trigger_handle_mock.hpp @@ -27,9 +27,8 @@ using ::testing::_; class MockTriggeHandle { public: - MockTriggeHandle(iox::popo::ConditionVariableData&, - const iox::function, - const uint64_t) noexcept {}; + MockTriggeHandle(iox::popo::ConditionVariableData&, const iox::function, const uint64_t) noexcept { + }; MockTriggeHandle() noexcept { } diff --git a/iceoryx_posh/test/moduletests/test_capro_service.cpp b/iceoryx_posh/test/moduletests/test_capro_service.cpp index c15a06b6d5..7f70ae201d 100644 --- a/iceoryx_posh/test/moduletests/test_capro_service.cpp +++ b/iceoryx_posh/test/moduletests/test_capro_service.cpp @@ -43,8 +43,8 @@ class ServiceDescription_test : public Test IdString_t testService; IdString_t testInstance; IdString_t testEvent; - void SetUp(){}; - void TearDown(){}; + void SetUp() {}; + void TearDown() {}; }; /// BEGIN CLASSHASH TESTS diff --git a/iceoryx_posh/test/moduletests/test_gw_channel.cpp b/iceoryx_posh/test/moduletests/test_gw_channel.cpp index 69f42440ee..23612ab7c7 100644 --- a/iceoryx_posh/test/moduletests/test_gw_channel.cpp +++ b/iceoryx_posh/test/moduletests/test_gw_channel.cpp @@ -35,12 +35,12 @@ struct StubbedIceoryxTerminal struct Options { }; - StubbedIceoryxTerminal(const iox::capro::ServiceDescription&, const Options&){}; + StubbedIceoryxTerminal(const iox::capro::ServiceDescription&, const Options&) {}; }; struct StubbedExternalTerminal { - StubbedExternalTerminal(IdString_t, IdString_t, IdString_t){}; + StubbedExternalTerminal(IdString_t, IdString_t, IdString_t) {}; }; using TestChannel = iox::gw::Channel; @@ -49,8 +49,8 @@ using TestChannel = iox::gw::Channel; @@ -59,8 +59,8 @@ using TestGatewayGeneric = iox::gw::StubbedGatewayGeneric; class GatewayGenericTest : public Test { public: - void SetUp(){}; - void TearDown(){}; + void SetUp() {}; + void TearDown() {}; std::unique_ptr sut{new TestGatewayGeneric{}}; }; diff --git a/iceoryx_posh/test/moduletests/test_mepoo_config.cpp b/iceoryx_posh/test/moduletests/test_mepoo_config.cpp index 3b1f6e9e46..c2744e07a8 100644 --- a/iceoryx_posh/test/moduletests/test_mepoo_config.cpp +++ b/iceoryx_posh/test/moduletests/test_mepoo_config.cpp @@ -62,11 +62,8 @@ TEST_F(MePooConfig_Test, AddingMempoolWhenTheMemPoolConfigContainerIsFullReturns { sut.addMemPool({SIZE, CHUNK_COUNT}); } - IOX_EXPECT_FATAL_FAILURE( - [&] { - sut.addMemPool({SIZE, CHUNK_COUNT}); - }, - iox::PoshError::MEPOO__MAXIMUM_NUMBER_OF_MEMPOOLS_REACHED); + IOX_EXPECT_FATAL_FAILURE([&] { sut.addMemPool({SIZE, CHUNK_COUNT}); }, + iox::PoshError::MEPOO__MAXIMUM_NUMBER_OF_MEMPOOLS_REACHED); } TEST_F(MePooConfig_Test, SetDefaultMethodAddsTheDefaultMemPoolConfigurationToTheMemPoolConfigContainer) diff --git a/iceoryx_posh/test/moduletests/test_mepoo_typed_mempool.cpp b/iceoryx_posh/test/moduletests/test_mepoo_typed_mempool.cpp index 85dc74ecb2..766e7ed82c 100644 --- a/iceoryx_posh/test/moduletests/test_mepoo_typed_mempool.cpp +++ b/iceoryx_posh/test/moduletests/test_mepoo_typed_mempool.cpp @@ -53,8 +53,8 @@ class TypedMemPool_test : public Test { } - void SetUp(){}; - void TearDown(){}; + void SetUp() {}; + void TearDown() {}; alignas(MemPool::CHUNK_MEMORY_ALIGNMENT) uint8_t m_rawMemory[NumberOfChunks * ChunkSize + MpmcLoFFLiMemoryRequirement]; diff --git a/iceoryx_posh/test/moduletests/test_popo_base_publisher.cpp b/iceoryx_posh/test/moduletests/test_popo_base_publisher.cpp index 5996590bb8..65c6155b8d 100644 --- a/iceoryx_posh/test/moduletests/test_popo_base_publisher.cpp +++ b/iceoryx_posh/test/moduletests/test_popo_base_publisher.cpp @@ -30,7 +30,7 @@ class StubbedBasePublisher : public iox::popo::BasePublisher { public: StubbedBasePublisher(iox::capro::ServiceDescription) - : iox::popo::BasePublisher::BasePublisher(){}; + : iox::popo::BasePublisher::BasePublisher() {}; using iox::popo::BasePublisher::port; }; diff --git a/iceoryx_posh/test/moduletests/test_popo_chunk_queue.cpp b/iceoryx_posh/test/moduletests/test_popo_chunk_queue.cpp index fc632e5aa0..fac40ca90e 100644 --- a/iceoryx_posh/test/moduletests/test_popo_chunk_queue.cpp +++ b/iceoryx_posh/test/moduletests/test_popo_chunk_queue.cpp @@ -92,8 +92,8 @@ template class ChunkQueue_test : public Test, public ChunkQueue_testBase { public: - void SetUp() override{}; - void TearDown() override{}; + void SetUp() override {}; + void TearDown() override {}; using ChunkQueueData_t = ChunkQueueData; @@ -268,8 +268,8 @@ template class ChunkQueueFiFo_test : public Test, public ChunkQueue_testBase { public: - void SetUp() override{}; - void TearDown() override{}; + void SetUp() override {}; + void TearDown() override {}; using ChunkQueueData_t = ChunkQueueData; @@ -332,8 +332,8 @@ template class ChunkQueueSoFi_test : public Test, public ChunkQueue_testBase { public: - void SetUp() override{}; - void TearDown() override{}; + void SetUp() override {}; + void TearDown() override {}; using ChunkQueueData_t = ChunkQueueData; diff --git a/iceoryx_posh/test/moduletests/test_popo_client_port.cpp b/iceoryx_posh/test/moduletests/test_popo_client_port.cpp index fa992f98b6..c82fef1686 100644 --- a/iceoryx_posh/test/moduletests/test_popo_client_port.cpp +++ b/iceoryx_posh/test/moduletests/test_popo_client_port.cpp @@ -48,7 +48,7 @@ class ClientPort_test : public Test const ClientOptions& clientOptions, iox::mepoo::MemoryManager& memoryManager) : portData( - serviceDescription, runtimeName, iox::roudi::DEFAULT_UNIQUE_ROUDI_ID, clientOptions, &memoryManager) + serviceDescription, runtimeName, iox::roudi::DEFAULT_UNIQUE_ROUDI_ID, clientOptions, &memoryManager) { } diff --git a/iceoryx_posh/test/moduletests/test_popo_server_port_common.hpp b/iceoryx_posh/test/moduletests/test_popo_server_port_common.hpp index b3994ef1f1..17f846dbf0 100644 --- a/iceoryx_posh/test/moduletests/test_popo_server_port_common.hpp +++ b/iceoryx_posh/test/moduletests/test_popo_server_port_common.hpp @@ -47,7 +47,7 @@ class ServerPort_test : public Test const ServerOptions& serverOptions, MemoryManager& memoryManager) : portData( - serviceDescription, runtimeName, iox::roudi::DEFAULT_UNIQUE_ROUDI_ID, serverOptions, &memoryManager) + serviceDescription, runtimeName, iox::roudi::DEFAULT_UNIQUE_ROUDI_ID, serverOptions, &memoryManager) { } diff --git a/iceoryx_posh/test/moduletests/test_popo_smart_chunk_common.hpp b/iceoryx_posh/test/moduletests/test_popo_smart_chunk_common.hpp index a2d0210bb7..535595410b 100644 --- a/iceoryx_posh/test/moduletests/test_popo_smart_chunk_common.hpp +++ b/iceoryx_posh/test/moduletests/test_popo_smart_chunk_common.hpp @@ -55,7 +55,7 @@ class MockPublisherInterface : public PublisherInterface return mockSend(std::move(s)); } - MOCK_METHOD(void, mockSend, (SampleProducerType &&), (noexcept)); + MOCK_METHOD(void, mockSend, (SampleProducerType&&), (noexcept)); }; struct SampleTestCase @@ -92,7 +92,7 @@ class MockRequestInterface : public RpcInterface), mockSend, (RequestProducerType &&), (noexcept)); + MOCK_METHOD((iox::expected), mockSend, (RequestProducerType&&), (noexcept)); }; class RequestTestCase @@ -130,7 +130,7 @@ class MockResponseInterface : public RpcInterface), mockSend, (ResponseProducerType &&), (noexcept)); + MOCK_METHOD((iox::expected), mockSend, (ResponseProducerType&&), (noexcept)); }; class ResponseTestCase diff --git a/iceoryx_posh/test/moduletests/test_popo_used_chunk_list.cpp b/iceoryx_posh/test/moduletests/test_popo_used_chunk_list.cpp index 035bf9cae5..fc408a849f 100644 --- a/iceoryx_posh/test/moduletests/test_popo_used_chunk_list.cpp +++ b/iceoryx_posh/test/moduletests/test_popo_used_chunk_list.cpp @@ -43,7 +43,7 @@ class UsedChunkList_test : public Test memoryManager.configureMemoryManager(mempoolconf, memoryAllocator, memoryAllocator); }; - void TearDown() override{}; + void TearDown() override {}; SharedChunk getChunkFromMemoryManager() { diff --git a/iceoryx_posh/test/moduletests/test_popo_waitset.cpp b/iceoryx_posh/test/moduletests/test_popo_waitset.cpp index a6c6fc1ca6..916cb4e014 100644 --- a/iceoryx_posh/test/moduletests/test_popo_waitset.cpp +++ b/iceoryx_posh/test/moduletests/test_popo_waitset.cpp @@ -369,7 +369,7 @@ class WaitSet_test : public Test { return (m_isEventBased) ? iox::nullopt : iox::popo::WaitSetIsConditionSatisfiedCallback( - iox::in_place, *this, &SimpleEventClass::hasTriggered); + iox::in_place, *this, &SimpleEventClass::hasTriggered); } iox::popo::WaitSetIsConditionSatisfiedCallback @@ -378,7 +378,7 @@ class WaitSet_test : public Test m_simpleState1TriggerCallback = state; return (m_isEventBased) ? iox::nullopt : iox::popo::WaitSetIsConditionSatisfiedCallback( - iox::in_place, *this, &SimpleEventClass::hasTriggered); + iox::in_place, *this, &SimpleEventClass::hasTriggered); } iox::popo::WaitSetIsConditionSatisfiedCallback @@ -387,7 +387,7 @@ class WaitSet_test : public Test m_simpleState2TriggerCallback = state; return (m_isEventBased) ? iox::nullopt : iox::popo::WaitSetIsConditionSatisfiedCallback( - iox::in_place, *this, &SimpleEventClass::hasTriggered); + iox::in_place, *this, &SimpleEventClass::hasTriggered); } bool hasTriggered() const diff --git a/iceoryx_posh/test/moduletests/test_posh_runtime_single_process.cpp b/iceoryx_posh/test/moduletests/test_posh_runtime_single_process.cpp index a85f00981d..0d4141761c 100644 --- a/iceoryx_posh/test/moduletests/test_posh_runtime_single_process.cpp +++ b/iceoryx_posh/test/moduletests/test_posh_runtime_single_process.cpp @@ -45,9 +45,9 @@ class PoshRuntimeSingleProcess_test : public Test { } - virtual void SetUp(){}; + virtual void SetUp() {}; - virtual void TearDown(){}; + virtual void TearDown() {}; }; TEST_F(PoshRuntimeSingleProcess_test, ConstructorPoshRuntimeSingleProcessIsSuccess) diff --git a/iceoryx_posh/test/moduletests/test_roudi_cmd_line_parser_config_file_option.cpp b/iceoryx_posh/test/moduletests/test_roudi_cmd_line_parser_config_file_option.cpp index 06c8f692be..99492cb467 100644 --- a/iceoryx_posh/test/moduletests/test_roudi_cmd_line_parser_config_file_option.cpp +++ b/iceoryx_posh/test/moduletests/test_roudi_cmd_line_parser_config_file_option.cpp @@ -32,7 +32,7 @@ using namespace iox::config; class CmdLineParserConfigFileOption_test : public Test { public: - void SetUp(){}; + void SetUp() {}; void TearDown() { // Reset optind to be able to parse again diff --git a/iceoryx_posh/test/moduletests/test_roudi_mempool_introspection.cpp b/iceoryx_posh/test/moduletests/test_roudi_mempool_introspection.cpp index cd4e69f26c..66efc6b04f 100644 --- a/iceoryx_posh/test/moduletests/test_roudi_mempool_introspection.cpp +++ b/iceoryx_posh/test/moduletests/test_roudi_mempool_introspection.cpp @@ -55,7 +55,7 @@ class SegmentMock { public: SegmentMock() = default; - SegmentMock(const SegmentMock&){}; + SegmentMock(const SegmentMock&) {}; MePooMemoryManager_MOCK& getMemoryManager() { @@ -91,7 +91,7 @@ class MemPoolIntrospectionAccess SegmentManagerMock& segmentManager, MockPublisherPortUserAccess&& publisherPort) : iox::roudi::MemPoolIntrospection( - memoryManager, segmentManager, std::move(publisherPort)) + memoryManager, segmentManager, std::move(publisherPort)) { } MockPublisherPortUserAccess& getPublisherPort() diff --git a/iceoryx_posh/test/moduletests/test_roudi_process.cpp b/iceoryx_posh/test/moduletests/test_roudi_process.cpp index 8ea2c9f0af..d56e126827 100644 --- a/iceoryx_posh/test/moduletests/test_roudi_process.cpp +++ b/iceoryx_posh/test/moduletests/test_roudi_process.cpp @@ -40,7 +40,7 @@ class IpcInterfaceUser_Mock : public iox::roudi::Process public: IpcInterfaceUser_Mock() : iox::roudi::Process( - "TestProcess", DEFAULT_DOMAIN_ID, 200, PosixUser("foo"), HeartbeatPool::Index::INVALID, 255) + "TestProcess", DEFAULT_DOMAIN_ID, 200, PosixUser("foo"), HeartbeatPool::Index::INVALID, 255) { } MOCK_METHOD1(sendViaIpcChannel, void(IpcMessage)); diff --git a/iceoryx_posh/test/moduletests/test_version_info.cpp b/iceoryx_posh/test/moduletests/test_version_info.cpp index 0f4e8dcc97..8a1bdb5e15 100644 --- a/iceoryx_posh/test/moduletests/test_version_info.cpp +++ b/iceoryx_posh/test/moduletests/test_version_info.cpp @@ -27,10 +27,10 @@ using namespace iox::version; class VersionInfo_test : public Test { public: - VersionInfo_test(){}; - virtual ~VersionInfo_test(){}; - virtual void SetUp(){}; - virtual void TearDown(){}; + VersionInfo_test() {}; + virtual ~VersionInfo_test() {}; + virtual void SetUp() {}; + virtual void TearDown() {}; }; TEST_F(VersionInfo_test, SerializationWorkingOnOurVersion) diff --git a/iceoryx_posh/test/stubs/stub_gateway_generic.hpp b/iceoryx_posh/test/stubs/stub_gateway_generic.hpp index b82a0ab2ef..e6422998c1 100644 --- a/iceoryx_posh/test/stubs/stub_gateway_generic.hpp +++ b/iceoryx_posh/test/stubs/stub_gateway_generic.hpp @@ -39,7 +39,7 @@ class StubbedGatewayGeneric : public TestGatewayGeneric { public: StubbedGatewayGeneric() - : TestGatewayGeneric(iox::capro::Interfaces::INTERNAL){}; + : TestGatewayGeneric(iox::capro::Interfaces::INTERNAL) {}; void loadConfiguration(const config::GatewayConfig&) noexcept {