This is an automated email from the ASF dual-hosted git repository. bneradt pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/trafficserver-libswoc.git
commit 8870ee18ffc5e515b7275bc138c2a4a49e420f30 Author: Chris McFarlen <[email protected]> AuthorDate: Mon Sep 22 17:23:12 2025 -0500 Remove maybe problematic constructors on storage_type (#12493) * Remove maybe problematic constructors on storage_type * remove the code rather than comment. --- code/include/swoc/IPRange.h | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/code/include/swoc/IPRange.h b/code/include/swoc/IPRange.h index 64740b3..3cc3327 100644 --- a/code/include/swoc/IPRange.h +++ b/code/include/swoc/IPRange.h @@ -609,14 +609,6 @@ class IPRangeView { std::monostate _nil; ///< No data present. IP4Range const *_4; ///< IPv4 range. IP6Range const *_6; ///< IPv6 range. - void const *_void; ///< Used only for fast copy in construction and assignment. - - // These constructors are needed to make the default construction / assignent methods work. - // Otherwise the compiler thinks the union isn't initialized. - storage_type() = default; - storage_type(std::monostate) {} - storage_type(storage_type const &that); - storage_type &operator=(storage_type const &rhs); }; public: @@ -1894,14 +1886,6 @@ IPSpace<PAYLOAD>::iterator::operator--(int) -> self_type { // +++ IPRange +++ -inline IPRangeView::storage_type::storage_type(IPRangeView::storage_type const &that) : _void(that._void) {} - -inline IPRangeView::storage_type & -IPRangeView::storage_type::operator=(IPRangeView::storage_type const &rhs) { - _void = rhs._void; - return *this; -} - inline IP4Range::IP4Range(string_view const &text) { this->load(text); }
