zturner added inline comments.

================
Comment at: unittests/Utility/Helpers/TestUtilities.cpp:17
+
+std::string lldb_private::GetInputFile(llvm::Twine name) {
+  llvm::SmallString<128> result = llvm::sys::path::parent_path(TestMainArgv0);
----------------
this should be `const Twine &name`.  Also can you call this 
`GetInputFilePath()`?  At first I was assuming it would return the contents of 
the file.


https://reviews.llvm.org/D34683



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to