[PATCH] D14348: Add Printable class to simplify the construction of Print helpers.
Matthias Braun via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 3 13:52:16 PST 2015
MatzeB updated this revision to Diff 41799.
MatzeB added a comment.
Updated version which uses std::function<void(raw_ostream&)> similar to C++ IO manipulators directly instead of adding a pointless shim class around the std::function.
I still found the code slightly easier to read when giving the concept an explicit name with "typedef std::function<void(raw_ostream&)> Printable", though I have no strong opinion here and can change that if you prefer the plain form without typedef.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 8586 bytes
Desc: not available
More information about the llvm-commits