Remove unused cmake::IssueMessage overload
All callers now pass a full backtrace so we do not need the alternative that takes a cmListFileContext directly. Drop this overload to remove the code duplication.
This commit is contained in:
parent
563bf9dd8a
commit
0f96ef00cb
|
@ -2784,41 +2784,6 @@ void cmake::IssueMessage(cmake::MessageType t, std::string const& text,
|
|||
displayMessage(t, msg);
|
||||
}
|
||||
|
||||
//----------------------------------------------------------------------------
|
||||
void cmake::IssueMessage(cmake::MessageType t, std::string const& text,
|
||||
cmListFileContext const& lfc,
|
||||
bool force)
|
||||
{
|
||||
if (!force)
|
||||
{
|
||||
// override the message type, if needed, for warnings and errors
|
||||
cmake::MessageType override = this->ConvertMessageType(t);
|
||||
if (override != t)
|
||||
{
|
||||
t = override;
|
||||
force = true;
|
||||
}
|
||||
}
|
||||
|
||||
if (!force && !this->IsMessageTypeVisible(t))
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
std::ostringstream msg;
|
||||
if (!this->PrintMessagePreamble(t, msg))
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
// Add the immediate context.
|
||||
msg << (lfc.Line ? " at " : " in ") << lfc;
|
||||
|
||||
printMessageText(msg, text);
|
||||
|
||||
displayMessage(t, msg);
|
||||
}
|
||||
|
||||
//----------------------------------------------------------------------------
|
||||
std::vector<std::string> cmake::GetDebugConfigs()
|
||||
{
|
||||
|
|
|
@ -358,9 +358,6 @@ class cmake
|
|||
void IssueMessage(cmake::MessageType t, std::string const& text,
|
||||
cmListFileBacktrace const& backtrace = cmListFileBacktrace(),
|
||||
bool force = false);
|
||||
void IssueMessage(cmake::MessageType t, std::string const& text,
|
||||
cmListFileContext const& lfc,
|
||||
bool force = false);
|
||||
|
||||
///! run the --build option
|
||||
int Build(const std::string& dir,
|
||||
|
|
Loading…
Reference in New Issue