[Lldb-commits] [PATCH] D29510: Remove LIBLLDB_LOG_VERBOSE category

[Jim Ingham via lldb-commits]

Not to be snarky, but that's why we put comments in headers...

Jim

> On Feb 3, 2017, at 4:47 PM, Zachary Turner via Phabricator <reviews at reviews.llvm.org> wrote:
> 
> zturner added a comment.
> 
> I guess the same way you would know how to use any part of a library or API.  The first time you've ever used an API, you don't know how it works, so you don't know it's capabilities.  So you fiddle around, read the source code, trudge through some compiler errors, then you learned some more about how it works.  Probably someone using this would try to print something, and get a compiler error that triggers a helpful static_assert which says "missing format provider for type = Foo!".  That gives them the next piece of the puzzle they need to understand a little bit more.  Or they look at the code for `formatv` to see how it works and see all the documentation.
> 
> It sounds analogous to asking "how does someone new to the codebase know that commands in LLDB are implemented by something which inherits from CommandObject?".  Of course you don't if you're brand new, but once you figure it out you don't really have to spend much time thinking about it again.
> 
> 
> https://reviews.llvm.org/D29510
> 
> 
>