Consider removing `thiserror`

image-rs / image-webp

Apache License 2.0

41 stars 9 forks source link

Consider removing `thiserror` #66

Closed RazrFalcon closed 5 months ago

RazrFalcon commented 5 months ago

It's a pretty heavy dependency thanks to proc-macros. By removing it we could improve compilation times and cargo vendor archive size. I plan to use this library in resvg and I care about bloat quite a lot.

kornelski commented 5 months ago

quick-error is a good replacement IMHO. It has superset of thiserror's functionality, but uses macro_rules

fintelia commented 5 months ago

I'll look into switching to quick-error. Would be nice to avoid proc macros