mirror of
https://github.com/status-im/nim-stew.git
synced 2025-01-25 03:20:26 +00:00
4acc3866ed
It turned out that Nim's Option type is broken for `not nil` and `requiresInit` types. It will report an udesired warning due to the way `none` is defined. The `Result` type doesn't suffer from this problem, so I've transitioned some APIs in NBC to use the new `results.Opt[T]` type (alias of `Result[T, void]`). Perhaps we should renamed `Result` to `Res` to make the naming more consistent and to allow the code to more easily fit in the 80 character per-line budget.
Candidates for standard library inclusion - functions and helpers that are too small to put in their own git repository