https://github.com/jimingham approved this pull request.

LGTM I like that you also changed "Update" to "ModuleWasUpdated", that's more 
accurate.  `m_initialized` isn't quite the right name, it means "needs work" 
not "initialized" which sounds like it should happen once.
If you can think of a better name for that off-hand, that would be a little 
clearer, but I can't think of a good one, and it's not crucial.

https://github.com/llvm/llvm-project/pull/101130
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to