search

ex0dus-0x / structmap

Procedural macro crate for conversion between Rust structs and associative containers.
MIT License
58 stars 10 forks source link

Implemented StringMap and Removed Panic #8

Closed pedrosoares closed 2 years ago

pedrosoares commented 2 years ago

Implemented from_stringmap functionality. Removed Panic from struct field not found, panic at this point its very hard to handle.

Shoult i add a new parameter to configure the panic handling or use Result as return instead the Struct itself?

ex0dus-0x commented 2 years ago

This looks really good! Thank you for all the substantial changes and apologies for not checking back on this.

I think having a Result be returned if a panic behavior occurs is the better pattern. We can move that to a separate pull request.