change `from_reader` API to match that of `serde_yaml` and `serde_json`

[droundy]

The order of the two generic arguments is swapped, which makes changing formats needlessly painful. This is obviously a breaking change, but would definitely ease use. I imagine there may be similar issues across the board. It might be even nicer for serde to define the API using a trait.