[Lldb-commits] [PATCH] D43662: [Utility] Simplify and generalize the CleanUp helper, NFC

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Feb 23 14:03:09 PST 2018


labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.

This looks great, thanks. I've been wanting to do something about the CleanUp class for a long time. I have just a couple of nits, but no need to go through review again.



================
Comment at: unittests/Utility/CleanUpTest.cpp:1
+//===-- CleanUptest.cpp -----------------------------------------*- C++ -*-===//
+//
----------------
s/test/Test/


================
Comment at: unittests/Utility/CleanUpTest.cpp:13
+
+#include <iostream>
+
----------------
Looks like this is left-over from testing.


https://reviews.llvm.org/D43662





More information about the lldb-commits mailing list