Skip to content

Commit 202dc12

Browse files
committed
[Reproducer] Pass FileSpec by const-ref. (NFC)
Fix two functions where we were passing FileSpecs by value, while we could pass by const reference. llvm-svn: 361459
1 parent 24374ae commit 202dc12

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

lldb/include/lldb/Utility/Reproducer.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -113,12 +113,12 @@ class FileProvider : public Provider<FileProvider> {
113113

114114
class DataRecorder {
115115
public:
116-
DataRecorder(FileSpec filename, std::error_code &ec)
116+
DataRecorder(const FileSpec &filename, std::error_code &ec)
117117
: m_filename(std::move(filename)),
118118
m_os(m_filename.GetPath(), ec, llvm::sys::fs::F_Text), m_record(true) {}
119119

120120
static llvm::Expected<std::unique_ptr<DataRecorder>>
121-
Create(FileSpec filename);
121+
Create(const FileSpec &filename);
122122

123123
template <typename T> void Record(const T &t, bool newline = false) {
124124
if (!m_record)

lldb/source/Utility/Reproducer.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ bool Loader::HasFile(StringRef file) {
221221
}
222222

223223
llvm::Expected<std::unique_ptr<DataRecorder>>
224-
DataRecorder::Create(FileSpec filename) {
224+
DataRecorder::Create(const FileSpec &filename) {
225225
std::error_code ec;
226226
auto recorder = llvm::make_unique<DataRecorder>(std::move(filename), ec);
227227
if (ec)

0 commit comments

Comments
 (0)