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
Unknown Object (File)
Tue, Nov 11, 6:50 AM
Unknown Object (File)
Mon, Nov 10, 2:59 AM
Unknown Object (File)
Tue, Nov 4, 5:20 AM
Unknown Object (File)
Mon, Nov 3, 11:13 PM
Unknown Object (File)
Mon, Nov 3, 10:49 PM
Unknown Object (File)
Mon, Nov 3, 10:29 PM
Unknown Object (File)
Tue, Oct 28, 3:02 AM
Unknown Object (File)
Mon, Oct 27, 6:29 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.