Use core::string_view directly instead of alias

Replace all uses of string_view and basic_string_view with
core::string_view and core::basic_string_view respectively
throughout the codebase. This improves clarity in both documentation
and source code by explicitly showing that the library uses
boost::core::string_view.

The type aliases in string_type.hpp are retained for backward
compatibility, but all internal usage now references the
core:: qualified names directly.

Changes include:
- All header files (.hpp)
- All inline implementation files (.ipp)
- Example files (.cpp, .hpp)
- Test files (.cpp, .hpp)
- Documentation files (.qbk)

Closes #3046

Author: ssam18

.gitignore

@@ -5,3 +5,4 @@ bin64/
 Win32/
 x64/
 
+replace_string_view.py

doc/qbk/04_http/08_chunked_encoding.qbk

@@ -179,7 +179,7 @@ interface provided by __parser__:
         void
         callback(
             std::uint64_t size,         // Size of the chunk, zero for the last chunk
-            string_view extensions,     // The chunk-extensions in raw form
+            core::string_view extensions,     // The chunk-extensions in raw form
             error_code& ec);            // May be set by the callback to indicate an error
         ```
     ]
@@ -205,7 +205,7 @@ interface provided by __parser__:
         std::size_t
         callback(
             std::uint64_t remain,       // Octets remaining in this chunk, includes `body`
-            string_view body,           // A buffer holding some or all of the remainder of the chunk body
+            core::string_view body,           // A buffer holding some or all of the remainder of the chunk body
             error_code& ec);            // May be set by the callback to indicate an error
         ```
     ]

doc/qbk/07_concepts/Fields.qbk

@@ -30,7 +30,7 @@ In this table:
 * `c` denotes a (possibly const) value of type `F`.
 * `b` is a value of type `bool`
 * `n` is a value of type `boost::optional<std::uint64_t>`.
