subtle icon indicating copy to clipboard operation
subtle copied to clipboard

Change `ConditionallySelectable` supertrait

Open tarcieri opened this issue 1 year ago • 0 comments
trafficstars

To resolve #94, removes the Copy supertrait bound on ConditionallySelectable, replacing it with Sized instead.

It turns out the bound is only used in the default implementation of ConditionallySelectable::conditional_swap, and is easy to replace by slightly changing that default implementation.

Removing this supertrait bound is arguably a breaking change since it means types which impl ConditionallySelectable can no longer be assumed to be Copy, so also bumps the version to 3.0.0-pre.

Alternative to #118, #136

tarcieri avatar Aug 03 '24 23:08 tarcieri