RFR: 8189088: Add intrusive doubly-linked list utility [v3]

Kim Barrett kbarrett at openjdk.org
Sat Oct 7 20:45:04 UTC 2023


On Wed, 4 Oct 2023 10:46:41 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> Cursory glance.
> 
> The list hook element costs 16 bytes on 64-bit. Would it be possible to get a single-linked variant? For many (most?) cases, traversing backward or random access deletion is not needed.

That's certainly possible, but is a different RFE/PR.  IntrusiveForwardList, as a cognate for std::forward_list.

> nit, "subobject" confused me a little. Maybe "element" or "member" ?

"subobject" (C++14 1.8/2) is the correct term.  "member" would have worked,
but doesn't with the change to use a function accessor rather than a
pointer-to-data-member.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/15896#issuecomment-1751813935
PR Review Comment: https://git.openjdk.org/jdk/pull/15896#discussion_r1349572236


More information about the hotspot-dev mailing list