Upcoming `Result` review - comments and thoughts welcome!

[shirleyquirk]

I find myself overloading `valueOr` to work with `Result[void,E]` what's the 
reason for leaving that out?

How much work would it be to avoid copies when possible? It feels very 
edgecasey, pattern matching whether `expr` in `?expr` is an nnkCall etc