From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 4812A3858292; Tue, 5 Jul 2022 14:11:33 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 4812A3858292 From: "redi at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug libstdc++/106201] filesystem::directory_iterator is a borrowable range? Date: Tue, 05 Jul 2022 14:11:33 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: libstdc++ X-Bugzilla-Version: 13.0 X-Bugzilla-Keywords: rejects-valid X-Bugzilla-Severity: normal X-Bugzilla-Who: redi at gcc dot gnu.org X-Bugzilla-Status: ASSIGNED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: redi at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: assigned_to cf_reconfirmed_on keywords bug_status everconfirmed Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: gcc-bugs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-bugs mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 05 Jul 2022 14:11:33 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D106201 Jonathan Wakely changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|unassigned at gcc dot gnu.org |redi at gcc dot gnu= .org Last reconfirmed| |2022-07-05 Keywords| |rejects-valid Status|UNCONFIRMED |ASSIGNED Ever confirmed|0 |1 --- Comment #1 from Jonathan Wakely --- (In reply to Patrick Welche from comment #0) > Oh: https://cplusplus.github.io/LWG/issue3480 > But: it says "could be fixed ... (as libstdc++ currently does anyway)" > Issue looks resolved? That ensures that directory iterators are ranges, which is what your second for_each call depends on. The third one fails but not because of anything to do with borrowable range= s. The problem is that the filesystem::swap(path&, path&) overload is found by= ADL and considered by overload resolution when checking whether counted_iterator is swappable. Overload resolution checks whether the iterator can be converted to a path, which ca= uses constraint recursion. The following change fixes this by removing filesystem::swap(path&, path&) = from the ADL result set: --- a/libstdc++-v3/include/bits/fs_path.h +++ b/libstdc++-v3/include/bits/fs_path.h @@ -591,6 +591,8 @@ namespace __detail return __result; } + friend void swap(path& __lhs, path& __rhs) noexcept { __lhs.swap(__rhs= ); } + private: enum class _Type : unsigned char { _Multi =3D 0, _Root_name, _Root_dir, _Filename @@ -732,8 +734,6 @@ namespace __detail /// @{ /// @relates std::filesystem::path - inline void swap(path& __lhs, path& __rhs) noexcept { __lhs.swap(__rhs);= } - size_t hash_value(const path& __p) noexcept; /// @}=