[Lldb-commits] [PATCH] D42182: Add LLDB_LOG_ERROR (?)

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Jan 17 10:31:47 PST 2018


jingham added a comment.

This seems fine as a bit of functionality but I'm worried about the name.  There's nothing in "LLDB_LOG_ERROR" that indicates that the error gets cleared, but that's actually a pretty important piece of its business.  Before this propagates further is there a better name we can come up with that's not too horrible to type or scan but conveys this?  LLDB_LOG_AND_CLEAR_ERROR is not too bad, though if there's a better single word that conveys logging & handling that would be nicer.  But I can't think of one off the top of my head.


https://reviews.llvm.org/D42182





More information about the lldb-commits mailing list