labath accepted this revision. labath added a comment. This revision is now accepted and ready to land.
This looks fine. When you see a pattern like `if(log) { LLDB_LOGF(...) }`, feel free to delete the surrounding `if(log)`. It is no longer necessary, and the reason it is still there is simply because Jonas's vim script wasn't smart enough to delete it in some cases. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67896/new/ https://reviews.llvm.org/D67896 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits