================ @@ -145,15 +148,27 @@ std::string CXXFunctionSummaryFormat::GetDescription() { return std::string(sstr.GetString()); } +std::string CXXFunctionSummaryFormat::GetName() { return m_description; } + +std::string CXXFunctionSummaryFormat::GetSummaryKindName() { return "c++"; } + ScriptSummaryFormat::ScriptSummaryFormat(const TypeSummaryImpl::Flags &flags, const char *function_name, const char *python_script) : TypeSummaryImpl(Kind::eScript, flags), m_function_name(), m_python_script(), m_script_function_sp() { - if (function_name) + // Take preference in the python script name over the function name.; + if (function_name) { m_function_name.assign(function_name); - if (python_script) + m_script_formatter_name = function_name; + } + if (python_script) { m_python_script.assign(python_script); + m_script_formatter_name = python_script; + } + + m_script_formatter_name = m_script_formatter_name.erase( ---------------- Michael137 wrote:
Can we add a comment here for why this is necessary? https://github.com/llvm/llvm-project/pull/102708 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits