lattices: `Bimorphism`s using `impl From` instead of `fn new()` can create some type inference issues #1228

MingweiSamuel · 2024-05-23T22:00:50Z

due to the From<T> for T blanket impl

The text was updated successfully, but these errors were encountered:

…ism` to prevent bad type inference, fix hydro-project#1228 BREAKING CHANGE: use `KeyedBimorphism::new` instead of `KeyedBimorphism::from`

…ism` to prevent bad type inference, fix #1228 (#1234) BREAKING CHANGE: use `KeyedBimorphism::new` instead of `KeyedBimorphism::from`

MingweiSamuel self-assigned this May 24, 2024

MingweiSamuel changed the title ~~lattices: using impl From instead of fn new() can create some type inference issues~~ lattices: Bimorphisms using impl From instead of fn new() can create some type inference issues May 24, 2024

MingweiSamuel closed this as completed in 9ba5803 May 24, 2024

rohitkulshreshtha pushed a commit that referenced this issue May 24, 2024

fix(lattices)!: use new() instead of From::from for `KeyedBimorph…

bd660e9

…ism` to prevent bad type inference, fix #1228 (#1234) BREAKING CHANGE: use `KeyedBimorphism::new` instead of `KeyedBimorphism::from`

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

lattices: `Bimorphism`s using `impl From` instead of `fn new()` can create some type inference issues #1228

lattices: `Bimorphism`s using `impl From` instead of `fn new()` can create some type inference issues #1228

MingweiSamuel commented May 23, 2024

lattices: Bimorphisms using impl From instead of fn new() can create some type inference issues #1228

lattices: Bimorphisms using impl From instead of fn new() can create some type inference issues #1228

Comments

MingweiSamuel commented May 23, 2024

lattices: `Bimorphism`s using `impl From` instead of `fn new()` can create some type inference issues #1228

lattices: `Bimorphism`s using `impl From` instead of `fn new()` can create some type inference issues #1228