Repository navigation
Modernize codebase: optional instead of out-args #109
Description
The codebase in several places uses constructions like functions that return a bool to communicate success/failure, and then actually put their output in a by-ref argument, e.g.:
template<typename Ctx> bool ToString(const Ctx& ctx, std::string& ret) constWith the code now using C++17, I think those should be retrofitted to use std::optional instead:
template<typename Ctx> std::optional<std::string> ToString(const Ctx& ctx) const(and many other examples)
Reacted by Antoine Poinsot
Activity
[-]Modernize codebase[/-][+]Modernize codebase: optional instead of out-args[/+]