Add [[nodiscard]] attribute

[PJK]

c2x introduces language-level nodiscard attribute. We should use it in a backwards-compatible way for all functions that can fail, not checking the return value has caused bug even internally (e.g. #246)

https://clang.llvm.org/docs/AttributeReference.html