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

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Jan 17 11:02:20 PST 2018


jingham added a comment.

Seems important to me that a name tells what it does without having to know the implementation details of what it acts on.  Particularly for folks new to the code, having to know one less thing to understand how something that's important to the logic flow of the program behaves will make comprehending the code easier.  It's only a little bump to understanding, but the fewer of those we have the better, IMO.


https://reviews.llvm.org/D42182





More information about the lldb-commits mailing list