[all-commits] [llvm/llvm-project] f1ef91: [lldb][Progress] Separate title and details (#77547)
Chelsea Cassanova via All-commits
all-commits at lists.llvm.org
Tue Jan 16 07:57:31 PST 2024
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: f1ef910b97d6acb80480b79a4144541311369cc9
https://github.com/llvm/llvm-project/commit/f1ef910b97d6acb80480b79a4144541311369cc9
Author: Chelsea Cassanova <chelsea_cassanova at apple.com>
Date: 2024-01-16 (Tue, 16 Jan 2024)
Changed paths:
M lldb/include/lldb/Core/DebuggerEvents.h
M lldb/include/lldb/Core/Progress.h
M lldb/source/Core/Progress.cpp
M lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp
M lldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp
M lldb/source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.cpp
M lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
M lldb/source/Plugins/SymbolLocator/Default/SymbolLocatorDefault.cpp
M lldb/test/API/functionalities/progress_reporting/TestProgressReporting.py
Log Message:
-----------
[lldb][Progress] Separate title and details (#77547)
Per this RFC:
https://discourse.llvm.org/t/rfc-improve-lldb-progress-reporting/75717
on improving progress reports, this commit separates the title field and
details field so that the title specifies the category that the progress
report falls under. The details field is added as a part of the
constructor for progress reports and by default is an empty string. In addition, changes the total amount of progress completed into a std::optional. Also
updates the test to check for details being correctly reported from the
event structured data dictionary.
More information about the All-commits
mailing list