Skip to content

Commit

Permalink
Auto merge of #116167 - RalfJung:structural-eq, r=lcnr
Browse files Browse the repository at this point in the history
remove StructuralEq trait

The documentation given for the trait is outdated: *all* function pointers implement `PartialEq` and `Eq` these days. So the `StructuralEq` trait doesn't really seem to have any reason to exist any more.

One side-effect of this PR is that we allow matching on some consts that do not implement `Eq`. However, we already allowed matching on floats and consts containing floats, so this is not new, it is just allowed in more cases now. IMO it makes no sense at all to allow float matching but also sometimes require an `Eq` instance. If we want to require `Eq` we should adjust rust-lang/rust#115893 to check for `Eq`, and rule out float matching for good.

Fixes rust-lang/rust#115881
  • Loading branch information
bors committed Jan 26, 2024
2 parents f3d5406 + 890296b commit 48d3fb7
Showing 0 changed files with 0 additions and 0 deletions.

0 comments on commit 48d3fb7

Please sign in to comment.