You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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).
The text was updated successfully, but these errors were encountered:
Yeah, now that std has a official reverse order wrapper, it would no doubt be best to use that. I have just pushed a bunch of changes to 0.3 to update it to more modern Rust, including Reverse support.
All the
RadixHeapMap
operations requireK: Radix + Ord + Copy
. It seems the intended way to make a min-heap is via theRevOrd
wrapper, for which aRadix
implementation is defined. ButRevOrd
can’t actually be used since it doesn’t deriveCopy
.Perhaps a
Radix
impl forstd::cmp::Reverse
should be added, sinceReverse
derivesCopy
at least in 1.26 beta (rust-lang/rust#47379).The text was updated successfully, but these errors were encountered: