[haiku-commits] Re: haiku: hrev44499 - src/apps/debugger/debugger_interface
- From: Ingo Weinhold <ingo_weinhold@xxxxxx>
- To: haiku-commits@xxxxxxxxxxxxx
- Date: Thu, 09 Aug 2012 13:32:06 +0200
On 2012-08-09 at 01:59:53 [+0200], anevilyak@xxxxxxxxx wrote:
> diff --git a/src/apps/debugger/debugger_interface/DebuggerInterface.cpp
> b/src/apps/debugger/debugger_interface/DebuggerInterface.cpp
> index ae3eb33..2504389 100644
> --- a/src/apps/debugger/debugger_interface/DebuggerInterface.cpp
> +++ b/src/apps/debugger/debugger_interface/DebuggerInterface.cpp
[...]
> @@ -251,6 +252,9 @@ status_t
> DebuggerInterface::Init()
> {
> // create the architecture
> + // TODO: this probably needs to be rethought a bit,
> + // since especially when we eventually support remote debugging,
> + // the architecture will depend on the target machine, not the host
> #ifdef ARCH_x86
> fArchitecture = new(std::nothrow) ArchitectureX86(this);
> #else
Well, actually there is little to be rethought as the plan for this class is
pretty clear. It will remain as it is with the exception that it will be
renamed to LocalDebugInterface and a DebugInterface interface will be pulled
out of it. A RemoteDebugInterface implementation will be added for the remote
debugging support.
CU, Ingo
Other related posts:
