mod disjoint_mut: Remove unused trait bounds and derives

[nnethercote]

Just some unnecessary code I noticed while looking at disjoint_mut.rs.

[kkysen]

Is it important to remove the unused TranslateRange and SliceBounds impls? I added them so that we can index DisjointMuts in all of the same ways normal slices can be done. The unused ones are DCEd, right?

[kkysen]

The trait bounds we can get rid of, though.

[nnethercote]

The unused ones are DCEd, right?

Yes. It's just a code complexity/comprehensibility thing, not a performance thing. I was trying to understand some code, then I looked for how it was used, then I found it wasn't used. I'm a firm believer in generally not keeping around dead code, but this isn't my codebase, so I'm happy to defer.

[nnethercote]

I have removed the first commit.

[kkysen]

The unused ones are DCEd, right?

Yes. It's just a code complexity/comprehensibility thing, not a performance thing. I was trying to understand some code, then I looked for how it was used, then I found it wasn't used. I'm a firm believer in generally not keeping around dead code, but this isn't my codebase, so I'm happy to defer.

Ah. I think we'd like to keep these to keep the API similar to normal slices, even if all of them aren't actively used, as we may use them in the future.