shafik added inline comments.
================
Comment at: lldb/include/lldb/Symbol/ClangASTImporter.h:247
+ struct StdModuleScope {
+ StdModuleHandler m_handler;
----------------
aprantl wrote:
> Can you add a `///` one-liner for this class?
This definitely needs a description, it took me a bit to understand what its
purpose was and it feels a little subtle. Something along the lines of "RAII
class to ensure that the StdModuleHandler is properly set and unset...."
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59537/new/
https://reviews.llvm.org/D59537
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
