diff --git a/.github/workflows/build-test.yml b/.github/workflows/build-test.yml index b9988093ab..757d27128d 100644 --- a/.github/workflows/build-test.yml +++ b/.github/workflows/build-test.yml @@ -7,7 +7,7 @@ on: push: branches: [ master ] pull_request: - branches: [ master, release* ] + branches: [ master, release*, iox-* ] jobs: pre-flight-check: diff --git a/.github/workflows/lint_pull_request.yml b/.github/workflows/lint_pull_request.yml index f8e840a120..64728ebb4e 100644 --- a/.github/workflows/lint_pull_request.yml +++ b/.github/workflows/lint_pull_request.yml @@ -2,7 +2,7 @@ name: Code Linting on Pull-Requests on: pull_request: - branches: [ master, release* ] + branches: [ master, release*, iox-* ] jobs: clang-tidy-review: diff --git a/iceoryx_examples/complexdata/topic_data.hpp b/iceoryx_examples/complexdata/topic_data.hpp index 4d647c7f45..f89017f798 100644 --- a/iceoryx_examples/complexdata/topic_data.hpp +++ b/iceoryx_examples/complexdata/topic_data.hpp @@ -18,7 +18,7 @@ #define IOX_EXAMPLES_COMPLEXDATA_TOPIC_DATA_HPP #include "iceoryx_dust/cxx/forward_list.hpp" -#include "iceoryx_hoofs/cxx/list.hpp" +#include "iox/list.hpp" #include "iox/optional.hpp" #include "iox/stack.hpp" #include "iox/string.hpp" diff --git a/iceoryx_hoofs/include/iceoryx_hoofs/internal/cxx/list.inl b/iceoryx_hoofs/container/include/iox/detail/list.inl similarity index 99% rename from iceoryx_hoofs/include/iceoryx_hoofs/internal/cxx/list.inl rename to iceoryx_hoofs/container/include/iox/detail/list.inl index 1f319722ce..445f246caa 100644 --- a/iceoryx_hoofs/include/iceoryx_hoofs/internal/cxx/list.inl +++ b/iceoryx_hoofs/container/include/iox/detail/list.inl @@ -15,18 +15,14 @@ // // SPDX-License-Identifier: Apache-2.0 -#ifndef IOX_HOOFS_CXX_LIST_INL -#define IOX_HOOFS_CXX_LIST_INL +#ifndef IOX_HOOFS_CONTAINER_LIST_INL +#define IOX_HOOFS_CONTAINER_LIST_INL - -#include "iceoryx_hoofs/cxx/list.hpp" +#include "iox/list.hpp" #include "iox/logging.hpp" - namespace iox { -namespace cxx -{ /// @NOLINTJUSTIFICATION m_data fields are explicitly initialized whenever a new element is inserted /// into the list /// @NOLINTNEXTLINE(cppcoreguidelines-pro-type-member-init,hicpp-member-init) @@ -699,7 +695,6 @@ inline bool list::isInvalidIterOrDifferentLists(const const_iterato return isInvalidIterator(iter); } -} // namespace cxx } // namespace iox -#endif // IOX_HOOFS_CXX_LIST_INL +#endif // IOX_HOOFS_CONTAINER_LIST_INL diff --git a/iceoryx_hoofs/include/iceoryx_hoofs/cxx/list.hpp b/iceoryx_hoofs/container/include/iox/list.hpp similarity index 99% rename from iceoryx_hoofs/include/iceoryx_hoofs/cxx/list.hpp rename to iceoryx_hoofs/container/include/iox/list.hpp index 0fa95e8b8b..e2acc674ec 100644 --- a/iceoryx_hoofs/include/iceoryx_hoofs/cxx/list.hpp +++ b/iceoryx_hoofs/container/include/iox/list.hpp @@ -15,8 +15,8 @@ // // SPDX-License-Identifier: Apache-2.0 -#ifndef IOX_HOOFS_CXX_LIST_HPP -#define IOX_HOOFS_CXX_LIST_HPP +#ifndef IOX_HOOFS_CONTAINER_LIST_HPP +#define IOX_HOOFS_CONTAINER_LIST_HPP #include "iceoryx_hoofs/cxx/requires.hpp" #include "iox/uninitialized_array.hpp" @@ -28,8 +28,6 @@ namespace iox { -namespace cxx -{ /// @brief C++11 compatible bi-directional list implementation. /// @details Adjustments in the API were done to not use exceptions and serve the requirement of /// a data structure movable over shared memory. @@ -386,9 +384,8 @@ class list size_type m_size{0U}; }; // list -} // namespace cxx } // namespace iox -#include "iceoryx_hoofs/internal/cxx/list.inl" +#include "iox/detail/list.inl" -#endif // IOX_HOOFS_CXX_LIST_HPP +#endif // IOX_HOOFS_CONTAINER_LIST_HPP diff --git a/iceoryx_hoofs/legacy/include/iceoryx_hoofs/cxx/list.hpp b/iceoryx_hoofs/legacy/include/iceoryx_hoofs/cxx/list.hpp new file mode 100644 index 0000000000..13924e1e82 --- /dev/null +++ b/iceoryx_hoofs/legacy/include/iceoryx_hoofs/cxx/list.hpp @@ -0,0 +1,34 @@ +// Copyright (c) 2023 by Mathias Kraus . All rights reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. +// +// SPDX-License-Identifier: Apache-2.0 + +#ifndef IOX_HOOFS_CXX_LIST_HPP +#define IOX_HOOFS_CXX_LIST_HPP + +#include "iox/detail/deprecation_marker.hpp" +#include "iox/list.hpp" + +IOX_DEPRECATED_HEADER_SINCE(3, "Please include 'iox/list.hpp' instead.") + +namespace iox +{ +namespace cxx +{ +/// @deprecated use 'iox::list' instead of 'iox::cxx::list' +using iox::list; +} // namespace cxx +} // namespace iox + +#endif // IOX_HOOFS_CXX_LIST_HPP diff --git a/iceoryx_hoofs/test/moduletests/test_cxx_list.cpp b/iceoryx_hoofs/test/moduletests/test_cxx_list.cpp index 725a95a903..d6614a0aa7 100644 --- a/iceoryx_hoofs/test/moduletests/test_cxx_list.cpp +++ b/iceoryx_hoofs/test/moduletests/test_cxx_list.cpp @@ -15,11 +15,11 @@ // // SPDX-License-Identifier: Apache-2.0 -#include "iceoryx_hoofs/cxx/list.hpp" #include "iceoryx_hoofs/error_handling/error_handling.hpp" #include "iceoryx_hoofs/testing/ctor_and_assignment_operator_test_class.hpp" #include "iceoryx_hoofs/testing/fatal_failure.hpp" #include "iox/attributes.hpp" +#include "iox/list.hpp" #include "iox/logging.hpp" #include "test.hpp" @@ -27,7 +27,7 @@ namespace { using namespace ::testing; -using namespace iox::cxx; +using namespace iox; using namespace iox::testing; constexpr uint64_t TESTLISTCAPACITY{10U}; diff --git a/iceoryx_posh/include/iceoryx_posh/internal/roudi/introspection/process_introspection.hpp b/iceoryx_posh/include/iceoryx_posh/internal/roudi/introspection/process_introspection.hpp index fd8b01906d..0a75f9d800 100644 --- a/iceoryx_posh/include/iceoryx_posh/internal/roudi/introspection/process_introspection.hpp +++ b/iceoryx_posh/include/iceoryx_posh/internal/roudi/introspection/process_introspection.hpp @@ -17,12 +17,12 @@ #ifndef IOX_POSH_ROUDI_INTROSPECTION_PROCESS_INTROSPECTION_HPP #define IOX_POSH_ROUDI_INTROSPECTION_PROCESS_INTROSPECTION_HPP -#include "iceoryx_hoofs/cxx/list.hpp" #include "iceoryx_hoofs/internal/concurrent/periodic_task.hpp" #include "iceoryx_posh/iceoryx_posh_types.hpp" #include "iceoryx_posh/internal/popo/ports/publisher_port_user.hpp" #include "iceoryx_posh/roudi/introspection_types.hpp" #include "iox/function.hpp" +#include "iox/list.hpp" #include @@ -94,7 +94,7 @@ class ProcessIntrospection void send() noexcept; private: - using ProcessList_t = cxx::list; + using ProcessList_t = iox::list; ProcessList_t m_processList; bool m_processListNewData{true}; // true because we want to have a valid field, even with an empty list diff --git a/iceoryx_posh/include/iceoryx_posh/internal/roudi/process_manager.hpp b/iceoryx_posh/include/iceoryx_posh/internal/roudi/process_manager.hpp index 93dbe1141b..9d7c9aae37 100644 --- a/iceoryx_posh/include/iceoryx_posh/internal/roudi/process_manager.hpp +++ b/iceoryx_posh/include/iceoryx_posh/internal/roudi/process_manager.hpp @@ -17,7 +17,6 @@ #ifndef IOX_POSH_ROUDI_PROCESS_MANAGER_HPP #define IOX_POSH_ROUDI_PROCESS_MANAGER_HPP -#include "iceoryx_hoofs/cxx/list.hpp" #include "iceoryx_hoofs/posix_wrapper/posix_access_rights.hpp" #include "iceoryx_posh/internal/mepoo/segment_manager.hpp" #include "iceoryx_posh/internal/roudi/introspection/process_introspection.hpp" @@ -27,6 +26,7 @@ #include "iceoryx_posh/mepoo/chunk_header.hpp" #include "iceoryx_posh/version/compatibility_check_level.hpp" #include "iceoryx_posh/version/version_info.hpp" +#include "iox/list.hpp" #include #include @@ -46,7 +46,7 @@ class ProcessManagerInterface class ProcessManager : public ProcessManagerInterface { public: - using ProcessList_t = cxx::list; + using ProcessList_t = iox::list; using PortConfigInfo = iox::runtime::PortConfigInfo; enum class TerminationFeedback diff --git a/iceoryx_posh/test/integrationtests/test_publisher_subscriber_communication.cpp b/iceoryx_posh/test/integrationtests/test_publisher_subscriber_communication.cpp index 2090f6e8a1..cc55e7bcbd 100644 --- a/iceoryx_posh/test/integrationtests/test_publisher_subscriber_communication.cpp +++ b/iceoryx_posh/test/integrationtests/test_publisher_subscriber_communication.cpp @@ -16,7 +16,6 @@ // SPDX-License-Identifier: Apache-2.0 #include "iceoryx_dust/cxx/forward_list.hpp" -#include "iceoryx_hoofs/cxx/list.hpp" #include "iceoryx_hoofs/testing/barrier.hpp" #include "iceoryx_hoofs/testing/watch_dog.hpp" #include "iceoryx_posh/popo/publisher.hpp" @@ -24,6 +23,7 @@ #include "iceoryx_posh/roudi_env/minimal_roudi_config.hpp" #include "iceoryx_posh/runtime/posh_runtime.hpp" #include "iceoryx_posh/testing/roudi_gtest.hpp" +#include "iox/list.hpp" #include "iox/optional.hpp" #include "iox/stack.hpp" #include "iox/string.hpp"