`error.rs`: consider dropping negatives

[ojeda]

From @nbdd0121 #287 (comment):

We might want to drop all negatives. All C side macros define error codes as positive, and it's only used as negative values to tell part between success and failure. Rust side shouldn't use that pattern at all and should use Result exclusively anyway, so negative values are not meaningful.