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
F151254084: D49789.diff
Tue, Apr 7, 3:35 AM
Unknown Object (File)
Sat, Apr 4, 2:10 PM
Unknown Object (File)
Sun, Mar 29, 6:21 AM
Unknown Object (File)
Sat, Mar 28, 11:54 AM
Unknown Object (File)
Sun, Mar 15, 11:16 PM
Unknown Object (File)
Thu, Mar 12, 7:51 PM
Unknown Object (File)
Mar 3 2026, 3:19 PM
Unknown Object (File)
Mar 1 2026, 6:55 AM
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.