[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
|
|
|
//===-- ProcessInstanceInfoTest.cpp ---------------------------------------===//
|
2019-03-05 02:48:00 +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
|
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
|
|
|
#include "lldb/Target/Process.h"
|
|
|
|
#include "gtest/gtest.h"
|
|
|
|
|
|
|
|
using namespace lldb_private;
|
|
|
|
|
|
|
|
namespace {
|
|
|
|
/// A very simple resolver which fails for even ids and returns a simple string
|
|
|
|
/// for odd ones.
|
|
|
|
class DummyUserIDResolver : public UserIDResolver {
|
|
|
|
protected:
|
C.128 override, virtual keyword handling
Summary:
According to [C128] "Virtual functions should specify exactly one
of `virtual`, `override`, or `final`", I've added override where a
virtual function is overriden but the explicit `override` keyword
was missing. Whenever both `virtual` and `override` were specified,
I removed `virtual`. As C.128 puts it:
> [...] writing more than one of these three is both redundant and
> a potential source of errors.
I anticipate a discussion about whether or not to add `override` to
destructors but I went for it because of an example in [ISOCPP1000].
Let me repeat the comment for you here:
Consider this code:
```
struct Base {
virtual ~Base(){}
};
struct SubClass : Base {
~SubClass() {
std::cout << "It works!\n";
}
};
int main() {
std::unique_ptr<Base> ptr = std::make_unique<SubClass>();
}
```
If for some odd reason somebody removes the `virtual` keyword from the
`Base` struct, the code will no longer print `It works!`. So adding
`override` to destructors actively protects us from accidentally
breaking our code at runtime.
[C128]: https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#c128-virtual-functions-should-specify-exactly-one-of-virtual-override-or-final
[ISOCPP1000]: https://github.com/isocpp/CppCoreGuidelines/issues/1000#issuecomment-476951555
Reviewers: teemperor, JDevlieghere, davide, shafik
Reviewed By: teemperor
Subscribers: kwk, arphaman, kadircet, lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D61440
llvm-svn: 359868
2019-05-03 18:03:28 +08:00
|
|
|
llvm::Optional<std::string> DoGetUserName(id_t uid) override {
|
2019-03-05 02:48:00 +08:00
|
|
|
if (uid % 2)
|
|
|
|
return ("user" + llvm::Twine(uid)).str();
|
|
|
|
return llvm::None;
|
|
|
|
}
|
|
|
|
|
C.128 override, virtual keyword handling
Summary:
According to [C128] "Virtual functions should specify exactly one
of `virtual`, `override`, or `final`", I've added override where a
virtual function is overriden but the explicit `override` keyword
was missing. Whenever both `virtual` and `override` were specified,
I removed `virtual`. As C.128 puts it:
> [...] writing more than one of these three is both redundant and
> a potential source of errors.
I anticipate a discussion about whether or not to add `override` to
destructors but I went for it because of an example in [ISOCPP1000].
Let me repeat the comment for you here:
Consider this code:
```
struct Base {
virtual ~Base(){}
};
struct SubClass : Base {
~SubClass() {
std::cout << "It works!\n";
}
};
int main() {
std::unique_ptr<Base> ptr = std::make_unique<SubClass>();
}
```
If for some odd reason somebody removes the `virtual` keyword from the
`Base` struct, the code will no longer print `It works!`. So adding
`override` to destructors actively protects us from accidentally
breaking our code at runtime.
[C128]: https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#c128-virtual-functions-should-specify-exactly-one-of-virtual-override-or-final
[ISOCPP1000]: https://github.com/isocpp/CppCoreGuidelines/issues/1000#issuecomment-476951555
Reviewers: teemperor, JDevlieghere, davide, shafik
Reviewed By: teemperor
Subscribers: kwk, arphaman, kadircet, lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D61440
llvm-svn: 359868
2019-05-03 18:03:28 +08:00
|
|
|
llvm::Optional<std::string> DoGetGroupName(id_t gid) override {
|
2019-03-05 02:48:00 +08:00
|
|
|
if (gid % 2)
|
|
|
|
return ("group" + llvm::Twine(gid)).str();
|
|
|
|
return llvm::None;
|
|
|
|
}
|
|
|
|
};
|
|
|
|
} // namespace
|
|
|
|
|
|
|
|
TEST(ProcessInstanceInfo, Dump) {
|
|
|
|
ProcessInstanceInfo info("a.out", ArchSpec("x86_64-pc-linux"), 47);
|
|
|
|
info.SetUserID(1);
|
|
|
|
info.SetEffectiveUserID(2);
|
|
|
|
info.SetGroupID(3);
|
|
|
|
info.SetEffectiveGroupID(4);
|
|
|
|
|
|
|
|
DummyUserIDResolver resolver;
|
|
|
|
StreamString s;
|
|
|
|
info.Dump(s, resolver);
|
|
|
|
EXPECT_STREQ(R"( pid = 47
|
|
|
|
name = a.out
|
|
|
|
file = a.out
|
|
|
|
arch = x86_64-pc-linux
|
|
|
|
uid = 1 (user1)
|
|
|
|
gid = 3 (group3)
|
|
|
|
euid = 2 ()
|
|
|
|
egid = 4 ()
|
|
|
|
)",
|
|
|
|
s.GetData());
|
|
|
|
}
|
|
|
|
|
|
|
|
TEST(ProcessInstanceInfo, DumpTable) {
|
|
|
|
ProcessInstanceInfo info("a.out", ArchSpec("x86_64-pc-linux"), 47);
|
|
|
|
info.SetUserID(1);
|
|
|
|
info.SetEffectiveUserID(2);
|
|
|
|
info.SetGroupID(3);
|
|
|
|
info.SetEffectiveGroupID(4);
|
|
|
|
|
|
|
|
DummyUserIDResolver resolver;
|
|
|
|
StreamString s;
|
|
|
|
|
|
|
|
const bool show_args = false;
|
|
|
|
const bool verbose = true;
|
|
|
|
ProcessInstanceInfo::DumpTableHeader(s, show_args, verbose);
|
|
|
|
info.DumpAsTableRow(s, resolver, show_args, verbose);
|
|
|
|
EXPECT_STREQ(
|
2019-10-04 05:57:01 +08:00
|
|
|
R"(PID PARENT USER GROUP EFF USER EFF GROUP TRIPLE ARGUMENTS
|
|
|
|
====== ====== ========== ========== ========== ========== ============================== ============================
|
|
|
|
47 0 user1 group3 2 4 x86_64-pc-linux
|
2019-03-05 02:48:00 +08:00
|
|
|
)",
|
|
|
|
s.GetData());
|
|
|
|
}
|
2019-08-26 21:03:21 +08:00
|
|
|
|
|
|
|
TEST(ProcessInstanceInfo, DumpTable_invalidUID) {
|
2019-10-04 05:57:01 +08:00
|
|
|
ProcessInstanceInfo info("a.out", ArchSpec("aarch64-unknown-linux-android"), 47);
|
2019-08-26 21:03:21 +08:00
|
|
|
|
|
|
|
DummyUserIDResolver resolver;
|
|
|
|
StreamString s;
|
|
|
|
|
|
|
|
const bool show_args = false;
|
|
|
|
const bool verbose = false;
|
|
|
|
ProcessInstanceInfo::DumpTableHeader(s, show_args, verbose);
|
|
|
|
info.DumpAsTableRow(s, resolver, show_args, verbose);
|
|
|
|
EXPECT_STREQ(
|
2019-10-04 05:57:01 +08:00
|
|
|
R"(PID PARENT USER TRIPLE NAME
|
|
|
|
====== ====== ========== ============================== ============================
|
|
|
|
47 0 aarch64-unknown-linux-android a.out
|
2019-08-26 21:03:21 +08:00
|
|
|
)",
|
|
|
|
s.GetData());
|
|
|
|
}
|
2019-10-11 18:56:54 +08:00
|
|
|
|
|
|
|
TEST(ProcessInstanceInfoMatch, Name) {
|
|
|
|
ProcessInstanceInfo info_bar, info_empty;
|
|
|
|
info_bar.GetExecutableFile().SetFile("/foo/bar", FileSpec::Style::posix);
|
|
|
|
|
|
|
|
ProcessInstanceInfoMatch match;
|
|
|
|
match.SetNameMatchType(NameMatch::Equals);
|
|
|
|
match.GetProcessInfo().GetExecutableFile().SetFile("bar",
|
|
|
|
FileSpec::Style::posix);
|
|
|
|
|
|
|
|
EXPECT_TRUE(match.Matches(info_bar));
|
|
|
|
EXPECT_FALSE(match.Matches(info_empty));
|
|
|
|
|
|
|
|
match.GetProcessInfo().GetExecutableFile() = FileSpec();
|
|
|
|
EXPECT_TRUE(match.Matches(info_bar));
|
|
|
|
EXPECT_TRUE(match.Matches(info_empty));
|
|
|
|
}
|
2020-03-13 05:34:44 +08:00
|
|
|
|
|
|
|
TEST(ProcessInstanceInfo, Yaml) {
|
|
|
|
std::string buffer;
|
|
|
|
llvm::raw_string_ostream os(buffer);
|
|
|
|
|
|
|
|
// Serialize.
|
|
|
|
ProcessInstanceInfo info("a.out", ArchSpec("x86_64-pc-linux"), 47);
|
|
|
|
info.SetUserID(1);
|
|
|
|
info.SetEffectiveUserID(2);
|
|
|
|
info.SetGroupID(3);
|
|
|
|
info.SetEffectiveGroupID(4);
|
|
|
|
llvm::yaml::Output yout(os);
|
|
|
|
yout << info;
|
|
|
|
os.flush();
|
|
|
|
|
|
|
|
// Deserialize.
|
|
|
|
ProcessInstanceInfo deserialized;
|
|
|
|
llvm::yaml::Input yin(buffer);
|
|
|
|
yin >> deserialized;
|
|
|
|
|
|
|
|
EXPECT_EQ(deserialized.GetNameAsStringRef(), info.GetNameAsStringRef());
|
|
|
|
EXPECT_EQ(deserialized.GetArchitecture(), info.GetArchitecture());
|
|
|
|
EXPECT_EQ(deserialized.GetUserID(), info.GetUserID());
|
|
|
|
EXPECT_EQ(deserialized.GetGroupID(), info.GetGroupID());
|
|
|
|
EXPECT_EQ(deserialized.GetEffectiveUserID(), info.GetEffectiveUserID());
|
|
|
|
EXPECT_EQ(deserialized.GetEffectiveGroupID(), info.GetEffectiveGroupID());
|
|
|
|
}
|
|
|
|
|
|
|
|
TEST(ProcessInstanceInfoList, Yaml) {
|
|
|
|
std::string buffer;
|
|
|
|
llvm::raw_string_ostream os(buffer);
|
|
|
|
|
|
|
|
// Serialize.
|
|
|
|
ProcessInstanceInfo info("a.out", ArchSpec("x86_64-pc-linux"), 47);
|
|
|
|
info.SetUserID(1);
|
|
|
|
info.SetEffectiveUserID(2);
|
|
|
|
info.SetGroupID(3);
|
|
|
|
info.SetEffectiveGroupID(4);
|
|
|
|
ProcessInstanceInfoList list;
|
|
|
|
list.push_back(info);
|
|
|
|
llvm::yaml::Output yout(os);
|
|
|
|
yout << list;
|
|
|
|
os.flush();
|
|
|
|
|
|
|
|
// Deserialize.
|
|
|
|
ProcessInstanceInfoList deserialized;
|
|
|
|
llvm::yaml::Input yin(buffer);
|
|
|
|
yin >> deserialized;
|
|
|
|
|
|
|
|
ASSERT_EQ(deserialized.size(), static_cast<size_t>(1));
|
|
|
|
EXPECT_EQ(deserialized[0].GetNameAsStringRef(), info.GetNameAsStringRef());
|
|
|
|
EXPECT_EQ(deserialized[0].GetArchitecture(), info.GetArchitecture());
|
|
|
|
EXPECT_EQ(deserialized[0].GetUserID(), info.GetUserID());
|
|
|
|
EXPECT_EQ(deserialized[0].GetGroupID(), info.GetGroupID());
|
|
|
|
EXPECT_EQ(deserialized[0].GetEffectiveUserID(), info.GetEffectiveUserID());
|
|
|
|
EXPECT_EQ(deserialized[0].GetEffectiveGroupID(), info.GetEffectiveGroupID());
|
|
|
|
}
|