Page MenuHomeFreeBSD

atf: Switch from std::auto_ptr<> to std::unique_ptr<>
ClosedPublic

Authored by jhb on Apr 11 2025, 6:00 PM.
Tags
None
Referenced Files
F139460107: D49789.id153728.diff
Fri, Dec 12, 8:36 AM
Unknown Object (File)
Fri, Nov 28, 8:43 PM
Unknown Object (File)
Thu, Nov 27, 3:23 AM
Unknown Object (File)
Nov 11 2025, 6:50 AM
Unknown Object (File)
Nov 10 2025, 2:59 AM
Unknown Object (File)
Nov 4 2025, 5:20 AM
Unknown Object (File)
Nov 3 2025, 11:13 PM
Unknown Object (File)
Nov 3 2025, 10:49 PM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb requested review of this revision.Apr 11 2025, 6:00 PM
This revision is now accepted and ready to land.Apr 16 2025, 12:00 PM

Upstream has apparently fixed this differently using shared_ptr (not sure why, that seems wasteful and inefficient given there weren't any pointer assignments).

Upstream has apparently fixed this differently using shared_ptr

Huh, the only results for shared_ptr I see upstream are in NEWS claiming

Replace auto_ptr with std::shared_ptr (fixes modern C++ compliance).

But it's actually unique_ptr as in your review, from https://github.com/freebsd/atf/commit/f053ab687f6e27aa264f599ecbfc5ef27ad4e2d3
Pull request to update NEWS submitted: https://github.com/freebsd/atf/pull/103

Hah, cute. Ok then, that means I can still merge this and it will just fall out when doing our next vendor import.