[lldb][NFC] Fix all formatting errors in .cpp file headers
Summary:
A *.cpp file header in LLDB (and in LLDB) should like this:
```
//===-- TestUtilities.cpp -------------------------------------------------===//
```
However in LLDB most of our source files have arbitrary changes to this format and
these changes are spreading through LLDB as folks usually just use the existing
source files as templates for their new files (most notably the unnecessary
editor language indicator `-*- C++ -*-` is spreading and in every review
someone is pointing out that this is wrong, resulting in people pointing out that this
is done in the same way in other files).
This patch removes most of these inconsistencies including the editor language indicators,
all the different missing/additional '-' characters, files that center the file name, missing
trailing `===//` (mostly caused by clang-format breaking the line).
Reviewers: aprantl, espindola, jfb, shafik, JDevlieghere
Reviewed By: JDevlieghere
Subscribers: dexonsmith, wuzish, emaste, sdardis, nemanjai, kbarton, MaskRay, atanasyan, arphaman, jfb, abidh, jsji, JDevlieghere, usaxena95, lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D73258
2020-01-24 15:23:27 +08:00
|
|
|
//===-- TestUtilities.cpp -------------------------------------------------===//
|
2017-06-29 21:02:11 +08:00
|
|
|
//
|
2019-01-19 16:50:56 +08:00
|
|
|
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
|
|
|
|
// See https://llvm.org/LICENSE.txt for license information.
|
|
|
|
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
|
2017-06-29 21:02:11 +08:00
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
|
|
|
#include "TestUtilities.h"
|
|
|
|
#include "llvm/ADT/SmallString.h"
|
unittests: Use yaml2obj as a library instead of an external process
Summary:
Recently, yaml2obj has been turned into a library. This means we can use
it from our unit tests directly, instead of shelling out to an external
process. This patch does just that.
Reviewers: JDevlieghere, aadsm, espindola, jdoerfert
Subscribers: emaste, mgorny, arichardson, MaskRay, jhenderson, abrachet, lldb-commits
Differential Revision: https://reviews.llvm.org/D65949
llvm-svn: 369374
2019-08-20 20:28:36 +08:00
|
|
|
#include "llvm/ObjectYAML/yaml2obj.h"
|
2017-06-29 21:02:11 +08:00
|
|
|
#include "llvm/Support/FileSystem.h"
|
|
|
|
#include "llvm/Support/Path.h"
|
2019-05-07 04:01:21 +08:00
|
|
|
#include "llvm/Support/Program.h"
|
unittests: Use yaml2obj as a library instead of an external process
Summary:
Recently, yaml2obj has been turned into a library. This means we can use
it from our unit tests directly, instead of shelling out to an external
process. This patch does just that.
Reviewers: JDevlieghere, aadsm, espindola, jdoerfert
Subscribers: emaste, mgorny, arichardson, MaskRay, jhenderson, abrachet, lldb-commits
Differential Revision: https://reviews.llvm.org/D65949
llvm-svn: 369374
2019-08-20 20:28:36 +08:00
|
|
|
#include "llvm/Support/YAMLTraits.h"
|
|
|
|
#include "gtest/gtest.h"
|
|
|
|
|
|
|
|
using namespace lldb_private;
|
2017-06-29 21:02:11 +08:00
|
|
|
|
|
|
|
extern const char *TestMainArgv0;
|
|
|
|
|
|
|
|
std::string lldb_private::GetInputFilePath(const llvm::Twine &name) {
|
|
|
|
llvm::SmallString<128> result = llvm::sys::path::parent_path(TestMainArgv0);
|
|
|
|
llvm::sys::fs::make_absolute(result);
|
|
|
|
llvm::sys::path::append(result, "Inputs", name);
|
2020-01-29 03:23:46 +08:00
|
|
|
return std::string(result.str());
|
2017-06-29 21:02:11 +08:00
|
|
|
}
|
2019-05-07 04:01:21 +08:00
|
|
|
|
unittests: Use yaml2obj as a library instead of an external process
Summary:
Recently, yaml2obj has been turned into a library. This means we can use
it from our unit tests directly, instead of shelling out to an external
process. This patch does just that.
Reviewers: JDevlieghere, aadsm, espindola, jdoerfert
Subscribers: emaste, mgorny, arichardson, MaskRay, jhenderson, abrachet, lldb-commits
Differential Revision: https://reviews.llvm.org/D65949
llvm-svn: 369374
2019-08-20 20:28:36 +08:00
|
|
|
llvm::Expected<TestFile> TestFile::fromYaml(llvm::StringRef Yaml) {
|
|
|
|
const auto *Info = testing::UnitTest::GetInstance()->current_test_info();
|
|
|
|
assert(Info);
|
|
|
|
llvm::SmallString<128> Name;
|
|
|
|
int FD;
|
|
|
|
if (std::error_code EC = llvm::sys::fs::createTemporaryFile(
|
|
|
|
llvm::Twine(Info->test_case_name()) + "-" + Info->name(), "test", FD,
|
|
|
|
Name))
|
|
|
|
return llvm::errorCodeToError(EC);
|
|
|
|
llvm::FileRemover Remover(Name);
|
|
|
|
{
|
|
|
|
llvm::raw_fd_ostream OS(FD, /*shouldClose*/ true);
|
|
|
|
llvm::yaml::Input YIn(Yaml);
|
2019-09-14 00:00:28 +08:00
|
|
|
if (!llvm::yaml::convertYAML(YIn, OS, [](const llvm::Twine &Msg) {}))
|
|
|
|
return llvm::createStringError(llvm::inconvertibleErrorCode(),
|
|
|
|
"convertYAML() failed");
|
unittests: Use yaml2obj as a library instead of an external process
Summary:
Recently, yaml2obj has been turned into a library. This means we can use
it from our unit tests directly, instead of shelling out to an external
process. This patch does just that.
Reviewers: JDevlieghere, aadsm, espindola, jdoerfert
Subscribers: emaste, mgorny, arichardson, MaskRay, jhenderson, abrachet, lldb-commits
Differential Revision: https://reviews.llvm.org/D65949
llvm-svn: 369374
2019-08-20 20:28:36 +08:00
|
|
|
}
|
|
|
|
return TestFile(Name, std::move(Remover));
|
|
|
|
}
|
|
|
|
|
|
|
|
llvm::Expected<TestFile> TestFile::fromYamlFile(const llvm::Twine &Name) {
|
|
|
|
auto BufferOrError =
|
|
|
|
llvm::MemoryBuffer::getFile(GetInputFilePath(Name), /*FileSize*/ -1,
|
|
|
|
/*RequiresNullTerminator*/ false);
|
|
|
|
if (!BufferOrError)
|
|
|
|
return llvm::errorCodeToError(BufferOrError.getError());
|
|
|
|
return fromYaml(BufferOrError.get()->getBuffer());
|
|
|
|
}
|
|
|
|
|
|
|
|
TestFile::~TestFile() {
|
|
|
|
if (!Name)
|
|
|
|
return;
|
|
|
|
if (std::error_code EC =
|
|
|
|
llvm::sys::fs::remove(*Name, /*IgnoreNonExisting*/ false))
|
|
|
|
GTEST_LOG_(WARNING) << "Failed to delete `" << Name->c_str()
|
|
|
|
<< "`: " << EC.message();
|
2019-07-25 01:56:10 +08:00
|
|
|
}
|