Switched all `Into` for DataType to `From`

mit-pdos / noria

Fast web applications through dynamic, partially-stateful dataflow

Apache License 2.0

4.99k stars 242 forks source link

Switched all `Into` for DataType to `From` #161

Closed jonathanGB closed 4 years ago

jonathanGB commented 4 years ago

It is preferred to implement From over Into, per the Rust documentation. Thus, this PR switches all Intos to Froms, for DataType specifically.

At the same time, I added some conversions for DataType that were only available for &'_ DataType.

jonhoo commented 4 years ago

Excellent! Let's squash and merge when CI passes :)