Consider replacing ParseIntError usage with a new Error struct

[kaidokert]

Using ParseIntError is clunky as it can't be directly constructed, hence hacks like make_parse_int_err to make_neg_overflow_err. It could be cleaner to just have a proper Error struct for this.

Same applies for usage of core::fmt::Error