Using inline namespaces

sean-parent · sean-parent · commit b6778509d3a9 · 2025-06-25T15:18:30.000-07:00
diff --git a/.gitignore b/.gitignore
@@ -10,3 +10,4 @@ build_asl/
 .vs/
 .vscode/settings.json
 /.cache
+compile_commands.json
diff --git a/adobe/any_regular.hpp b/adobe/any_regular.hpp
@@ -301,7 +301,7 @@ const vtable_t any_regular_model_remote<T>::vtable_s = {
 
 /**************************************************************************************************/
 
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
diff --git a/adobe/any_regular_fwd.hpp b/adobe/any_regular_fwd.hpp
@@ -14,7 +14,7 @@
 
 namespace adobe {
 
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
@@ -24,8 +24,6 @@ class any_regular_t;
 
 } // namespace version_1
 
-using version_1::any_regular_t;
-
 } // namespace adobe
 
 /**************************************************************************************************/
diff --git a/adobe/array.hpp b/adobe/array.hpp
@@ -36,7 +36,7 @@ inline void push_back(array_t& v, any_regular_t x) { v.push_back(std::move(x));
 #ifdef ADOBE_STD_SERIALIZATION
 
 // To be findable by ADL, the operator must be in the same namespace as `any_regular_t.`
-namespace version_1 {
+inline namespace version_1 {
 
 inline std::ostream& operator<<(std::ostream& out, const array_t& x) {
     out << begin_sequence;
diff --git a/adobe/closed_hash.hpp b/adobe/closed_hash.hpp
@@ -117,7 +117,7 @@ struct set_next_fn<implementation::closed_hash_iterator<T, V>> {
 
 #ifndef ADOBE_NO_DOCUMENTATION
 
-namespace version_1 {
+inline namespace version_1 {
 
 #endif
 
@@ -133,7 +133,8 @@ A \c closed_hash_set is a hash based associative container, similar to a hash_se
 
 \model_of
     - \ref concept_regular_type
-    - [UniqueHashedAssociativeContainer](https://www.boost.org/sgi/stl/UniqueHashedAssociativeContainer.html)
+    -
+[UniqueHashedAssociativeContainer](https://www.boost.org/sgi/stl/UniqueHashedAssociativeContainer.html)
 
 \todo
     - re-order parameters so key_function is after comparison - to be consistent with
@@ -631,17 +632,16 @@ pair.
 
 \model_of
     - \ref concept_regular_type
-    - [UniqueHashedAssociativeContainer](https://www.boost.org/sgi/stl/UniqueHashedAssociativeContainer.html)
+    -
+[UniqueHashedAssociativeContainer](https://www.boost.org/sgi/stl/UniqueHashedAssociativeContainer.html)
 
 
 */
 
 template <typename Key, typename T, typename Hash, typename Pred, typename A>
-class closed_hash_map
-    : public closed_hash_set<std::pair<Key, T>, get_element<0>, Hash, Pred, A> {
+class closed_hash_map : public closed_hash_set<std::pair<Key, T>, get_element<0>, Hash, Pred, A> {
 
-    using set_type =
-        closed_hash_set<std::pair<Key, T>, get_element<0>, Hash, Pred, A>;
+    using set_type = closed_hash_set<std::pair<Key, T>, get_element<0>, Hash, Pred, A>;
 
 public:
     typedef T mapped_type;
diff --git a/adobe/closed_hash_fwd.hpp b/adobe/closed_hash_fwd.hpp
@@ -22,7 +22,7 @@
 /**************************************************************************************************/
 
 namespace adobe {
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
@@ -38,9 +38,6 @@ class closed_hash_map;
 
 } // namespace version_1
 
-using version_1::closed_hash_map;
-using version_1::closed_hash_set;
-
 } // namespace adobe
 
 /**************************************************************************************************/
diff --git a/adobe/copy_on_write.hpp b/adobe/copy_on_write.hpp
@@ -31,7 +31,7 @@ namespace adobe {
 
 /**************************************************************************************************/
 
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
@@ -377,10 +377,6 @@ void copy_on_write<T, A>::release_default() {
 
 /**************************************************************************************************/
 
-using version_1::copy_on_write;
-
-/**************************************************************************************************/
-
 } // namespace adobe
 
 /**************************************************************************************************/
diff --git a/adobe/dictionary.hpp b/adobe/dictionary.hpp
@@ -28,7 +28,7 @@
 /**************************************************************************************************/
 
 namespace adobe {
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
@@ -69,8 +69,6 @@ std::ostream& operator<<(std::ostream& out, const dictionary_t& x);
 
 } // namespace version_1
 
-using version_1::get_value;
-
 } // namespace adobe
 
 /**************************************************************************************************/
diff --git a/adobe/dictionary_fwd.hpp b/adobe/dictionary_fwd.hpp
@@ -19,7 +19,7 @@
 /**************************************************************************************************/
 
 namespace adobe {
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
@@ -29,8 +29,6 @@ typedef closed_hash_map<name_t, any_regular_t> dictionary_t;
 
 } // namespace version_1
 
-using version_1::dictionary_t;
-
 } // namespace adobe
 
 /**************************************************************************************************/
diff --git a/adobe/empty.hpp b/adobe/empty.hpp
@@ -22,7 +22,7 @@
 /**************************************************************************************************/
 
 namespace adobe {
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
@@ -53,8 +53,6 @@ std::ostream& operator<<(std::ostream& stream, const empty_t&);
 
 } // namespace version_1
 
-using version_1::empty_t;
-
 /**************************************************************************************************/
 
 /*!
diff --git a/adobe/function.hpp b/adobe/function.hpp
@@ -151,7 +151,7 @@ typename F::concept_t* create_model(const T& x) {
 
 /**************************************************************************************************/
 
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
@@ -300,7 +300,6 @@ const typename function<R()>::vtable_type function<R()>::model<T>::vtable_s = {
 /**************************************************************************************************/
 
 } // namespace version_1
-using namespace version_1;
 } // namespace adobe
 
 /**************************************************************************************************/
diff --git a/adobe/future.hpp b/adobe/future.hpp
@@ -23,7 +23,7 @@ namespace adobe {
 
 /**************************************************************************************************/
 
-namespace details {
+namespace detail {
 
 struct any_packaged_task_ {
 
@@ -59,7 +59,7 @@ struct any_packaged_task_ {
 void async_(any_packaged_task_&&);
 void async_(const std::chrono::steady_clock::time_point&, any_packaged_task_&&);
 
-} // namespace details
+} // namespace detail
 
 /**************************************************************************************************/
 
@@ -73,7 +73,7 @@ auto async(F&& f, Args&&... args) -> std::future<typename adobe::invoke_result_t
     auto p = packaged_t(forward<F>(f), forward<Args>(args)...);
     auto result = p.get_future();
 
-    details::async_(move(p));
+    detail::async_(move(p));
 
     return result;
 }
@@ -92,7 +92,7 @@ auto async(const std::chrono::time_point<std::chrono::steady_clock, Duration>& w
     auto p = packaged_t(forward<F>(f), forward<Args>(args)...);
     auto result = p.get_future();
 
-    details::async_(when, move(p));
+    detail::async_(when, move(p));
     return result;
 }
 
@@ -158,7 +158,7 @@ class shared_task_queue {
     };
 
     static auto make_task_queue_() -> std::shared_ptr<task_queue_>;
-    void async_(details::any_packaged_task_&&);
+    void async_(detail::any_packaged_task_&&);
 
     static void continue_(const std::shared_ptr<task_queue_>&);
     static void suspend_(const std::shared_ptr<task_queue_>&);
diff --git a/adobe/memory.hpp b/adobe/memory.hpp
@@ -388,7 +388,7 @@ F uninitialized_move(I f, I l, F r) {
 
 /**************************************************************************************************/
 
-namespace version_1 {
+inline namespace version_1 {
 
 //! \addtogroup memory
 //! @{
diff --git a/adobe/memory_fwd.hpp b/adobe/memory_fwd.hpp
@@ -16,7 +16,7 @@
 /**************************************************************************************************/
 
 namespace adobe {
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
@@ -27,8 +27,6 @@ class capture_allocator;
 
 } // namespace version_1
 
-using version_1::capture_allocator;
-
 } // namespace adobe
 
 /**************************************************************************************************/
diff --git a/adobe/string_fwd.hpp b/adobe/string_fwd.hpp
@@ -17,7 +17,7 @@
 /**************************************************************************************************/
 
 namespace adobe {
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
diff --git a/adobe/task.hpp b/adobe/task.hpp
@@ -67,7 +67,7 @@ typedef std::function<void()>* cancellation_token_registration;
 
 /**************************************************************************************************/
 
-namespace details {
+namespace detail {
 
 /**************************************************************************************************/
 
@@ -142,7 +142,7 @@ struct cancel_state {
 
 /**************************************************************************************************/
 
-} // namespace details
+} // namespace detail
 
 /**************************************************************************************************/
 
@@ -174,9 +174,9 @@ class cancellation_token {
 private:
     friend class cancellation_token_source;
     cancellation_token() {}
-    explicit cancellation_token(std::shared_ptr<details::cancel_state> s) : state_(std::move(s)) {}
+    explicit cancellation_token(std::shared_ptr<detail::cancel_state> s) : state_(std::move(s)) {}
 
-    std::shared_ptr<details::cancel_state> state_;
+    std::shared_ptr<detail::cancel_state> state_;
 };
 
 inline bool operator!=(const cancellation_token& x, const cancellation_token& y) {
@@ -187,7 +187,7 @@ inline bool operator!=(const cancellation_token& x, const cancellation_token& y)
 
 class cancellation_token_source {
 public:
-    cancellation_token_source() : state_(std::make_shared<details::cancel_state>()) {}
+    cancellation_token_source() : state_(std::make_shared<detail::cancel_state>()) {}
 
     void cancel() const { state_->cancel(); }
 
@@ -199,7 +199,7 @@ class cancellation_token_source {
     }
 
 private:
-    std::shared_ptr<details::cancel_state> state_;
+    std::shared_ptr<detail::cancel_state> state_;
 };
 
 inline bool operator!=(const cancellation_token_source& x, const cancellation_token_source& y) {
diff --git a/adobe/vector.hpp b/adobe/vector.hpp
@@ -36,7 +36,7 @@
 /**************************************************************************************************/
 
 namespace adobe {
-namespace version_1 {
+inline namespace version_1 {
 
 /*!
 \defgroup container Containers
diff --git a/adobe/vector_fwd.hpp b/adobe/vector_fwd.hpp
@@ -17,7 +17,7 @@
 /**************************************************************************************************/
 
 namespace adobe {
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
diff --git a/source/any_regular.cpp b/source/any_regular.cpp
@@ -45,7 +45,7 @@ namespace adobe {
 
 /**************************************************************************************************/
 
-namespace version_1 {
+inline namespace version_1 {
 
 std::ostream& operator<<(std::ostream& out, const dictionary_t& x) {
     typedef table_index<const name_t, const dictionary_t::value_type> index_type;
diff --git a/source/empty.cpp b/source/empty.cpp
@@ -14,7 +14,7 @@
 /**************************************************************************************************/
 
 namespace adobe {
-namespace version_1 {
+inline namespace version_1 {
 
 /**************************************************************************************************/
 
diff --git a/source/future.cpp b/source/future.cpp
diff --git a/source/memory.cpp b/source/memory.cpp