-* `s` is a value of type [link beast.ref.boost__beast__string_view `string_view`].
+* `s` is a value of type [link beast.ref.boost__beast__string_view `core::string_view`].
 * `v` is a value of type `unsigned int` representing the HTTP-version.
 
 [table Valid expressions
@@ -43,7 +43,7 @@ In this table:
     ]
 ][
     [`c.get_method_impl()`]
-    [`string_view`]
+    [`core::string_view`]
     [
         Returns the method text.
         The implementation only calls this function for request
@@ -52,14 +52,14 @@ In this table:
     ]
 ][
     [`c.get_target_impl()`]
-    [`string_view`]
+    [`core::string_view`]
     [
         Returns the target string.
         The implementation only calls this function for request headers.
     ]
 ][
     [`c.get_reason_impl()`]
-    [`string_view`]
+    [`core::string_view`]
     [
         Returns the obsolete request text.
         The implementation only calls this for response headers when

doc/qbk/08_design/1_http_message.qbk

@@ -257,12 +257,12 @@ struct header<true, Fields> : Fields
     int         version;
 
     verb        method() const;
-    string_view method_string() const;
+    core::string_view method_string() const;
     void        method(verb);
-    void        method(string_view);
+    void        method(core::string_view);
 
-    string_view target(); const;
-    void        target(string_view);
+    core::string_view target(); const;
+    void        target(core::string_view);
 
 private:
     verb method_;
@@ -274,8 +274,8 @@ struct header<false, Fields> : Fields
 {
     int         version;
     int         result;
-    string_view reason() const;
-    void        reason(string_view);
+    core::string_view reason() const;
+    void        reason(core::string_view);
 };
 ```
 
@@ -298,14 +298,14 @@ struct header<false, Fields> : Fields
     int     version;
     int     status;
 
-    string_view
+    core::string_view
     reason() const
     {
         return this->reason_impl(); // protected member of Fields
     }
 
     void
-    reason(string_view s)
+    reason(core::string_view s)
     {
         this->reason_impl(s);       // protected member of Fields
     }

doc/qbk/release_notes.qbk

@@ -258,7 +258,7 @@
 [*Miscellaneous]
 
 * [issue 2363] Remove `BOOST_BEAST_USE_STD_STRING_VIEW`
-* [issue 2417] use boost::core::string_view. This improves inter-conversion between string_view implementations. Some observable differences for users:
+* [issue 2417] use boost::core::string_view. This improves inter-conversion between core::string_view implementations. Some observable differences for users:
  * `core::string_view` no longer supports the `.to_string()` or `.clear()` extensions from Utility
  * code that relied on `.max_size()` returning `.size(),` needs to be fixed to use `.size()` instead
  * `remove_suffix()` and `remove_prefix()` were more lenient than the standard specs; be sure you don't rely on it clamping the argument to valid range
@@ -426,7 +426,7 @@
 * [issue 1956] Deprecate `string_param` (API Change)
   ['Actions Required]
   `string_param`, which was previously the argument type when setting field values
-  has been replaced by `string_view`. Because of this, it is no longer possible to
+  has been replaced by `core::string_view`. Because of this, it is no longer possible to
   set message field values directly as integrals.
   Users are requied to convert numeric arguments to a string type prior to calling
   `fields::set` et. al.

example/doc/http_examples.hpp

@@ -376,7 +376,7 @@ response<empty_body>
 do_head_request(
     SyncStream& stream,
     DynamicBuffer& buffer,
-    string_view target,
+    core::string_view target,
     error_code& ec)
 {
     // Do some type checking to be a good citizen
@@ -979,7 +979,7 @@ print_chunked_body(
     // after each chunk header and also after the last chunk.
     auto header_cb =
     [&](std::uint64_t size,         // Size of the chunk, or zero for the last chunk
-        string_view extensions,     // The raw chunk-extensions string. Already validated.
+        core::string_view extensions,     // The raw chunk-extensions string. Already validated.
         error_code& ev)             // We can set this to indicate an error
     {
         // Parse the chunk extensions so we can access them easily
@@ -1008,7 +1008,7 @@ print_chunked_body(
     // more times for each piece of a chunk body.
     auto body_cb =            
     [&](std::uint64_t remain,   // The number of bytes left in this chunk
-        string_view body,       // A buffer holding chunk body data
+        core::string_view body,       // A buffer holding chunk body data
         error_code& ec)         // We can set this to indicate an error
     {
         // If this is the last piece of the chunk body,

include/boost/beast/_experimental/test/impl/stream.ipp

@@ -123,7 +123,7 @@ template<class Executor>
 basic_stream<Executor>::
 basic_stream(
     net::io_context& ioc,
-    string_view s)
+    core::string_view s)
     : in_(detail::stream_service::make_impl(ioc.get_executor(), nullptr))
 {
     in_->b.commit(net::buffer_copy(
@@ -136,7 +136,7 @@ basic_stream<Executor>::
 basic_stream(
     net::io_context& ioc,
     fail_count& fc,
-    string_view s)
+    core::string_view s)
     : in_(detail::stream_service::make_impl(ioc.get_executor(), &fc))
 {
     in_->b.commit(net::buffer_copy(
@@ -161,7 +161,7 @@ connect(basic_stream& remote)
 }
 
 template<class Executor>
-string_view
+core::string_view
 basic_stream<Executor>::
 str() const
 {
@@ -175,7 +175,7 @@ str() const
 template<class Executor>
 void
 basic_stream<Executor>::
-append(string_view s)
+append(core::string_view s)
 {
     std::lock_guard<std::mutex> lock{in_->m};
     in_->b.commit(net::buffer_copy(

include/boost/beast/_experimental/test/stream.hpp

@@ -281,7 +281,7 @@ class basic_stream
     */
     basic_stream(
         net::io_context& ioc,
-        string_view s);
+        core::string_view s);
 
     /** Construct a stream
 
@@ -301,7 +301,7 @@ class basic_stream
     basic_stream(
         net::io_context& ioc,
         fail_count& fc,
-        string_view s);
+        core::string_view s);
 
     /// Establish a connection
     void
@@ -333,12 +333,12 @@ class basic_stream
     }
 
     /// Returns a string view representing the pending input data
-    string_view
+    core::string_view
     str() const;
 
     /// Appends a string to the pending input data
     void
-    append(string_view s);
+    append(core::string_view s);
 
     /// Clear the pending input area
     void

include/boost/beast/core/detail/impl/temporary_buffer.ipp

@@ -23,15 +23,15 @@ namespace detail {
 
 void
 temporary_buffer::
-append(string_view s)
+append(core::string_view s)
 {
     grow(s.size());
     unchecked_append(s);
 }
 
 void
 temporary_buffer::
-append(string_view s1, string_view s2)
+append(core::string_view s1, core::string_view s2)
 {
     grow(s1.size() + s2.size());
     unchecked_append(s1);
@@ -40,7 +40,7 @@ append(string_view s1, string_view s2)
 
 void
 temporary_buffer::
-unchecked_append(string_view s)
+unchecked_append(core::string_view s)
 {
     auto n = s.size();
     std::memcpy(&data_[size_], s.data(), n);

include/boost/beast/core/detail/static_ostream.hpp

@@ -50,7 +50,7 @@ class static_ostream_buffer
     {
     }
 
-    string_view
+    core::string_view
     str() const
     {
         if(! s_.empty())
@@ -128,7 +128,7 @@ class static_ostream : public std::basic_ostream<char>
         imbue(std::locale::classic());
     }
 
-    string_view
+    core::string_view
     str() const
     {
         return osb_.str();