You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
more and more formerly centralized services done in the dispatcher are now becoming module-controlled
we used to have these processes use modules::core::Error as their error type
when we expanded stuff out to modules, we kept that error type as is
modules do whatever, and we ended up adding a bunch of wrappers in modules::core::Error to handle whatever other complaints come from modules
to break out of this practice of making more and more wrappers in modules::core::Error, we'd need a more general error type
RuntimeError (the module-code-message tuple from the oasis-core side) is general enough, but lacks the abort-ness information
module::CallResult can retain abort-ness information, but it's coupled into a Result-like enum, and it's hardcoded to have a cbor::Value payload on the Ok branch
I guess Box would be fine, but eh, extra allocations
The text was updated successfully, but these errors were encountered:
imported from a discussion on slack:
The text was updated successfully, but these errors were encountered: