mirror of
https://github.com/RGBCube/serenity
synced 2025-07-24 21:47:43 +00:00
HackStudio: Highlight register changes, minor disassembly view fix
Also adds shortcuts for step in (F11), step out (Shift-F11) and step over (F10).
This commit is contained in:
parent
16da91b7e7
commit
5fd88e51c5
4 changed files with 49 additions and 4 deletions
|
@ -52,21 +52,21 @@ void DebugInfoWidget::init_toolbar()
|
||||||
pthread_mutex_unlock(Debugger::the().continue_mutex());
|
pthread_mutex_unlock(Debugger::the().continue_mutex());
|
||||||
});
|
});
|
||||||
|
|
||||||
m_singlestep_action = GUI::Action::create("Step Over", Gfx::Bitmap::load_from_file("/res/icons/16x16/debug-step-over.png"), [&](auto&) {
|
m_singlestep_action = GUI::Action::create("Step Over", { Mod_None, Key_F10 }, Gfx::Bitmap::load_from_file("/res/icons/16x16/debug-step-over.png"), [&](auto&) {
|
||||||
pthread_mutex_lock(Debugger::the().continue_mutex());
|
pthread_mutex_lock(Debugger::the().continue_mutex());
|
||||||
Debugger::the().set_continue_type(Debugger::ContinueType::SourceStepOver);
|
Debugger::the().set_continue_type(Debugger::ContinueType::SourceStepOver);
|
||||||
pthread_cond_signal(Debugger::the().continue_cond());
|
pthread_cond_signal(Debugger::the().continue_cond());
|
||||||
pthread_mutex_unlock(Debugger::the().continue_mutex());
|
pthread_mutex_unlock(Debugger::the().continue_mutex());
|
||||||
});
|
});
|
||||||
|
|
||||||
m_step_in_action = GUI::Action::create("Step In", Gfx::Bitmap::load_from_file("/res/icons/16x16/debug-step-in.png"), [&](auto&) {
|
m_step_in_action = GUI::Action::create("Step In", { Mod_None, Key_F11 }, Gfx::Bitmap::load_from_file("/res/icons/16x16/debug-step-in.png"), [&](auto&) {
|
||||||
pthread_mutex_lock(Debugger::the().continue_mutex());
|
pthread_mutex_lock(Debugger::the().continue_mutex());
|
||||||
Debugger::the().set_continue_type(Debugger::ContinueType::SourceSingleStep);
|
Debugger::the().set_continue_type(Debugger::ContinueType::SourceSingleStep);
|
||||||
pthread_cond_signal(Debugger::the().continue_cond());
|
pthread_cond_signal(Debugger::the().continue_cond());
|
||||||
pthread_mutex_unlock(Debugger::the().continue_mutex());
|
pthread_mutex_unlock(Debugger::the().continue_mutex());
|
||||||
});
|
});
|
||||||
|
|
||||||
m_step_out_action = GUI::Action::create("Step Out", Gfx::Bitmap::load_from_file("/res/icons/16x16/debug-step-out.png"), [&](auto&) {
|
m_step_out_action = GUI::Action::create("Step Out", { Mod_Shift, Key_F11 },Gfx::Bitmap::load_from_file("/res/icons/16x16/debug-step-out.png"), [&](auto&) {
|
||||||
pthread_mutex_lock(Debugger::the().continue_mutex());
|
pthread_mutex_lock(Debugger::the().continue_mutex());
|
||||||
Debugger::the().set_continue_type(Debugger::ContinueType::SourceStepOut);
|
Debugger::the().set_continue_type(Debugger::ContinueType::SourceStepOut);
|
||||||
pthread_cond_signal(Debugger::the().continue_cond());
|
pthread_cond_signal(Debugger::the().continue_cond());
|
||||||
|
@ -168,7 +168,12 @@ void DebugInfoWidget::update_state(const Debug::DebugSession& debug_session, con
|
||||||
{
|
{
|
||||||
m_variables_view->set_model(VariablesModel::create(regs));
|
m_variables_view->set_model(VariablesModel::create(regs));
|
||||||
m_backtrace_view->set_model(BacktraceModel::create(debug_session, regs));
|
m_backtrace_view->set_model(BacktraceModel::create(debug_session, regs));
|
||||||
m_registers_view->set_model(RegistersModel::create(regs));
|
if (m_registers_view->model()) {
|
||||||
|
auto& previous_registers = static_cast<RegistersModel*>(m_registers_view->model())->raw_registers();
|
||||||
|
m_registers_view->set_model(RegistersModel::create(regs, previous_registers));
|
||||||
|
} else {
|
||||||
|
m_registers_view->set_model(RegistersModel::create(regs));
|
||||||
|
}
|
||||||
auto selected_index = m_backtrace_view->model()->index(0);
|
auto selected_index = m_backtrace_view->model()->index(0);
|
||||||
if (!selected_index.is_valid()) {
|
if (!selected_index.is_valid()) {
|
||||||
dbg() << "Warning: DebugInfoWidget: backtrace selected index is invalid";
|
dbg() << "Warning: DebugInfoWidget: backtrace selected index is invalid";
|
||||||
|
|
|
@ -81,6 +81,7 @@ void DisassemblyWidget::program_stopped()
|
||||||
{
|
{
|
||||||
m_disassembly_view->set_model({});
|
m_disassembly_view->set_model({});
|
||||||
m_function_name_label->set_text("");
|
m_function_name_label->set_text("");
|
||||||
|
hide_disassembly("Program isn't running");
|
||||||
}
|
}
|
||||||
|
|
||||||
void DisassemblyWidget::show_disassembly()
|
void DisassemblyWidget::show_disassembly()
|
||||||
|
|
|
@ -29,6 +29,7 @@
|
||||||
namespace HackStudio {
|
namespace HackStudio {
|
||||||
|
|
||||||
RegistersModel::RegistersModel(const PtraceRegisters& regs)
|
RegistersModel::RegistersModel(const PtraceRegisters& regs)
|
||||||
|
: m_raw_registers(regs)
|
||||||
{
|
{
|
||||||
m_registers.append({ "eax", regs.eax });
|
m_registers.append({ "eax", regs.eax });
|
||||||
m_registers.append({ "ebx", regs.ebx });
|
m_registers.append({ "ebx", regs.ebx });
|
||||||
|
@ -48,6 +49,27 @@ RegistersModel::RegistersModel(const PtraceRegisters& regs)
|
||||||
m_registers.append({ "gs", regs.gs });
|
m_registers.append({ "gs", regs.gs });
|
||||||
}
|
}
|
||||||
|
|
||||||
|
RegistersModel::RegistersModel(const PtraceRegisters& current_regs, const PtraceRegisters& previous_regs)
|
||||||
|
: m_raw_registers(current_regs)
|
||||||
|
{
|
||||||
|
m_registers.append({ "eax", current_regs.eax, current_regs.eax != previous_regs.eax });
|
||||||
|
m_registers.append({ "ebx", current_regs.ebx, current_regs.ebx != previous_regs.ebx });
|
||||||
|
m_registers.append({ "ecx", current_regs.ecx, current_regs.ecx != previous_regs.ecx });
|
||||||
|
m_registers.append({ "edx", current_regs.edx, current_regs.edx != previous_regs.edx });
|
||||||
|
m_registers.append({ "esp", current_regs.esp, current_regs.esp != previous_regs.esp });
|
||||||
|
m_registers.append({ "ebp", current_regs.ebp, current_regs.ebp != previous_regs.ebp });
|
||||||
|
m_registers.append({ "esi", current_regs.esi, current_regs.esi != previous_regs.esi });
|
||||||
|
m_registers.append({ "edi", current_regs.edi, current_regs.edi != previous_regs.edi });
|
||||||
|
m_registers.append({ "eip", current_regs.eip, current_regs.eip != previous_regs.eip });
|
||||||
|
m_registers.append({ "eflags", current_regs.eflags, current_regs.eflags != previous_regs.eflags });
|
||||||
|
m_registers.append({ "cs", current_regs.cs, current_regs.cs != previous_regs.cs });
|
||||||
|
m_registers.append({ "ss", current_regs.ss, current_regs.ss != previous_regs.ss });
|
||||||
|
m_registers.append({ "ds", current_regs.ds, current_regs.ds != previous_regs.ds });
|
||||||
|
m_registers.append({ "es", current_regs.es, current_regs.es != previous_regs.es });
|
||||||
|
m_registers.append({ "fs", current_regs.fs, current_regs.ds != previous_regs.fs });
|
||||||
|
m_registers.append({ "gs", current_regs.gs, current_regs.gs != previous_regs.gs });
|
||||||
|
}
|
||||||
|
|
||||||
RegistersModel::~RegistersModel()
|
RegistersModel::~RegistersModel()
|
||||||
{
|
{
|
||||||
}
|
}
|
||||||
|
@ -74,6 +96,13 @@ GUI::Variant RegistersModel::data(const GUI::ModelIndex& index, GUI::ModelRole r
|
||||||
{
|
{
|
||||||
auto& reg = m_registers[index.row()];
|
auto& reg = m_registers[index.row()];
|
||||||
|
|
||||||
|
if (role == GUI::ModelRole::ForegroundColor) {
|
||||||
|
if (reg.changed)
|
||||||
|
return Color(Color::Red);
|
||||||
|
else
|
||||||
|
return Color(Color::Black);
|
||||||
|
}
|
||||||
|
|
||||||
if (role == GUI::ModelRole::Display) {
|
if (role == GUI::ModelRole::Display) {
|
||||||
if (index.column() == Column::Register)
|
if (index.column() == Column::Register)
|
||||||
return reg.name;
|
return reg.name;
|
||||||
|
|
|
@ -35,6 +35,7 @@ namespace HackStudio {
|
||||||
struct RegisterData {
|
struct RegisterData {
|
||||||
String name;
|
String name;
|
||||||
u32 value;
|
u32 value;
|
||||||
|
bool changed { false };
|
||||||
};
|
};
|
||||||
|
|
||||||
class RegistersModel final : public GUI::Model {
|
class RegistersModel final : public GUI::Model {
|
||||||
|
@ -44,6 +45,11 @@ public:
|
||||||
return adopt(*new RegistersModel(regs));
|
return adopt(*new RegistersModel(regs));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static RefPtr<RegistersModel> create(const PtraceRegisters& current_regs, const PtraceRegisters& previous_regs)
|
||||||
|
{
|
||||||
|
return adopt(*new RegistersModel(current_regs, previous_regs));
|
||||||
|
}
|
||||||
|
|
||||||
enum Column {
|
enum Column {
|
||||||
Register,
|
Register,
|
||||||
Value,
|
Value,
|
||||||
|
@ -58,9 +64,13 @@ public:
|
||||||
virtual GUI::Variant data(const GUI::ModelIndex&, GUI::ModelRole) const override;
|
virtual GUI::Variant data(const GUI::ModelIndex&, GUI::ModelRole) const override;
|
||||||
virtual void update() override;
|
virtual void update() override;
|
||||||
|
|
||||||
|
const PtraceRegisters& raw_registers() const { return m_raw_registers; }
|
||||||
|
|
||||||
private:
|
private:
|
||||||
explicit RegistersModel(const PtraceRegisters& regs);
|
explicit RegistersModel(const PtraceRegisters& regs);
|
||||||
|
RegistersModel(const PtraceRegisters& current_regs, const PtraceRegisters& previous_regs);
|
||||||
|
|
||||||
|
PtraceRegisters m_raw_registers;
|
||||||
Vector<RegisterData> m_registers;
|
Vector<RegisterData> m_registers;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue