Skip to content

impl Radix for RevOrd isn’t helpful since RevOrd isn’t Copy #1

Closed
@andersk

Description

@andersk

All the RadixHeapMap operations require K: Radix + Ord + Copy. It seems the intended way to make a min-heap is via the RevOrd wrapper, for which a Radix implementation is defined. But RevOrd can’t actually be used since it doesn’t derive Copy.

Perhaps a Radix impl for std::cmp::Reverse should be added, since Reverse derives Copy at least in 1.26 beta (rust-lang/rust#47379).

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions