[Lldb-commits] [PATCH] D20570: Add unit tests for ModuleCache

[Pavel Labath via lldb-commits]

labath added inline comments.

================
Comment at: unittests/Utility/ModuleCacheTest.cpp:74
@@ +73,3 @@
+{
+    HostInfo::Initialize();
+    ObjectFileELF::Initialize();
----------------
ovyalov wrote:
> If it's one-off initialization can we use setUpTestCase for these purposes (for HostInfo::Initialize and ObjectFileELF::Initialize)?
Good point. I have moved this to the static method (including the variable initialization below, as they are not modified). I have also added the corresponding tearDown method.


Repository:
  rL LLVM

http://reviews.llvm.org/D20570