Skip to content

Commit 08dfa8b

Browse files
committed
[lldb] add test
1 parent 1639f6a commit 08dfa8b

File tree

4 files changed

+15
-3
lines changed

4 files changed

+15
-3
lines changed

lldb/include/lldb/Target/UnixSignals.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ class UnixSignals {
3131

3232
llvm::StringRef GetSignalAsStringRef(int32_t signo) const;
3333

34-
llvm::StringRef GetSignalNoDescription(int32_t signo) const;
34+
llvm::StringRef GetSignalNumberDescription(int32_t signo) const;
3535

3636
std::string
3737
GetSignalDescription(int32_t signo,

lldb/source/Commands/CommandObjectProcess.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1620,7 +1620,7 @@ class CommandObjectProcessHandle : public CommandObjectParsed {
16201620
(stop ? "true " : "false"), (notify ? "true " : "false"));
16211621

16221622
llvm::StringRef sig_description =
1623-
signals_sp->GetSignalNoDescription(signo);
1623+
signals_sp->GetSignalNumberDescription(signo);
16241624
if (!sig_description.empty()) {
16251625
str.PutCString(" ");
16261626
str.PutCString(sig_description);

lldb/source/Target/UnixSignals.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ llvm::StringRef UnixSignals::GetSignalAsStringRef(int32_t signo) const {
137137
return pos->second.m_name;
138138
}
139139

140-
llvm::StringRef UnixSignals::GetSignalNoDescription(int32_t signo) const {
140+
llvm::StringRef UnixSignals::GetSignalNumberDescription(int32_t signo) const {
141141
const auto pos = m_signals.find(signo);
142142
if (pos == m_signals.end())
143143
return {};

lldb/unittests/Signals/UnixSignalsTest.cpp

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -148,6 +148,18 @@ TEST(UnixSignalsTest, GetAsString) {
148148
signals.GetSignalDescription(16, 3, 0x1233, 0x1234, 0x5678));
149149
}
150150

151+
TEST(UnixSignalsTest, GetNumberDescription) {
152+
TestSignals signals;
153+
154+
ASSERT_EQ("DESC2", signals.GetSignalNumberDescription(2));
155+
ASSERT_EQ("DESC4", signals.GetSignalNumberDescription(4));
156+
ASSERT_EQ("DESC8", signals.GetSignalNumberDescription(8));
157+
ASSERT_EQ("DESC16", signals.GetSignalNumberDescription(16));
158+
159+
// Unknown signal number.
160+
ASSERT_EQ("", signals.GetSignalNumberDescription(100));
161+
}
162+
151163
TEST(UnixSignalsTest, VersionChange) {
152164
TestSignals signals;
153165

0 commit comments

Comments
 (0)