search

vacp2p / nim-quic

QUIC for Nim
40 stars 3 forks source link

Remove Questionable #35

Closed Menduist closed 2 years ago

Menduist commented 2 years ago

or "why Questionable is awesome"

ref https://github.com/status-im/nim-libp2p/pull/725#issuecomment-1204149877

I tried to introduce a light version of Questionable, but just handling without hello =? someOption is already quite heavy (macros, templates, procs..) So instead I made my own bad version :)

arnetheduck commented 2 years ago

instead of std/options (which has fundamental issues / will be replaced soon or broken incompatibly), we're moving most code over to results.Opt instead - it would probably make sense to follow that trend here also