[PATCH] D20592: Add FIXMEs to all derived classes of std::error_category.

Lang Hames via llvm-commits llvm-commits at lists.llvm.org
Tue May 24 13:01:19 PDT 2016


lhames added a comment.

Oh - I see this was motivated by http://reviews.llvm.org/D20550.

Maybe a re-worded FIXME would work:

  // FIXME: This class is only here to support the transition to llvm::Error. It will be
  // removed once this transition is complete. Clients should prefer to deal with the
  // Error value directly, rather than converting to error_code.  

It is a bit more verbose though.


http://reviews.llvm.org/D20592





More information about the llvm-commits mailing list