Standardize naming of errors
It's redundant to have "Error" suffixes in each individual error case.
Also @enterprisey suggested that rather than having a weirdly named Error::Json
, it could be Error::InvalidJson
, Error::MissingField
, etc.
https://github.com/serde-rs/json/blob/master/src/error.rs#L40 has some categories that make for logical splits. Unclear if ThisError
can do it, but we don't actually gain that much from the macro anymore IMO.
This would be a breaking change.