[lldb] Fix grammar in error message emitted by IRExecutionUnit

The error message "Couldn't lookup symbols" emitted from IRExecutionUnit
is grammatically incorrect. "Lookup" is noun when spelled without a
space. Update the error message to use the verb "look up" instead.
This commit is contained in:
Jonas Devlieghere 2023-08-24 11:33:14 -07:00
parent 6be47fb8be
commit a902015f54
No known key found for this signature in database
GPG Key ID: 49CC0BD90FDEED4D
7 changed files with 13 additions and 12 deletions

View File

@ -288,6 +288,7 @@ class LLDB(DebuggerBase):
"use of undeclared identifier",
"no member named",
"Couldn't lookup symbols",
"Couldn't look up symbols",
"reference to local variable",
"invalid use of 'this' outside of a non-static member function",
]

View File

@ -419,7 +419,7 @@ void IRExecutionUnit::GetRunnableInfo(Status &error, lldb::addr_t &func_addr,
if (m_failed_lookups.size()) {
StreamString ss;
ss.PutCString("Couldn't lookup symbols:\n");
ss.PutCString("Couldn't look up symbols:\n");
bool emitNewLine = false;

View File

@ -38,5 +38,5 @@ class ImportStdModule(TestBase):
# Try to access our mock std::vector. This should fail but not crash LLDB as the
# std::vector template should be missing from the std module.
self.expect(
"expr (size_t)v.size()", substrs=["Couldn't lookup symbols"], error=True
"expr (size_t)v.size()", substrs=["Couldn't look up symbols"], error=True
)

View File

@ -90,7 +90,7 @@ class TestCase(TestBase):
self.expect(
"expr const int *i = &A::int_val; *i",
error=True,
substrs=["Couldn't lookup symbols:"],
substrs=["Couldn't look up symbols:"],
)
# This should work on all platforms.

View File

@ -23,14 +23,14 @@ class TestCase(TestBase):
# for them and just treats them as normal variables (which will lead
# to linker errors as they are not defined anywhere).
self.expect(
"expr A::int128_max", error=True, substrs=["Couldn't lookup symbols:"]
"expr A::int128_max", error=True, substrs=["Couldn't look up symbols:"]
)
self.expect(
"expr A::uint128_max", error=True, substrs=["Couldn't lookup symbols:"]
"expr A::uint128_max", error=True, substrs=["Couldn't look up symbols:"]
)
self.expect(
"expr A::int128_min", error=True, substrs=["Couldn't lookup symbols:"]
"expr A::int128_min", error=True, substrs=["Couldn't look up symbols:"]
)
self.expect(
"expr A::uint128_min", error=True, substrs=["Couldn't lookup symbols:"]
"expr A::uint128_min", error=True, substrs=["Couldn't look up symbols:"]
)

View File

@ -35,19 +35,19 @@ class TestCase(TestBase):
self.expect(
"expr ClassWithDefaultedCtor().foo()",
error=True,
substrs=["Couldn't lookup symbols:"],
substrs=["Couldn't look up symbols:"],
)
# FIXME: Calling deleted constructors should fail before linking.
self.expect(
"expr ClassWithDeletedCtor(1).value",
error=True,
substrs=["Couldn't lookup symbols:"],
substrs=["Couldn't look up symbols:"],
)
self.expect(
"expr ClassWithDeletedDefaultCtor().value",
error=True,
substrs=["Couldn't lookup symbols:"],
substrs=["Couldn't look up symbols:"],
)
@skipIfWindows # Can't find operator new.

View File

@ -30,12 +30,12 @@ class UniqueTypesTestCase4(TestBase):
)
self.expect(
"expression ns::FooDouble::value",
substrs=["Couldn't lookup symbols"],
substrs=["Couldn't look up symbols"],
error=True,
)
self.expect(
"expression ns::FooInt::value",
substrs=["Couldn't lookup symbols"],
substrs=["Couldn't look up symbols"],
error=True,
)